Re: [gentoo-portage-dev] [PATCH] SyncManager: redirect command stderr to stdout (bug 566132)

2015-11-23 Thread Alexander Berntsen
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA512

Sorry for not being up to date what this does, but the bug complains
specifically about GitHub mirrors, while the patch itself seems far
more invasive. Is it the case that they should *never* go to STDERR?

- -- 
Alexander
berna...@gentoo.org
https://secure.plaimi.net/~alexander
-BEGIN PGP SIGNATURE-
Version: GnuPG v2

iQIcBAEBCgAGBQJWUuzpAAoJENQqWdRUGk8BMdQQAKACO5bGwnXxbyNRuh/KzIiH
e38mzzvxDibJTTz32XSsOn1w9StZkSWAcDwT8Ef4QGeQJxmLTXUW+5eUsAVoaNxy
VY81iQbDmcSf+erljYEuUBBOFDMORYGwhSROdvvLE/0d3LFP6AurWx9a4kb229j7
DB8HmpjXkkpT65YgJvp2vD51x+K5HsDOKTObBcDreiea21u2X6DNcGbVOK35IXs3
fbTovSrL39qTk8zPSvuwePHlfWelsTCmj/+2EmLVJMTem/cTo0uxCfGCOmwALusk
FI4YcV1zf1cMrL1tqU9B7bYSH8oci4Rfs80IanieDwpd59V32xQ8ckyOP8ME5gG/
xbqipfwbWJyyUFErL7BZE9/H90tib9uofR9mxP9BylXjsDMbu/sC7OA2ElBNVilp
TJGvmRb3jdIWsSddNsOgdN2XofR20G5Ow/NKEvpueM42Pk4s6lqYwZODmsU5kOsS
wSw6ErQO6RCKfu0iQhQo3VS4EjP95sVl21nXtJk6v8AkNwNkU1TNZ8f1Ds5CaALR
79XpFL3JQy/zwd0uMY7kOjgezJJeu4nhfSz2k1ZDNLHokTcqhqR4WdXqzkWA2iZq
Y6p6CLWY57TIiih0ASt+NSfrkRl1628irZyVcyoVKDpaWCVcWq+G1sAl4pmAXpXV
nVhyL0zqartr+TNlFz2N
=g6l5
-END PGP SIGNATURE-



Re: [gentoo-portage-dev] [PATCH] SyncManager: redirect command stderr to stdout (bug 566132)

2015-11-23 Thread Zac Medico
On 11/23/2015 02:39 AM, Alexander Berntsen wrote:
> Sorry for not being up to date what this does, but the bug complains
> specifically about GitHub mirrors, while the patch itself seems far
> more invasive. Is it the case that they should *never* go to STDERR?

Since the sync modules can call a variety of different commands, it's
hard to predict which ones will unnecessarily write to stderr. So, the
assumption here is that the process exit status is a more reliable
indicator for errors. For example, the python code in GitSync will write
an error message to stderr if the git process exits with non-zero status.
-- 
Thanks,
Zac