[PATCH 04 of 10] chg: drop --cwd /
Jun Wu
quark at fb.com
Fri Mar 4 06:37:18 EST 2016
On 03/03/2016 02:44 PM, Yuya Nishihara wrote:
> I don't like the implicit chdir() introduced by the next patch. Probably we
> can do "--daemon-postexec chdir:/" instead. It's common for a daemon process
> to chdir to / once its boot-up sequence is completed.
Sorry for the late response but I am the London oncall this week and was busy
handling internal stuffs.
I think --daemon-postexec chdir:/ works but is not a good fit because:
- For chgserver, chdir / should be always performed, regardless of what the
client says.
- It needs extra work to make multiple "postexec" actions (say you want to do
both chdir and unlink). Turning --daemon-postexec into a multiple flag will
make "--daemon-postexec none" look weird.
I will skip this one temporarily for now.
More information about the Mercurial-devel
mailing list