[svk-commit] r2293 - trunk/lib/SVK/Mirror/Backend

nobody at bestpractical.com nobody at bestpractical.com
Thu Jan 25 09:41:04 EST 2007


Author: audreyt
Date: Thu Jan 25 09:40:58 2007
New Revision: 2293

Modified:
   trunk/lib/SVK/Mirror/Backend/SVNRa.pm

Log:
* SVK::Mirror::Backend::SVNRa - Probe for replaying from googlecode  
  could fail with RA_DAV_REQUEST_FAILED instead of UNSUPPORTED_FEATURE,
  which used to caused a crash on every sync.

* Also squash a uninitialized warning in SVNRa.pm.

Modified: trunk/lib/SVK/Mirror/Backend/SVNRa.pm
==============================================================================
--- trunk/lib/SVK/Mirror/Backend/SVNRa.pm	(original)
+++ trunk/lib/SVK/Mirror/Backend/SVNRa.pm	Thu Jan 25 09:40:58 2007
@@ -306,7 +306,8 @@
     # FIXME: if we do ^c here $err would be empty. do something else.
     return $self->_has_replay(0)
       if $err->apr_err == $SVN::Error::RA_NOT_IMPLEMENTED      # ra_svn
-      || $err->apr_err == $SVN::Error::UNSUPPORTED_FEATURE;    # ra_dav
+      || $err->apr_err == $SVN::Error::UNSUPPORTED_FEATURE     # ra_dav
+      || $err->apr_err == $SVN::Error::RA_DAV_REQUEST_FAILED;  # ra_dav (googlecode)
     die $err->expanded_message;
 }
 
@@ -374,6 +375,9 @@
 sub find_rev_from_changeset {
     my ($self, $changeset) = @_;
     my $t = $self->mirror->get_svkpath;
+
+    no warnings 'uninitialized'; # $s_changeset below may be undef
+
     return $t->search_revision
 	( cmp => sub {
 	      my $rev = shift;


More information about the svk-commit mailing list