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

Adding optional trailing .8 for has_lib searches to prevent false negatives for JPEG lib #271

Closed

Conversation

twolfson
Copy link

This patch adds the conditional for a .8 ending to the libjpeg.so file.

Debug notes

I recently moved to a new laptop on Linux Mint 14 Cinnamon and was running into the same issues as #254.

I tried moving back to the old version of Cairo, 1.10.2, which had worked on my older machine but no luck. I verified multiple times that I had the proper apt-get dependencies met as listed in the wiki.

After some more debugging, I found that ./util/has_lib.sh jpeg was returning false when it should be true. Upon grepping my ldconfig, I found the following:

$ ldconfig -p | grep jpeg
    libopenjpeg.so.2 (libc6,x86-64) => /usr/lib/x86_64-linux-gnu/libopenjpeg.so.2
    libmjpegutils-1.9.so.0 (libc6,x86-64) => /usr/lib/libmjpegutils-1.9.so.0
    liblavjpeg-1.9.so.0 (libc6,x86-64) => /usr/lib/liblavjpeg-1.9.so.0
    libjpeg.so.8 (libc6,x86-64) => /usr/lib/x86_64-linux-gnu/libjpeg.so.8
    libjpeg.so.8 (libc6) => /usr/lib/i386-linux-gnu/libjpeg.so.8

As a result, I added in my conditional, re-built, and everything magically went back to working.

@gauze
Copy link

gauze commented Apr 26, 2013

I had to add .62 to mine to get it to link against that version

@kangax
Copy link
Collaborator

kangax commented Oct 9, 2013

This should be now fixed with #327

@kangax kangax closed this Oct 9, 2013
@twolfson twolfson deleted the bug/false.negative.for.jpeg branch October 9, 2013 21:26
@twolfson
Copy link
Author

twolfson commented Oct 9, 2013

Awesome =)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants