Johannes Schindelin

Results 878 comments of Johannes Schindelin

Could you provide a small program (C source preferred) to reproduce the problem? `pdflatex` is a bit large, and I also don't know what `output.tex` you used.

> Any thought how we could fix this again? Not _completely_. As stated in the commit message of https://github.com/msys2/msys2-runtime/commit/9944652c41fb96f555d221865f4ed5d0c8b33514, the idea is to not confuse Git's `:./` for a path...

> Now, in your examples, the path lists all start with either `/` or `./`. Would this always be the case? If yes, I think we can reliably discern between...

> The broken package uses the following variable > `WEBINPUTS=.:../../../texk/web2c` (see [msys2/MINGW-packages#19921 (comment)](https://github.com/msys2/MINGW-packages/pull/19921#issuecomment-2042220562)). Well, the prefix `.:` would be easy enough to catch: it is neither a valid `scp` target...

> The two requirements, git path resolution and arbitrary path conversion, do not go well together unless the converter knows that it is in the context of `git` or `scp`....

> I'd be interested in the original intention of the change in [9944652](https://github.com/msys2/msys2-runtime/commit/9944652c41fb96f555d221865f4ed5d0c8b33514). Was it to satisfy the test suite? While it does satisfy Git's test suite, the underlying reason...

So here is [a PR](https://github.com/msys2/msys2-runtime/pull/212) with the quick fix; Once the PR build finishes, I hope that y'all can give it a whirl to see whether the MSYS2 runtime built...

> Simply revert 0042-fixup-Add-functionality-for-converting-UNIX-paths-in.patch fix this issue This is not a great suggestion, given that that patch was introduced to fix problems in the first place. The suggestion is to...

> To be more specific, revert that patch not only fix issue 187, it also fix this problem. And cause a regression. Great suggestion! > We may have a more...

> Unfortunately `msys2-runtime` lacks such a suite and so "our" requirements are now violated. @reneparis There is actually a test suite, and I worked a bit to get it working...