Skip to content

make-file-list: include the new /etc/msystem.d/ folder #621

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

Merged
merged 1 commit into from
Jun 3, 2025

Conversation

dscho
Copy link
Member

@dscho dscho commented Jun 3, 2025

As of filesystem-2025.05.08-1, the various definitions depending on the environment variable MSYSTEM are no longer in a single /etc/msystem file, but rather in files in the new /etc/msystem.d/ directory whose filenames reflect the value of said environment variable.

As suggested, let's include this directory in the payload of Git for Windows' installer so that users are not greeted with the message:

bash: /etc/msystem.d/MSYS: No such file or directory

As of filesystem-2025.05.08-1, the various definitions depending on the
environment variable `MSYSTEM` are no longer in a single `/etc/msystem`
file, but rather in files in the new `/etc/msystem.d/` directory whose
filenames reflect the value of said environment variable.

As per the suggestion in
git-for-windows/git-sdk-64#99 (comment),
let's include this directory in the payload of Git for Windows'
installer so that users are not greeted with the message:

	bash: /etc/msystem.d/MSYS: No such file or directory

Suggested-by: Jeremy Drake <github@jdrake.com>
Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
@dscho
Copy link
Member Author

dscho commented Jun 3, 2025

I will need to merge this PR quickly because -rc1 is scheduled for today and I haven't managed to get -rc0 out yet.

@dscho dscho requested a review from mjcheetham June 3, 2025 10:34
@dscho dscho merged commit b8a4cd6 into main Jun 3, 2025
18 checks passed
@dscho dscho deleted the accommodate-for-filesystem-2025.05.08-1 branch June 3, 2025 10:51
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.

3 participants