On Wed, Dec 10, 2008 at 07:54:17 +0100, Petr Rockai wrote: > More brooming. Please note that the first patch might need some reviewing, > since I wasn't completely sure what was the intention of the original code. I > just hope it was doing the same thing, namely creating an empty pristine > directory. Tests pass, but I'm not sure we test cross-repo-type gets, where > this is likely to be exercised.
Applied, thanks! Do not use replacePristine in copyFullRepository. ------------------------------------------------- > - withTempDir "newpristine" $ \newpris -> > - replacePristine torepository (toFilePath newpris) > + replacePristineFromSlurpy torepository empty_slurpy replacePristine does indeed appear to be the same as replacePristineFromSlurpy (except without writing anything out), so this does seem to do the right thing... and since this is the only use for this, I guess it makes sense to do this instead of refactoring replacePristine to be just a wrapper Remove now-unused replacePristine. ---------------------------------- Fine -- Eric Kow <http://www.nltg.brighton.ac.uk/home/Eric.Kow> PGP Key ID: 08AC04F9
signature.asc
Description: Digital signature
_______________________________________________ darcs-users mailing list [email protected] http://lists.osuosl.org/mailman/listinfo/darcs-users
