Re: [PATCH] diff: add support for reading files literally with --no-index
- From
Ævar Arnfjörð Bjarmason <avarab@gmail.com>
- Date
- Dec 20, 2018, 21:43 UTC
- Message-ID
- <871s6bg9hx.fsf@evledraar.gmail.com>
- In-Reply-To
- <20181220002610.43832-1-sandals@crustytoothpaste.net>
On Thu, Dec 20 2018, brian m. carlson wrote:
> We don't include a test for the pipe scenario because I couldn't get > that case to work in portable shell (although of course it works in > bash). I have, however, tested it on both macOS and Linux. No clue how > this works on Windows.
You can (and should) add tests using the PROCESS_SUBSTITUTION prereq Dennis used in an earlier version of this: https://public-inbox.org/git/20170113102021.6054-3-dennis@kaarsemaker.net/
Even if that wasn't possible a bash-specific test is better than no test, and can be had by including t/lib-bash.sh in your test, see e.g. the completion tests.