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

[19.03] ghostscript: link against "systemwide" liblcms2 #57365

Merged
merged 1 commit into from Jun 9, 2019

Conversation

Mic92
Copy link
Member

@Mic92 Mic92 commented Mar 11, 2019

GS ships with a fork of lcms2 ("lcms2mt"), but the ABI separation
between the fork and the original seems insufficient. If libgs is linked
alongside liblcms2 (for example, this is the case with imagemagick) then
it will call into the original library instead of the fork, causing
segfaults.

Follow the example of both Arch and Debian in this regard -- they both
use the systemwide lib instead of the fork.

backport of #57313

(cherry picked from commit 128bb7b)

Motivation for this change
Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option 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/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Assured whether relevant documentation is up to date
  • Fits CONTRIBUTING.md.

@Mic92 Mic92 changed the title ghostscript: link against "systemwide" liblcms2 [19.03] ghostscript: link against "systemwide" liblcms2 Mar 11, 2019
@MaxGabriel
Copy link

Hey all, is anything blocking this from getting merged (besides the new merge conflict)?

GS ships with a fork of lcms2 ("lcms2mt"), but the ABI separation
between the fork and the original seems insufficient. If libgs is linked
alongside liblcms2 (for example, this is the case with imagemagick) then
it will call into the original library instead of the fork, causing
segfaults.

Follow the example of both Arch and Debian in this regard -- they both
use the systemwide lib instead of the fork.

(cherry picked from commit 128bb7b)
@Mic92
Copy link
Member Author

Mic92 commented Jun 9, 2019

@GrahamcOfBorg build ghostscript

@Mic92 Mic92 merged commit 2cc45e8 into NixOS:release-19.03 Jun 9, 2019
@Mic92 Mic92 deleted the ghostscript branch June 9, 2019 16:32
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