Skip to content

Fix SOURCE_DATE_EPOCH in Makefile.win#14922

Merged
straight-shoota merged 1 commit intocrystal-lang:masterfrom
HertzDevil:bug/windows-source-date-epoch
Aug 21, 2024
Merged

Fix SOURCE_DATE_EPOCH in Makefile.win#14922
straight-shoota merged 1 commit intocrystal-lang:masterfrom
HertzDevil:bug/windows-source-date-epoch

Conversation

@HertzDevil
Copy link
Contributor

The SOURCE_DATE_EPOCH variable in Makefile.win has been broken after #14574:

  • The type built-in never accepts file names with forward slashes, so building the compiler locally always prints The syntax of the command is incorrect. to the console.
  • With that fixed, if the src\SOURCE_DATE_EPOCH file doesn't exist, which is most of the time, then instead The system cannot find the file specified. gets printed.
  • For some unknown reason, the % doesn't get seen by the shell when there is a ||.

This PR replaces the variable definition with one that works.

@HertzDevil HertzDevil added kind:bug A bug in the code. Does not apply to documentation, specs, etc. topic:infrastructure platform:windows Windows support based on the MSVC toolchain / Win32 API kind:regression Something that used to correctly work but no longer works labels Aug 20, 2024
@straight-shoota straight-shoota added this to the 1.14.0 milestone Aug 20, 2024
@straight-shoota straight-shoota merged commit bb75d3b into crystal-lang:master Aug 21, 2024
@HertzDevil HertzDevil deleted the bug/windows-source-date-epoch branch August 23, 2024 18:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind:bug A bug in the code. Does not apply to documentation, specs, etc. kind:regression Something that used to correctly work but no longer works platform:windows Windows support based on the MSVC toolchain / Win32 API topic:infrastructure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants