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

redo-apenwarr: switch to python3 and fix building man #83134

Merged
merged 1 commit into from Mar 22, 2020

Conversation

ck3d
Copy link
Contributor

@ck3d ck3d commented Mar 22, 2020

removed dependency to mkdoc, which is only needed to build the webpage.
Python3 is not supported in 0.42, so I had to update to current master.

Motivation for this change

switch to python3

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.

Removed dependency to mkdoc, which is only needed to build the webpage.
@markuskowa markuskowa merged commit ca7f78a into NixOS:master Mar 22, 2020
@markuskowa
Copy link
Member

Backported to 20.03 in bdd53db
(ZHF #80379)

@danderson
Copy link
Contributor

We raced each other! I was about to send the same PR :).

Just for the record: upstream released 0.42a specifically for python3 support, so the upgrade here is not to HEAD per se, but to the next official upstream release.

Thanks for fixing!

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

4 participants