From: Junio C Hamano <gitster@pobox.com>
Date: Wed, 31 Aug 2016 20:17:56 GMT
Subject: Re: [PATCH v2 3/4] cat-file --textconv/--filters: allow specifying the path separately
Message-ID: <xmqqinugzo97.fsf@gitster.mtv.corp.google.com>
In-Reply-To: <alpine.DEB.2.20.1608312133150.129229@virtualbox>

Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:

> However, I do not want to hold this patch series up just by being
> stubborn.

Peff and I disussed this further in the thread in which the message
<xmqqa8g2qcf3.fsf@gitster.mtv.corp.google.com> appears, and agreed
that it does not matter that much either way.

The comment to [v2 2/4] would be more important than this one, I
would think.  What should happen when a blob that is not ISREG is
given to filter_object()?  Giving a symlink contents as-is without
filtering would be OK.  Erroring out saying "regular file blob
expected" just like the function reacts to non-blob is also OK.
Just being silent and not doing anything is probably not.

Thanks.