Use relative paths in svndiff, so results can be exported by Apache

tools/svndiff.py

 #!/usr/bin/python
 '''
 Copyright 2012 Google Inc.
 
 Use of this source code is governed by a BSD-style license that can be
 found in the LICENSE file.
 '''
 
 '''
 Generates a visual diff of all pending changes in the local SVN (or git!)
@@ skipping 202 matching lines @@
     """Generates a visual diff of all pending changes in source_dir.
 
     @param path_to_skdiff
     @param dest_dir existing directory within which to write results
     @param source_dir
     """
     # Validate parameters, filling in default values if necessary and possible.
     path_to_skdiff = os.path.abspath(FindPathToSkDiff(path_to_skdiff))
     if not dest_dir:
         dest_dir = tempfile.mkdtemp()
-    dest_dir = os.path.abspath(dest_dir)
 
-    os.chdir(source_dir)
-    using_svn = os.path.isdir('.svn')
+    using_svn = os.path.isdir(os.path.join(source_dir, '.svn'))
 
     # Prepare temporary directories.
     modified_flattened_dir = os.path.join(dest_dir, 'modified_flattened')
     original_flattened_dir = os.path.join(dest_dir, 'original_flattened')
     diff_dir = os.path.join(dest_dir, 'diffs')
     for dir in [modified_flattened_dir, original_flattened_dir, diff_dir] :
         shutil.rmtree(dir, ignore_errors=True)
         os.mkdir(dir)
 
     # Get a list of all locally modified (including added/deleted) files,
     # descending subdirectories.
     if using_svn:
-        svn_repo = svn.Svn('.')
+        svn_repo = svn.Svn(source_dir)
         modified_file_paths = svn_repo.GetFilesWithStatus(
             svn.STATUS_ADDED | svn.STATUS_DELETED | svn.STATUS_MODIFIED)
     else:
         modified_file_paths = _GitGetModifiedFiles()

[epoger, 2013/09/13 20:26:27]

DO NOT COMMIT AS-IS... this won't work with git.

 
     # For each modified file:
     # 1. copy its current contents into modified_flattened_dir
     # 2. copy its original contents into original_flattened_dir
     for modified_file_path in modified_file_paths:
         if modified_file_path.endswith('.json'):
             # Special handling for JSON files, in the hopes that they
             # contain GM result summaries.
             (_unused, original_file_path) = tempfile.mkstemp()
             if using_svn:
                 svn_repo.ExportBaseVersionOfFile(
                     modified_file_path, original_file_path)
             else:
                 _GitExportBaseVersionOfFile(
                     modified_file_path, original_file_path)
             platform_prefix = re.sub(os.sep, '__',
                                      os.path.dirname(modified_file_path)) + '__'
             _CallJsonDiff(old_json_path=original_file_path,
-                          new_json_path=modified_file_path,
+                          new_json_path=os.path.join(source_dir, modified_file_path),
                           old_flattened_dir=original_flattened_dir,
                           new_flattened_dir=modified_flattened_dir,
                           filename_prefix=platform_prefix)
             os.remove(original_file_path)
         else:
             dest_filename = re.sub(os.sep, '__', modified_file_path)
             # If the file had STATUS_DELETED, it won't exist anymore...
             if os.path.isfile(modified_file_path):
                 shutil.copyfile(modified_file_path,
                                 os.path.join(modified_flattened_dir,
@@ skipping 37 matching lines @@
                       help='path to already-built skdiff tool; if not set, '
                       'will search for it in typical directories near this '
                       'script')
     parser.add_option(OPTION_SOURCE_DIR,
                       action='store', type='string',
                       default=os.path.join('expectations', 'gm'),
                       help='root directory within which to compare all ' +
                       'files; defaults to "%default"')
     (options, args) = parser.parse_args()
     Main(options, args)