Eric Wong wrote: > Seems reasonable, a few comments below. Thanks again. [...] > These should die on errors, right? Yep, they do. I didn't bother doing the CARP_NOT thing --- let's wait for a bug report or rainy day and then handle errors properly (by invalidating the cache when appropriate). Patches 1-2 are independent from patch 3. I'm sending them this way for no particular reason. Thoughts of all kinds welcome, as usual. Jonathan Nieder (3): git-svn: make Git::SVN::Editor a separate file git-svn: make Git::SVN::RA a separate file git-svn: use YAML format for mergeinfo cache when possible git-svn.perl | 1115 ++---------------------------------------- perl/Git/SVN/Editor.pm | 536 ++++++++++++++++++++ perl/Git/SVN/Fetcher.pm | 3 +- perl/Git/SVN/Memoize/YAML.pm | 93 ++++ perl/Git/SVN/Ra.pm | 658 +++++++++++++++++++++++++ perl/Makefile.PL | 3 + 6 files changed, 1320 insertions(+), 1088 deletions(-) create mode 100644 perl/Git/SVN/Editor.pm create mode 100644 perl/Git/SVN/Memoize/YAML.pm create mode 100644 perl/Git/SVN/Ra.pm -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html