[PATCH 1 of 3 V2] serve: rename --daemon-pipefds to --daemon-postexec

Augie Fackler raf at durin42.com
Tue Feb 23 14:21:37 EST 2016


On Mon, Feb 22, 2016 at 07:25:48PM +0000, Jun Wu wrote:
> On 02/22/2016 06:39 PM, David Soria Parra wrote:
> >This is a BC break. We might not care as it's use internally, but I
> >am not sure if other libraries talking to the daemon through that
> >interface are usign this option.
> >
> >I'd recommend to leave --daemon-pipefds as a deprecated option and
> >map it to postexec in case the latter one doesn't exist, instead of
> >risking breaking people, but that's just my 2cents.
>
> I guess it's okay since e22695b4472f is not marked as BC either. It's
> internal only and the name is confusing so I prefer deleting the old one
> directly.

I'm comfortable with this bc since it's marked as internal, and have
queued this series with (BC) added to patch 1. Thanks!

> _______________________________________________
> Mercurial-devel mailing list
> Mercurial-devel at mercurial-scm.org
> https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel


More information about the Mercurial-devel mailing list