Re: [PATCH 1/2] status: really ignore config with --porcelain
- From
Matthieu Moy <matthieu.moy@grenoble-inp.fr>
- Date
- Jun 24, 2013, 15:04 UTC
- Message-ID
- <vpqa9mfv9bs.fsf@anie.imag.fr>
- In-Reply-To
- <7v8v1zd0du.fsf@alter.siamese.dyndns.org>
Junio C Hamano <gitster@pobox.com> writes:
Show 5 quoted lines
> My knee-jerk reaction is that, because the > "--porcelain" output was designed to be extensible and scripts > reading from it is expected to ignore what it does not understand, > if the setting of status.branch is a problem, the reading side is > buggy and needs to be fixed.
It is extensible in the sense that the caller can provide more command-line options to get more output (i.e. say --branch --porcelain), but providing different results for the same call because of the configuration file is broken IMHO.
-- Matthieu Moy http://www-verimag.imag.fr/~moy/