Bug 151914

Summary: prepare-ChangeLog: Fix some warning messages when using svn
Product: WebKit Reporter: David Kilzer (:ddkilzer) <ddkilzer>
Component: Tools / TestsAssignee: David Kilzer (:ddkilzer) <ddkilzer>
Status: RESOLVED FIXED    
Severity: Normal CC: bfulgham, commit-queue, dbates, jmarcell, lforschler
Priority: P2    
Version: Other   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch v1 none

Description David Kilzer (:ddkilzer) 2015-12-05 05:02:45 PST
Running prepare-ChangeLog on an svn repository with a modified file in "InternalProject/Foo Bar/ClassName.m" causes the following spew:

$ ../WebKit/Tools/Scripts/prepare-ChangeLog InternalProject
  Running status to find changed, added, or removed files.
svn: E200005: '/SubversionWorkingDirectory/Dialogs' is not under version control
svn: E200005: '/SubversionWorkingDirectory/Dialogs/WBSJavaScriptDialogManager.m' is not under version control
svn: E200005: '/SubversionWorkingDirectory/Dialogs' is not under version control
svn: E200005: '/SubversionWorkingDirectory/Dialogs/WBSJavaScriptDialogManager.m' is not under version control
  Reviewing diff to determine which lines changed.
Use of uninitialized value $path in substitution (s///) at /WebKit/Tools/Scripts/VCSUtils.pm line 489, <DIFF> line 1.
Use of uninitialized value $path in substitution (s///) at /WebKit/Tools/Scripts/VCSUtils.pm line 489, <DIFF> line 2.
Use of uninitialized value $path in substitution (s///) at /WebKit/Tools/Scripts/VCSUtils.pm line 489, <DIFF> line 3.
Use of uninitialized value $path in substitution (s///) at /WebKit/Tools/Scripts/VCSUtils.pm line 489, <DIFF> line 4.
Use of uninitialized value $path in substitution (s///) at /WebKit/Tools/Scripts/VCSUtils.pm line 489, <DIFF> line 5.
Use of uninitialized value $path in substitution (s///) at /WebKit/Tools/Scripts/VCSUtils.pm line 489, <DIFF> line 6.
Use of uninitialized value $path in substitution (s///) at /WebKit/Tools/Scripts/VCSUtils.pm line 489, <DIFF> line 7.
Use of uninitialized value $path in substitution (s///) at /WebKit/Tools/Scripts/VCSUtils.pm line 489, <DIFF> line 8.
Use of uninitialized value $path in substitution (s///) at /WebKit/Tools/Scripts/VCSUtils.pm line 489, <DIFF> line 9.
Use of uninitialized value $path in substitution (s///) at /WebKit/Tools/Scripts/VCSUtils.pm line 489, <DIFF> line 10.
Use of uninitialized value $path in substitution (s///) at /WebKit/Tools/Scripts/VCSUtils.pm line 489, <DIFF> line 11.
Use of uninitialized value $path in substitution (s///) at /WebKit/Tools/Scripts/VCSUtils.pm line 489, <DIFF> line 12.
Use of uninitialized value $path in substitution (s///) at /WebKit/Tools/Scripts/VCSUtils.pm line 489, <DIFF> line 13.
  Extracting affected function names from source files.
  Change author: David Kilzer <ddkilzer@apple.com>.
  Running 'svn update' to update ChangeLog files.
    Updating 'InternalProject/ChangeLog':
    At revision 12345.
  Editing the InternalProject/ChangeLog file.
-- Please remember to include a detailed description in your ChangeLog entry. --
-- See <http://webkit.org/coding/contributing.html> for more info --
Comment 1 David Kilzer (:ddkilzer) 2015-12-05 05:14:35 PST
Created attachment 266713 [details]
Patch v1
Comment 2 WebKit Commit Bot 2015-12-05 09:08:37 PST
Comment on attachment 266713 [details]
Patch v1

Clearing flags on attachment: 266713

Committed r193514: <http://trac.webkit.org/changeset/193514>
Comment 3 WebKit Commit Bot 2015-12-05 09:08:39 PST
All reviewed patches have been landed.  Closing bug.