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

splix: enable JBIG #27027

Merged
merged 1 commit into from
Jul 2, 2017
Merged

splix: enable JBIG #27027

merged 1 commit into from
Jul 2, 2017

Conversation

emmanuelrosa
Copy link
Contributor

@emmanuelrosa emmanuelrosa commented Jul 1, 2017

Enables JBIG given that according to OpenPrinting.org, the patents have expired.
See https://www.openprinting.org/driver/splix/license/

Closes #25929

Motivation for this change
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
    • Linux
  • 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.

Sorry, something went wrong.

Enables JBIG given that according to OpenPrinting.org, the patents have expired.
See https://www.openprinting.org/driver/splix/license/

Closes NixOS#25929
@mention-bot
Copy link

@emmanuelrosa, thanks for your PR! By analyzing the history of the files in this pull request, we identified @peti, @urkud and @zimbatm to be potential reviewers.

@FRidh FRidh merged commit 9143cc6 into NixOS:master Jul 2, 2017
@FRidh
Copy link
Member

FRidh commented Jul 2, 2017

Ahh, I didn't check whether you tested it. Did you test it?

@emmanuelrosa
Copy link
Contributor Author

I attempted to, but the build never completed. Make and g++ ran, but even after running for a couple of hours no output was produced. Interestingly, I couldn't find any object files in the build's tmp directory. I don't know what was going on.

@emmanuelrosa emmanuelrosa deleted the splix-jbig branch July 2, 2017 12:44
@emmanuelrosa
Copy link
Contributor Author

I'm trying the build again. I'm baffled as to why so little C++ code would need more computation time to compile, but I'm giving it a go.

@emmanuelrosa emmanuelrosa mentioned this pull request Jul 2, 2017
8 tasks
@vcunat
Copy link
Member

vcunat commented Jul 3, 2017

Perhaps it needs jbig in build inputs, too... (though what you describe seems a different problem).

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