Re: New built-in fsmonitor: messages accidentally on stdout?

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





On 9/27/21 5:32 AM, Tao Klerks wrote:
Hi folks,

I did track down where that message was being issued onto stdout in
git-for-windows
(https://github.com/git-for-windows/git/blob/main/builtin/fsmonitor--daemon.c),
and I see that in the the latest version of the patch series it was
already switched to stderr
(https://github.com/jeffhostetler/git/blob/f08a63e197a0efe4d8f0452893238b0d3156e669/builtin/fsmonitor--daemon.c).

The only remaining thing I can ask for, I guess, is that the next Git
for Windows release ideally include this change/fix, even though the
built-in fsmonitor is not in "regular git" at all yet?

Yes, I've fixed that in my branch to print to stderr (and also
put it behind a feature flag).

If my current series does not make it into core Git in time
for the next release, we will update the experimental version
in the next git-for-windows release.

Thanks,
Jeff



[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux