[PATCH] verify: don't check for /dev/null files in delta

[Martin von Zweigbergk]

On Fri, Mar 27, 2015 at 1:09 PM Matt Mackall <mpm at selenic.com> wrote:

> On Fri, 2015-03-27 at 11:36 -0700, Martin von Zweigbergk wrote:
> > # HG changeset patch
> > # User Martin von Zweigbergk <martinvonz at google.com>
> > # Date 1427479422 25200
> > #      Fri Mar 27 11:03:42 2015 -0700
> > # Node ID 7714ea5cbb8270c7fcacb0014cb5d13304818178
> > # Parent  95cbc77c0cad04b04bd690bb3ba91e919e11d3b1
> > verify: don't check for /dev/null files in delta
> >
> > The manifest delta is generated from the revlog delta, which never
> > contains /dev/null, so remove the unnecessary check.
>
> I think this is here because some very old repos managed to
> commit /dev/null.. because our early patch importer didn't understand
> that /dev/null meant "delete". And we had nothing like our path auditor
> to prevent committing files outside a repo.
>

That's what I was wondering. I'll send another patch to document it
instead. Thanks.


>
> --
> Mathematics is the supreme nostalgia of our time.
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://selenic.com/pipermail/mercurial-devel/attachments/20150327/3df48303/attachment.html>