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

make: Add generate_data target for running generator scripts #13700

Merged

Conversation

straight-shoota
Copy link
Member

This target makes it easy to run all generator scripts at once. I want to add this as a check to the release process.

Copy link
Contributor

@HertzDevil HertzDevil left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think scripts/generate_windows_zone_names.cr is the only script that cannot work on Windows (i.e. cannot be called inside Makefile.win), because it still requires live tzdata on the host running the script

@straight-shoota
Copy link
Member Author

I think it should be possible to have that working on Windows with a copy of tzdata.

@HertzDevil
Copy link
Contributor

By the way, if we want to avoid tzdata entirely, the standard abbreviation names are also in CLDR, like this: https://github.com/unicode-org/cldr/blob/b24157695397c4c3d2d058c201da92d10c9d9022/common/main/en_NZ.xml#L165

Makefile.win Show resolved Hide resolved
@straight-shoota straight-shoota added this to the 1.10.0 milestone Sep 21, 2023
@straight-shoota straight-shoota merged commit b597473 into crystal-lang:master Sep 23, 2023
53 checks passed
@straight-shoota straight-shoota deleted the infra/make-generate_data branch September 23, 2023 07:22
@straight-shoota straight-shoota changed the title infra(make): Add generate_data target for running generator scripts make: Add generate_data target for running generator scripts Oct 6, 2023
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.

2 participants