> Sun Jun  6 16:21:25 CEST 2010  Petr Rockai <m...@mornfall.net>
> +renameChanged from to = modify $ \st -> st { changed = rename' $ changed st }
> +  where rename' = S.fromList . map renameone . S.toList
> +        renameone x | from `isPrefix` x = to `catPaths` relative from x
> +                    | otherwise = x
> +        relative (AnchoredPath from) (AnchoredPath x) = AnchoredPath $ drop 
> (length from) x

Just a quick comment here (without having read the patch properly).

Do we have to worry about things like initial/trailing slashes in from
and x?

If I understand correctly AnchoredPath is for paths relative to the tree
root?

>             unless (isNothing item) $ do
>                    modifyItem from Nothing
>                    modifyItem to item
> +                  renameChanged from to

One would have to know more about hashed-storage to see if this makes
sense.
> 
> Sun Jun  6 16:28:58 CEST 2010  Petr Rockai <m...@mornfall.net>
>   * Accept issue1860: (Un)applying move patches may lead to incomplete 
> pristine.


Accept issue1860: (Un)applying move patches may lead to incomplete pristine.
----------------------------------------------------------------------------
I'll apply this one straightaway

> hunk ./tests/failing-issue1860-incomplete-pristine.sh 1
> +#!/usr/bin/env bash
> +## Copyright (C) 2010 Petr Rockai

So for some reason, our EXAMPLE.sh file has a request to name the issue
and give a synopsis at the top of the file.  I'm sure Trent had a good
reason for it, but no feelings either way.  The important thing is that
we get the job done.

(and the licensing stuff below helps).

> +. lib                           # Load some portability helpers.
> +rm -rf R S                      # Another script may have left a mess.
> +darcs init      --repo R        # Create our test repos.
> +
> +cd R
> +mkdir tools
> +darcs rec -lam "add tools"
> +mkdir tools/cgi
> +echo bar > tools/cgi/README
> +darcs rec -lam "add cgi"
> +darcs move tools contrib
> +darcs rec -lam "rename tools/ to contrib/"
> +rm -rf contrib/cgi
> +darcs rec -lam "drop cgi"
> +cd ..
> +
> +darcs get R S
> +cd S
> +darcs show pristine
> +darcs unpull --last 2 -a
> +darcs show pristine
> +cd ..

Looks good.  Maybe boiling it down further (anonymising the names,
patches etc) would have been nice, but it doesn't make a difference.

-- 
Eric Kow <http://www.nltg.brighton.ac.uk/home/Eric.Kow>
PGP Key ID: 08AC04F9

Attachment: pgpeRqQ6Wivgq.pgp
Description: PGP signature

_______________________________________________
darcs-users mailing list
darcs-users@darcs.net
http://lists.osuosl.org/mailman/listinfo/darcs-users

Reply via email to