Thread (8 messages) 8 messages, 4 authors, 2017-05-01

Re: [PATCH] t7400: add BSLASHPSPEC prerequisite to 'add with \\ in path'

From: Johannes Schindelin <hidden>
Date: 2017-05-01 10:59:33

Hi Ramsay,


On Sun, 30 Apr 2017, Ramsay Jones wrote:
On 29/04/17 11:44, Johannes Schindelin wrote:
quoted
On Sat, 29 Apr 2017, Johannes Sixt wrote:
quoted
Am 29.04.2017 um 02:15 schrieb Ramsay Jones:
quoted
On 28/04/17 20:54, Johannes Sixt wrote:
quoted
Am 28.04.2017 um 05:09 schrieb Junio C Hamano:
quoted
Ramsay Jones [off-list ref] writes:
quoted
quoted
I don't observe these failures. Are you using a vanila MSYS2
environment?
Please note that the "vanilla MSYS2 environment" is *not* expected to
pass the test suite when compiling in MINGW mode. In fact, it is
expected to fail.

In 2015, I made a couple of changes to the MSYS2 runtime in
preparation for the big bump to Git for Windows 2.x (which switched
from the old MSys environment to the new MSYS2 environment), and
released Git for Windows 2.5.0 with a heavily patched msys-2.0.dll. My
hope was that those changes would be welcome in the MSYS2 project, but
well, they kinda weren't. So I was forced to abandon my original plan
to contribute all of those fixes to "upstream MSYS2".
Oh WOW. I didn't know you were maintaining your own version of
the MSYS2 runtime. That must be a huge PITA. :-D
I manage. The long years of maintaining Git for Windows as a fork of Git
helps. I use the exact same strategy: merging rebases.

Amazingly, the Cygwin project itself has been quite open to accept my
patches, and the only problem there is time: I would like to contribute
more patches there, I get really valuable feedback, and I just need to
find the time to iterate the patches so they can be accepted.
quoted
I even started collecting the exact tests that are failing with the
vanilla MSYS2 runtime vs Git for Windows' fork, when I still had hopes
that we could test things with AppVeyor (but the builds were already
too slow, we hit the timeout even before trying to run the tests, so I
gave up on that front):

	REM MSYS2's runtime does not carry Git for Windows' tweaks yet, so these
	tests cannot pass:
	set GIT_SKIP_TESTS='t0003 t0006 t0024 t1100 t1400 t1402 t1501 t1504 t1506
	t1508 t1513 t3001 t3070 t3200 t3301 t3400 t3404 t3513 t3703 t4116 t4150
	t4208 t4211 t5000 t5001 t5002 t5004 t5500 t5601 t5602 t5603 t5801 t6006
	t6018 t6041 t6130 t6132 t6300 t7201 t7400 t7501 t7502 t8002 t8006 t9001
	t9350 t9700 t9903'
I have only (fairly) recently installed MSYS2, so I've only ever
run the MinGW64 test-suite once, which for me failed on tests:

   t0003, t0006, t0026, t0060, t0200, t0204, t1100, t1400, t1402,
   t1501, t1504, t1506, t1508, t1513, t3001, t3070, t3200, t3301,
   t3400, t3404, t3406, t3703, t3903, t3905, t4208, t4211, t5000,
   t5500, t5516, t5601, t5602, t5603, t5615, t5801, t6006, t6018,
   t6030, t6038, t6130, t6132, t6300, t7201, t7400, t7401, t7406,
   t7501, t7610, t9001, t9020, t9350, t9700, t9903

(which I found somewhat intimidating!).
Yes, I expected the number to rise. Note that almost every patch in Git
for Windows' fork fixes a couple of test scripts at a time.
So, as you would expect, it hasn't improved much! :-P

Hmm, I was hoping to use this installation to test some git patches
on MinGW, but that looks like a lost cause. I may as well save some
disk space and delete it!
Hopefully I (or other Git for Windows contributors) will have some time to
make installing a Git for Windows development environment as easy as

	git clone --depth 1 https://github.com/git-for-windows/git-skd-64

Then you do not even need to worry to keep a local installation
up-to-date. You'd just reclone when (if) needed.

Ciao,
Dscho
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help