Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(449)

Side by Side Diff: tools/checkperms/checkperms.py

Issue 11227046: Add myself as an owner for tools/checkperms. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 8 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « tools/checkperms/OWNERS ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 #!/usr/bin/env python 1 #!/usr/bin/env python
2 # Copyright (c) 2012 The Chromium Authors. All rights reserved. 2 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
3 # Use of this source code is governed by a BSD-style license that can be 3 # Use of this source code is governed by a BSD-style license that can be
4 # found in the LICENSE file. 4 # found in the LICENSE file.
5 5
6 """Makes sure files have the right permissions. 6 """Makes sure files have the right permissions.
7 7
8 Some developers have broken SCM configurations that flip the svn:executable 8 Some developers have broken SCM configurations that flip the svn:executable
9 permission on for no good reason. Unix developers who run ls --color will then 9 permission on for no good reason. Unix developers who run ls --color will then
10 see .cc files in green and get confused. 10 see .cc files in green and get confused.
(...skipping 119 matching lines...) Expand 10 before | Expand all | Expand 10 after
130 'configure', 130 'configure',
131 'depcomp', 131 'depcomp',
132 'install-sh', 132 'install-sh',
133 'missing', 133 'missing',
134 'mkinstalldirs', 134 'mkinstalldirs',
135 'naclsdk', 135 'naclsdk',
136 'scons', 136 'scons',
137 ) 137 )
138 138
139 # File paths starting with one of these will be ignored as well. 139 # File paths starting with one of these will be ignored as well.
140 # Please consider fixing your file permissions, rather than adding to this list.
140 IGNORED_PATHS = ( 141 IGNORED_PATHS = (
141 # TODO(maruel): Detect ELF files. 142 # TODO(maruel): Detect ELF files.
142 'chrome/test/data/extensions/uitest/plugins', 143 'chrome/test/data/extensions/uitest/plugins',
143 'chrome/test/data/extensions/uitest/plugins_private', 144 'chrome/test/data/extensions/uitest/plugins_private',
144 'chrome/test/data/webrtc/linux/peerconnection_server', 145 'chrome/test/data/webrtc/linux/peerconnection_server',
145 'chrome/installer/mac/sign_app.sh.in', 146 'chrome/installer/mac/sign_app.sh.in',
146 'chrome/installer/mac/sign_versioned_dir.sh.in', 147 'chrome/installer/mac/sign_versioned_dir.sh.in',
147 'native_client_sdk/src/build_tools/sdk_tools/third_party/', 148 'native_client_sdk/src/build_tools/sdk_tools/third_party/',
148 'out/', 149 'out/',
149 # TODO(maruel): Fix these. 150 # TODO(maruel): Fix these.
(...skipping 347 matching lines...) Expand 10 before | Expand all | Expand 10 after
497 print '\nFAILED\n' 498 print '\nFAILED\n'
498 print '\n'.join(errors) 499 print '\n'.join(errors)
499 return 1 500 return 1
500 if not options.bare: 501 if not options.bare:
501 print '\nSUCCESS\n' 502 print '\nSUCCESS\n'
502 return 0 503 return 0
503 504
504 505
505 if '__main__' == __name__: 506 if '__main__' == __name__:
506 sys.exit(main()) 507 sys.exit(main())
OLDNEW
« no previous file with comments | « tools/checkperms/OWNERS ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698