From: Jeff King Date: Tue, 13 Dec 2016 19:12:45 GMT Subject: Re: [PATCH 3/2] t5547-push-quarantine: run the path separator test on Windows, too Message-ID: <20161213191245.we6422xpapfgd2gt@sigill.intra.peff.net> In-Reply-To: On Tue, Dec 13, 2016 at 08:09:31PM +0100, Johannes Sixt wrote: > 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