diff options
| author | nsylvain@chromium.org <nsylvain@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-10-19 23:47:51 +0000 |
|---|---|---|
| committer | nsylvain@chromium.org <nsylvain@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-10-19 23:47:51 +0000 |
| commit | cdc491e034aa38aaace2fea31a4f412709c4d21d (patch) | |
| tree | c6a7fad211ece4986bf6219e1c900299f9ac9103 /tools | |
| parent | e3de01ae77f4a4f116d9a6d42c1f6b4e22923a9e (diff) | |
| download | chromium_src-cdc491e034aa38aaace2fea31a4f412709c4d21d.zip chromium_src-cdc491e034aa38aaace2fea31a4f412709c4d21d.tar.gz chromium_src-cdc491e034aa38aaace2fea31a4f412709c4d21d.tar.bz2 | |
Don't lower the case for the base path anymore. This is not needed and
this makes it fail on Linux.
BUG:None
TEST:checkperms still succeeds.
Review URL: http://codereview.chromium.org/3927001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63134 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'tools')
| -rwxr-xr-x | tools/checkperms/checkperms.py | 9 |
1 files changed, 2 insertions, 7 deletions
diff --git a/tools/checkperms/checkperms.py b/tools/checkperms/checkperms.py index a396c55..abd48b6 100755 --- a/tools/checkperms/checkperms.py +++ b/tools/checkperms/checkperms.py @@ -101,8 +101,8 @@ WHITELIST_FILES_REGEX = re.compile(r'(%s)$' % '|'.join(WHITELIST_FILES)) # Set to true for more output. This is set by the command line options. VERBOSE = False -# In lowercase, using forward slashes as directory separators, ending in a -# forward slash. Set by the command line options. +# Using forward slashes as directory separators, ending in a forward slash. +# Set by the command line options. BASE_DIRECTORY = '' # The default if BASE_DIRECTORY is not set on the command line. @@ -292,11 +292,6 @@ Examples: print 'Using base directory:', BASE_DIRECTORY print 'Checking directory:', start_dir - # The base directory should be lower case from here on since it will be used - # for substring matching against whitelists that all assume lowercase, and we - # compile on case-insensitive systems. Plus, we always use slashes here since - # the include parsing code will also normalize to slashes. - BASE_DIRECTORY = BASE_DIRECTORY.lower() BASE_DIRECTORY = BASE_DIRECTORY.replace('\\', '/') start_dir = start_dir.replace('\\', '/') |