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

Merge stable #567

Merged
merged 4 commits into from
Jan 1, 2024
Merged

Merge stable #567

merged 4 commits into from
Jan 1, 2024

Conversation

ibuclaw
Copy link
Member

@ibuclaw ibuclaw commented Jan 1, 2024

  • Fix Issue 22212 - dmd version has -dirty suffix on windows

Typically, git-clone probes the filesystem to see if it handles the executable bit correctly and the `core.fileMode` variable is automatically set as necessary. However when building dmd releases using vagrant box method, git-clone is done on a Linux host (i.e a filesystem that handles the filemode correctly), then scp'd to a Windows VM (i.e an environment that loses the filemode). So when building Windows binaries, always ensure this variable is set to `false`.
@dlang-bot
Copy link
Contributor

Thanks for your pull request, @ibuclaw!

Bugzilla references

Auto-close Bugzilla Severity Description
22212 regression dmd version has -dirty suffix on windows

@ibuclaw ibuclaw merged commit 27d7e42 into master Jan 1, 2024
23 of 26 checks passed
@ibuclaw ibuclaw deleted the merge_stable branch January 1, 2024 21:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants