[svk-commit] r2908 - in trunk: lib/SVK

nobody at bestpractical.com nobody at bestpractical.com
Fri Jun 6 00:09:15 EDT 2008


Author: clsung
Date: Fri Jun  6 00:09:09 2008
New Revision: 2908

Modified:
   trunk/lib/SVK/XD.pm
   trunk/t/23commit.t

Log:
- to fix r2875
  - $target->report should never be '' to call subdir
  - since it use XD::to_add, and there's 
      $logger->info( "$st   $report$bin");
    in XD::_to_add()
    - so also modify the t/23commit.

Modified: trunk/lib/SVK/XD.pm
==============================================================================
--- trunk/lib/SVK/XD.pm	(original)
+++ trunk/lib/SVK/XD.pm	Fri Jun  6 00:09:09 2008
@@ -815,7 +815,7 @@
             my ($editor, $path) = @_;
             to_native($path, 'path');
             my $copath = $target->copath($path);
-            my $report = $target->report->subdir($path);
+            my $report = $target->_to_pclass($target->report)->subdir($path);
             lstat ($copath);
             $self->_do_add('A', $copath, $report, !-d _, %arg);
         },

Modified: trunk/t/23commit.t
==============================================================================
--- trunk/t/23commit.t	(original)
+++ trunk/t/23commit.t	Fri Jun  6 00:09:09 2008
@@ -323,7 +323,7 @@
         __('?   A/verbose')]);
 
 is_output ($svk, 'commit', [],
-	   ['Waiting for editor...',
+	   ['Waiting for editor...', 'A   A/verbose',
 	    'Committed revision 18.'], 'we got a commit by replacing a ? with an A');
 
 is_output ($svk, 'status', [],


More information about the svk-commit mailing list