Skip to content

has_lib.js gif detects libgif, but needs to detect libgif-dev specifically #1570

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

Open
domoritz opened this issue Apr 30, 2020 · 2 comments
Open
Labels

Comments

@domoritz
Copy link
Contributor

domoritz commented Apr 30, 2020

I am trying to install node canvas without libgif-dev.

The readme says it's optional

Screen Shot 2020-04-30 at 15 47 17

But my builds fail when I don't have it installed
Screen Shot 2020-04-30 at 15 45 27

Is this expected?

Your Environment

https://github.com/vega/vega-lite/pull/6406/checks?check_run_id=634869864

@zbjornson
Copy link
Collaborator

It looks like the GitHub Actions Ubuntu runner has libgif7 (giflib) already installed, but not libgif-dev, so the headers are missing:

$ ldconfig -p 2>/dev/null | grep -E gif
libgif.so.7 (libc6,x86-64) => /usr/lib/x86_64-linux-gnu/libgif.so.7

$ apt list --installed | grep gif
libgif7/bionic-updates,bionic-security,now 5.1.4-2ubuntu0.1 amd64 [installed,automatic]

(From https://github.com/zbjornson/node-canvas/runs/635551205?check_suite_focus=true)

I can't find the issue where this was discussed, but instead of using ldconfig we should be doing something like gcc -lgif and seeing if it succeeds.

The easiest fix for now is to install the other dependencies. Prebuilds for v14 should be here soon (#1568).

@zbjornson zbjornson added the Build label May 1, 2020
@domoritz
Copy link
Contributor Author

domoritz commented May 1, 2020

Thanks for the update and for pointing me to #1568.

@domoritz domoritz changed the title Build needs gif even though t's supposed to be optional Build needs gif even though it's supposed to be optional May 3, 2020
@zbjornson zbjornson changed the title Build needs gif even though it's supposed to be optional has_lib.js gif detects libgif, but needs to detect libgif-dev specifically May 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants