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 autogen.man reproducible #103974

Merged

Conversation

basile-henry
Copy link
Contributor

Motivation for this change

Working towards a reproducible iso_minimal (https://r13y.com/)

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Ensured that relevant documentation is up to date
  • Fits CONTRIBUTING.md.

@vikanezrimaya
Copy link
Member

I feel like this is something that should be merged in staging, judging by how many packages need to be rebuilt.

@doronbehar
Copy link
Contributor

Doesn't seem necessary to put this in 2 different commits. and yes it needs to target staging.

Keep the date format similar to what it is when we are not using the
MAN_PAGE_DATE env var. The man pages scripts seem to use different
formats in different pages, so we use the most common format for all.
@picnoir
Copy link
Member

picnoir commented Nov 16, 2020

It seems like we were working on that concurently :) #103986

Note we have a IRC channel to sync the r13y effort, #nixos-r13y on freenode.

[Edit] first 😅

@zimbatm

This comment has been minimized.

@basile-henry
Copy link
Contributor Author

Ah, sorry about the duplicated efforts @NinjaTrappeur! 😅 I'll join the IRC channel to coordinate for future r13y work 👍

@basile-henry
Copy link
Contributor Author

Is there anything I need to do to get this PR over the line? 😄

Copy link
Member

@picnoir picnoir left a comment

Choose a reason for hiding this comment

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

LGTM

@picnoir picnoir merged commit 3e4afc2 into NixOS:staging Nov 18, 2020
@basile-henry basile-henry deleted the basile-henry/reproducible-autogen-man branch November 18, 2020 16:16
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.

None yet

5 participants