-
-
Notifications
You must be signed in to change notification settings - Fork 14.7k
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
tetex: remove override libpng #270107
tetex: remove override libpng #270107
Conversation
@pbsds so moving the |
I'm not blocking, but in #259879 we found that the override should be kept, not inlined. |
I am reading: |
And in that thread I and kirilldry challenge the issue, since the by-name README states that using |
8a72d2e
to
16c1bde
Compare
@@ -1,4 +1,4 @@ | |||
{ lib, stdenv, fetchurl, flex, bison, zlib, libpng, ncurses, ed, automake }: | |||
{ lib, stdenv, fetchurl, flex, bison, zlib, libpng12, libpng ? libpng12, ncurses, ed, automake }: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
error: builder for '/nix/store/hapr60a6qhjwd2hhf5ljz2334pmfp5ll-tetex-3.0.drv' failed with exit code 2;
last 10 log lines:
> writepng.c:152:45: error: invalid use of incomplete typedef 'png_info' {aka 'struct png_info_def'}
> 152 | pdfbuf[pdfptr++] = png_info(img)->palette[i].blue;
> | ^~
The issue is explained here #259879 (comment)
Description of changes
#259879
with
libpng
it does not compile.Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Priorities
Add a 👍 reaction to pull requests you find important.