Checkdeps update to default to resolve_dotdot.

checkdeps/checkdeps.py

 #!/usr/bin/env python
 # Copyright 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 """Makes sure that files include headers from allowed directories.
 
 Checks DEPS files in the source tree for rules, and applies those rules to
 "#include" and "import" directives in the .cpp and .java source files.
 Any source file including something not permitted by the DEPS files will fail.
@@ skipping 25 matching lines @@
   """Parses include_rules from DEPS files and verifies files in the
   source tree against them.
   """
 
   def __init__(self,
                base_directory=None,
                verbose=False,
                being_tested=False,
                ignore_temp_rules=False,
                skip_tests=False,
                resolve_dotdot=False):

[agrieve, 2016/05/12 14:32:14]

nit: can you change this default to True as well? That way we won't need to
update the PRESUBMIT.py invokation to explicitly pass true.

[smaier, 2016/05/12 14:46:48]

Done.

     """Creates a new DepsChecker.
 
     Args:
       base_directory: OS-compatible path to root of checkout, e.g. C:\chr\src.
       verbose: Set to true for debug output.
       being_tested: Set to true to ignore the DEPS file at tools/checkdeps/DEPS.
       ignore_temp_rules: Ignore rules that start with Rule.TEMP_ALLOW ("!").
     """
     DepsBuilder.__init__(
         self, base_directory, verbose, being_tested, ignore_temp_rules)
@@ skipping 115 matching lines @@
       action='store_true', dest='skip_tests', default=False,
       help='Skip checking test files (best effort).')
   option_parser.add_option(
       '-v', '--verbose',
       action='store_true', default=False,
       help='Print debug logging')
   option_parser.add_option(
       '', '--json',
       help='Path to JSON output file')
   option_parser.add_option(
-      '', '--resolve-dotdot',
-      action='store_true', dest='resolve_dotdot', default=False,
+      '', '--no-resolve-dotdot',
+      action='store_false', dest='resolve_dotdot', default=True,
       help='resolve leading ../ in include directive paths relative '
            'to the file perfoming the inclusion.')
 
   options, args = option_parser.parse_args()
 
   deps_checker = DepsChecker(options.base_directory,
                              verbose=options.verbose,
                              ignore_temp_rules=options.ignore_temp_rules,
                              skip_tests=options.skip_tests,
                              resolve_dotdot=options.resolve_dotdot)
@@ skipping 27 matching lines @@
   if options.json:
     deps_checker.results_formatter = results.JSONResultsFormatter(
         options.json, deps_checker.results_formatter)
 
   deps_checker.CheckDirectory(start_dir)
   return deps_checker.Report()
 
 
 if '__main__' == __name__:
   sys.exit(main())