search

cygporter / git

Cygwin Git packaging paraphernalia
2 stars 2 forks source link

t5580-unc-paths.sh test failures #59

Open me-and opened 11 months ago

me-and commented 11 months ago

Using the Cygwin time machine, whatever has broken was broken by the change from cygport v0.36.1-1 and v0.36.2-1.

me-and commented 11 months ago

https://github.com/cygwin/cygport/commit/8b33f707c37f2673337b2509b96529896137cd51 looks most suspicious from a quick skim.

Full changelog is at https://github.com/cygwin/cygport/compare/0.36.1...0.36.2

me-and commented 11 months ago

And bisecting cygport confirms it: the test fails when using https://github.com/cygwin/cygport/commit/8b33f707c37f2673337b2509b96529896137cd51 and succeeds when using https://github.com/cygwin/cygport/commit/9b231ea1cdacfa9324d66ce6af7f07332e865551.

me-and commented 10 months ago

Okay, so the issue is because Cygport is enabling case sensitivity on the build directory, and that's causing the tests to fail. I think that won't actually cause a change in Git's behaviour, though, so this isn't blocking the release.

It's not clear what the correct solution here is, but I can at least block out the tests and release.