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

Do not discover user/host for reproducible builds #267

Merged
merged 2 commits into from Aug 4, 2023

Conversation

witekest
Copy link
Contributor

Allow to overwrite build user if SOURCE_DATE_EPOCH is set in order to make builds reproducible.
See https://reproducible-builds.org/ for why this is good and https://reproducible-builds.org/specs/source-date-epoch/ for the definition of this variable.

This PR was done while working on reproducible builds for openSUSE.

Allow to overwrite build user if SOURCE_DATE_EPOCH is set in order to
make builds reproducible.
See https://reproducible-builds.org/ for why this is good
and https://reproducible-builds.org/specs/source-date-epoch/
for the definition of this variable.

This PR was done while working on [reproducible builds for
openSUSE](https://en.opensuse.org/openSUSE:Reproducible_Builds).

Signed-off-by: Witek Bedyk <witold.bedyk@suse.com>
@roidelapluie
Copy link
Member

Can we abstract os.Getenv(sourceDateEpoch) == "" into a isReproducible() bool function?

The function should have a comment that says that when we think we are in a reproducible scenario, we try to do our best, but there is no guarantee that promu generates reproducible builds on its own.

Signed-off-by: Witek Bedyk <witold.bedyk@suse.com>
@roidelapluie roidelapluie merged commit c07f9af into prometheus:master Aug 4, 2023
4 checks passed
@witekest witekest deleted the reproducible_builds branch August 8, 2023 11:17
SuperQ added a commit that referenced this pull request Apr 2, 2024
* [FEATURE] Do not discover user/host for reproducible builds #267
* [BUGFIX] Fix exmaple/prometheus build error #274

Signed-off-by: SuperQ <superq@gmail.com>
@SuperQ SuperQ mentioned this pull request Apr 2, 2024
SuperQ added a commit that referenced this pull request Apr 2, 2024
* [FEATURE] Do not discover user/host for reproducible builds #267
* [BUGFIX] Fix example/prometheus build error #274

Signed-off-by: SuperQ <superq@gmail.com>
danielmellado pushed a commit to danielmellado/prometheus-promu that referenced this pull request Apr 24, 2024
* [FEATURE] Do not discover user/host for reproducible builds prometheus#267
* [BUGFIX] Fix example/prometheus build error prometheus#274

Signed-off-by: SuperQ <superq@gmail.com>
danielmellado pushed a commit to danielmellado/prometheus-promu that referenced this pull request Apr 24, 2024
* [FEATURE] Do not discover user/host for reproducible builds prometheus#267
* [BUGFIX] Fix example/prometheus build error prometheus#274

Signed-off-by: SuperQ <superq@gmail.com>
Signed-off-by: Daniel Mellado <dmellado@redhat.com>
danielmellado pushed a commit to danielmellado/prometheus-promu that referenced this pull request Apr 24, 2024
* [FEATURE] Do not discover user/host for reproducible builds prometheus#267
* [BUGFIX] Fix example/prometheus build error prometheus#274

Signed-off-by: SuperQ <superq@gmail.com>
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.

None yet

2 participants