Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adjust CWD magic to accommodate for the latest Windows previews #52

Merged

Conversation

dscho
Copy link
Member

@dscho dscho commented May 18, 2023

Reportedly Windows 11 build 25*** from Insider changed the current working directory logic a bit, and Cygwin's "magic" (or: "technologically sufficiently advanced") code needs to be adjusted accordingly.

This fixes git-for-windows/git#4429

@dscho dscho self-assigned this May 18, 2023
Reportedly Windows 11 build 25*** from Insider changed the current
working directory logic a bit, and Cygwin's "magic" (or:
"technologically sufficiently advanced") code needs to be adjusted
accordingly.

This fixes git-for-windows/git#4429

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
@dscho dscho force-pushed the adjust-cwd-magic-to-newest-windows branch from 61d302a to 611c81d Compare May 18, 2023 10:59
@mjcheetham mjcheetham marked this pull request as ready for review May 18, 2023 16:29
@mjcheetham
Copy link
Member

Confirmed that artifacts from the PR build fix git-for-windows/git#4429. Merging...

@mjcheetham mjcheetham merged commit b4fed42 into git-for-windows:main May 18, 2023
@dscho dscho deleted the adjust-cwd-magic-to-newest-windows branch May 18, 2023 16:52
dscho added a commit to dscho/MSYS2-packages that referenced this pull request May 18, 2023
… previews

This is the companion commit to
git-for-windows/msys2-runtime#52

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
@dscho
Copy link
Member Author

dscho commented May 18, 2023

I fear at some stage we may need to backport this to msys2-runtime-3.3...

dscho added a commit to dscho/MSYS2-packages that referenced this pull request May 18, 2023
…dows previews

This is the i686-specific companion commit to
git-for-windows/msys2-runtime#52

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
@dscho
Copy link
Member Author

dscho commented May 18, 2023

I fear at some stage we may need to backport this to msys2-runtime-3.3...

git-for-windows/MSYS2-packages#108

@jeremyd2019
Copy link

should this be upstreamed to cygwin, or do they prefer to wait for the final Windows release?

@dscho
Copy link
Member Author

dscho commented May 22, 2023

should this be upstreamed to cygwin, or do they prefer to wait for the final Windows release?

This should be upstreamed to Cygwin, I'm just a bit underwater, otherwise I'd already have done it.

@dscho
Copy link
Member Author

dscho commented May 22, 2023

should this be upstreamed to cygwin, or do they prefer to wait for the final Windows release?

This should be upstreamed to Cygwin

Here you go: https://inbox.sourceware.org/cygwin-patches/60e1e112b1c293a69bfa4df3fe5094e562898bbb.1684755365.git.johannes.schindelin@gmx.de/T/#u

@dscho
Copy link
Member Author

dscho commented May 25, 2023

@jeremyd2019 it was accepted in upstream: cygwin/cygwin@4840a56

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cygwin warning: couldn't compute FAST_CWD pointer
3 participants