| 1 |
On 11/23/2015 02:39 AM, Alexander Berntsen wrote: |
| 2 |
> Sorry for not being up to date what this does, but the bug complains |
| 3 |
> specifically about GitHub mirrors, while the patch itself seems far |
| 4 |
> more invasive. Is it the case that they should *never* go to STDERR? |
| 5 |
|
| 6 |
Since the sync modules can call a variety of different commands, it's |
| 7 |
hard to predict which ones will unnecessarily write to stderr. So, the |
| 8 |
assumption here is that the process exit status is a more reliable |
| 9 |
indicator for errors. For example, the python code in GitSync will write |
| 10 |
an error message to stderr if the git process exits with non-zero status. |
| 11 |
-- |
| 12 |
Thanks, |
| 13 |
Zac |
Archives