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

ghostscript: fix parallel building #86062

Merged
merged 1 commit into from May 8, 2020

Conversation

bennofs
Copy link
Contributor

@bennofs bennofs commented Apr 26, 2020

The build process of ghostscript has missing dependencies, causing
failures in parallel builds (see hydra build
https://hydra.nixos.org/build/117095669/, reported as ghostscript bug
https://bugs.ghostscript.com/show_bug.cgi?id=702364 here)

@bennofs
Copy link
Contributor Author

bennofs commented Apr 27, 2020

There is now also a patch available from upstream: http://git.ghostscript.com/?p=ghostpdl.git;a=commitdiff;h=1b4c3669a20c. Should we include that patch instead of disabling parallel make?

@FRidh
Copy link
Member

FRidh commented Apr 27, 2020

If there's a patch then let's just use it.

@FRidh FRidh added this to Needs review in Staging Apr 27, 2020
@bennofs
Copy link
Contributor Author

bennofs commented Apr 27, 2020

@FRidh agreed, changed to use patch

The build process of ghostscript has missing dependencies, causing
failures in parallel builds (see hydra build
https://hydra.nixos.org/build/117095669/, reported as ghostscript bug
https://bugs.ghostscript.com/show_bug.cgi?id=702364 here)
@bennofs
Copy link
Contributor Author

bennofs commented May 1, 2020

Oops, the patch needed to be rebased to our ghostscript version. Build is fixed now.

@bennofs bennofs changed the title ghostscript: disable parallel building ghostscript: fix parallel building May 1, 2020
@jtojnar jtojnar merged commit fc5f0be into NixOS:staging May 8, 2020
Staging automation moved this from Needs review to Done May 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Staging
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

3 participants