Re: [PATCH 3/2] t5547-push-quarantine: run the path separator test on Windows, too
- From
Jeff King <peff@peff.net>
- Date
- Dec 13, 2016, 19:12 UTC
- Message-ID
- <20161213191245.we6422xpapfgd2gt@sigill.intra.peff.net>
- In-Reply-To
- <d9d2580c-a2e5-d9f3-1f56-6814b2b2285d@kdbg.org>
On Tue, Dec 13, 2016 at 08:09:31PM +0100, Johannes Sixt wrote:
Show 10 quoted lines
> Am 12.12.2016 um 20:53 schrieb Jeff King: > > Johannes, please let me know if I am wrong about skipping the test on > > !MINGW. The appropriate check there would be ";" anyway, but I am not > > sure _that_ is allowed in paths, either. > > Here is a version for Windows. I'd prefer this patch on top instead > of squashing it into yours to keep the $PWD vs. $(pwd) explanation. > > The result is the same as yours in all practical matters; but this > version I have already tested.
Yeah, I'm happy to have this on top. The patch itself looks obviously correct. Thanks!
-Peff