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

man: obey SOURCE_DATE_EPOCH when generating man pages #1306

Merged
merged 1 commit into from
Nov 29, 2018

Conversation

cyphar
Copy link
Contributor

@cyphar cyphar commented Aug 23, 2018

Previously our man pages included the current time each time they were
generated. This causes an issue for reproducible builds, since each
re-build of a package that includes the man pages will have different
times listed in the man pages.

To fix this, add support for SOURCE_DATE_EPOCH (which is a standardised
packaging environment variable, designed to be used specifically for
this purpose
). spf13/cobra doesn't support this natively yet (though
I will push a patch for that as well), but it's simpler to fix it
directly in docker/cli.

Signed-off-by: Aleksa Sarai asarai@suse.de

Previously our man pages included the current time each time they were
generated. This causes an issue for reproducible builds, since each
re-build of a package that includes the man pages will have different
times listed in the man pages.

To fix this, add support for SOURCE_DATE_EPOCH (which is a standardised
packaging environment variable, designed to be used specifically for
this purpose[1]). spf13/cobra doesn't support this natively yet (though
I will push a patch for that as well), but it's simpler to fix it
directly in docker/cli.

[1]: https://reproducible-builds.org/specs/source-date-epoch/

Signed-off-by: Aleksa Sarai <asarai@suse.de>
@codecov-io
Copy link

Codecov Report

Merging #1306 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1306   +/-   ##
=======================================
  Coverage   54.73%   54.73%           
=======================================
  Files         292      292           
  Lines       19267    19267           
=======================================
  Hits        10545    10545           
  Misses       8063     8063           
  Partials      659      659

@bmwiedemann
Copy link

Related: spf13/cobra#667

Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

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

LGTM

@cyphar
Copy link
Contributor Author

cyphar commented Nov 29, 2018

/ping

Copy link
Collaborator

@vdemeester vdemeester left a comment

Choose a reason for hiding this comment

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

LGTM 🐯

@vdemeester vdemeester merged commit f64dc97 into docker:master Nov 29, 2018
@GordonTheTurtle GordonTheTurtle added this to the 19.03.0 milestone Nov 29, 2018
@cyphar cyphar deleted the obey-source_date_epoch branch November 29, 2018 10:21
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.

6 participants