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

pelican: Disable pandoc support for now #33175

Merged
merged 1 commit into from Dec 29, 2017

Conversation

johbo
Copy link
Contributor

@johbo johbo commented Dec 29, 2017

Version 3.7.1 is not compatible with the current (more recent) pandoc version.
Since pandoc support is optional in pelican, we can ship it without pandoc
support until a fix will be applied upstream.

Kept a note in the buildInputs so that the improvement opportunity can be
spotted again on future updates of pelican.

Motivation for this change

Noticed that the build of pelican is failing since a few days: https://hydra.nixos.org/job/nixpkgs/trunk/python36Packages.pelican.x86_64-linux

Got hit by it myself a few hours ago.

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
  • 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 nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

Version 3.7.1 is not compatible with the current (more recent) pandoc version.
Since pandoc support is optional in pelican, we can ship it without pandoc
support until a fix will be applied upstream.

Kept a note in the buildInputs so that the improvement opportunity can be
spotted again on future updates of pelican.
@johbo johbo requested a review from FRidh as a code owner December 29, 2017 17:20
@FRidh
Copy link
Member

FRidh commented Dec 29, 2017

How did it even work? During build-time it can find pandoc but afterwards it won't because it will no longer have any reference to it.

@johbo
Copy link
Contributor Author

johbo commented Dec 29, 2017

Actually that's a valid point, my assumption is that it did not work. I did not try it out myself though, I am not using the import feature.

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

3 participants