[darcs-users] Decoupling patch ids from metadata [was: Fixing a repo with duplicate patches]
David Roundy
droundy at darcs.net
Sat Dec 23 13:14:26 UTC 2006
On Fri, Dec 22, 2006 at 06:02:21PM +0100, Albert Reiner wrote:
> [David Roundy <droundy at darcs.net>, Thu, 21 Dec 2006 05:48:20 -0800]:
> > We just need to teach darcs to hide such lines (a one-line
> > modification to human_friendly), and to generate them (which
> > requires reading /dev/urandom or something).
>
> Just a thought: In this case, shouldn't darcs also escape any
> occurrences of these lines that the user may have entered as part of
> the description?
>
> If the user does not see darcs-internal-patchid in normal operation,
> she may well be tempted to use this string herself. In general,
> hidden "magic" strings are a bad idea IMO, especially in a tool that
> many people may want to use as a basis for implementing their own
> protocols.
Well, it won't hurt darcs at all if users *do* add such a string. If we
really were concerned about this, we could easily add an escaping mechanism
to distinguish user-provided darcs-internal lines from darcs-added ones.
--
David Roundy
http://www.darcs.net
More information about the darcs-users
mailing list