On Wed, Sep 24, 2008 at 11:48:38 -0400, David Roundy wrote:
> >     where flags2inv | UseOldFashionedInventory `elem` fs = [Darcs1_0]
> >                     | otherwise = [HashedInventory]
> 
> No, this fails if --hashed and --darcs-2 are both provided.  I'd
> rather have this be robust, since darcs-2 requires HashedInventory.

Gotcha... athough I think you meant if --old-fashioned and --darcs-2 are
both provided.

> >     where flags2inv | UseFormat2 `elem` fs = [HashedInventory]
> >                     | UseHashedInventory `elem` fs = [HashedInventory]
> >                     | UseOldFashionedInventory `elem` fs = [Darcs1_0]
> >                     | otherwise = flags2inv [HashedInventory]
> >
> > Which could simplify switching to some hypothetical inventory format...
> > or maybe I'm just in YAGNI-land
> 
> This seems like needless overkill.

Yeah.

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

Attachment: signature.asc
Description: Digital signature

_______________________________________________
darcs-users mailing list
[email protected]
http://lists.osuosl.org/mailman/listinfo/darcs-users

Reply via email to