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

Remove the ERLANG_COMPILE_DATE timestamp from erl_version.h #5589

Conversation

JeromeDeBretagne
Copy link
Contributor

@JeromeDeBretagne JeromeDeBretagne commented Jan 6, 2022

Do not generate the ERLANG_COMPILE_DATE timestamp in erl_version.h with the make_version script to ensure reproducible builds.

While preparing this PR, I have found that mkver.c seems totally unused in the builds, so the first commit removes this file.

Linked to #4417

It was maybe used to generate erl_version.h in the past but it is not
used in builds anymore, since replaced by the make_version script.
@IngelaAndin IngelaAndin added the team:VM Assigned to OTP team VM label Jan 8, 2022
@JeromeDeBretagne JeromeDeBretagne force-pushed the jdb/erts/remove-ERLANG_COMPILE_DATE branch from 8901b59 to 86c84a4 Compare January 9, 2022 14:12
@rickard-green rickard-green self-assigned this Jan 10, 2022
@rickard-green rickard-green added the testing currently being tested, tag is used by OTP internal CI label Jan 10, 2022
Do not generate the ERLANG_COMPILE_DATE timestamp in erl_version.h
with the make_version script to ensure reproducible builds.

Remove the corresponding ERTS_SAVED_COMPILE_TIME option and update
the configure scripts accordingly.
@JeromeDeBretagne JeromeDeBretagne force-pushed the jdb/erts/remove-ERLANG_COMPILE_DATE branch from 86c84a4 to 384e4a9 Compare January 19, 2022 22:20
@JeromeDeBretagne
Copy link
Contributor Author

I've just pushed a minor change, simply updating the Copyright header date to 2022 in the modified files.

@rickard-green rickard-green added testing currently being tested, tag is used by OTP internal CI and removed testing currently being tested, tag is used by OTP internal CI labels Jan 20, 2022
@rickard-green rickard-green merged commit 384e4a9 into erlang:maint Jan 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team:VM Assigned to OTP team VM testing currently being tested, tag is used by OTP internal CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants