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 man-page generation reproducible #112

Merged
merged 1 commit into from Sep 30, 2020
Merged

Conversation

bmwiedemann
Copy link
Contributor

@bmwiedemann bmwiedemann commented Sep 30, 2020

Make man-page generation reproducible

For this we behave similar to pod2man and use the input file mtime.

Also adapt date format to what doxygen-1.8.20 produces
to make the comment above true again.

LC_ALL=C is used so that %B is independent of language.

Also use UTC to be independent of timezone.

a simpler alternative approach would be to drop -d and accept pod2man's
ISO-8601 date format (e.g. 2019-10-15)

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

For this we behave similar to pod2man and use the input file mtime.

Also adapt date format to what doxygen-1.8.20 produces
to make the comment above true again.

LC_ALL=C is used so that %B is independent of language.

Also use UTC to be independent of timezone.

This patch was done while working on reproducible builds for openSUSE.
@troglobit
Copy link
Collaborator

Awesome, thank you!

@troglobit troglobit merged commit 72ebca3 into libnet:master Sep 30, 2020
1 check passed
@bmwiedemann bmwiedemann deleted the date branch September 30, 2020 06:16
@bmwiedemann
Copy link
Contributor Author

ah, one more note: the date -r is known to work with GNU date and FreeBSD date, but
https://man.openbsd.org/date.1 does not cover this usage. Maybe dropping the -d is simpler and more portable then.

@troglobit
Copy link
Collaborator

Yeah I missed your last content there. Going for std ISO date world be great and also simplify the code further. Mind giving that a whirl while you're at it?

@bmwiedemann
Copy link
Contributor Author

there you are: #113

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants