Skip to content
This repository has been archived by the owner on Jul 4, 2023. It is now read-only.

pstoedit: add X11 dependency #28595

Closed
wants to merge 1 commit into from
Closed

pstoedit: add X11 dependency #28595

wants to merge 1 commit into from

Conversation

ianml
Copy link
Contributor

@ianml ianml commented Apr 22, 2014

Otherwise fails with ld: library not found for -lXext:

/bin/sh ../libtool --tag=CXX   --mode=link clang++  -g -O2 -DINTERNALBOOL -no-undefined -module  -o libp2edrvmagick++.la -rpath /usr/local/Cellar/pstoedit/3.62_1/lib/pstoedit drvmagick++.lo initlibrary.lo -L/usr/local/Cellar/imagemagick/6.8.7-7/lib -lMagick++-6.Q16 -lMagickWand-6.Q16 -lMagickCore-6.Q16  libpstoedit.la -lgd 
libtool: link: clang++  -o .libs/libp2edrvmagick++.0.so -bundle  .libs/drvmagick++.o .libs/initlibrary.o   -L/usr/local/Cellar/imagemagick/6.8.7-7/lib /usr/local/Cellar/imagemagick/6.8.7-7/lib/libMagick++-6.Q16.dylib -L/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.9.sdk/usr/lib -L/usr/local/Cellar/freetype/2.5.2/lib -L/opt/X11/lib /usr/local/Cellar/imagemagick/6.8.7-7/lib/libMagickWand-6.Q16.dylib /usr/local/Cellar/imagemagick/6.8.7-7/lib/libMagickCore-6.Q16.dylib -llcms2 -lfftw3 -lfontconfig -lfreetype -lXext -lSM -lICE -lX11 -lXt -lbz2 -lz -lltdl ./.libs/libpstoedit.dylib -lstdc++ -ldl -lm -lgd  -O2   -pthread
ld: library not found for -lXext
clang: error: linker command failed with exit code 1 (use -v to see invocation)
make[1]: *** [libp2edrvmagick++.la] Error 1
make: *** [install-recursive] Error 1

@@ -4,12 +4,13 @@ class Pstoedit < Formula
homepage 'http://www.pstoedit.net'
url 'https://downloads.sourceforge.net/project/pstoedit/pstoedit/3.62/pstoedit-3.62.tar.gz'
sha1 '50d5a4e2fe0e0ff2f73cb094cb945b221083e742'
revision 1
revision 2
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This doesn't need bumped.

@ianml
Copy link
Contributor Author

ianml commented Apr 23, 2014

Seems like a duplicate of #25271. I'll need to look into this; don't have X11 in my env and this package shouldn't need it.

It seems silly to add depends_on :x11 if MacOS::X11.installed?

@ianml ianml closed this Apr 23, 2014
@ianml ianml deleted the pstoedit-x11 branch December 10, 2014 04:06
@Homebrew Homebrew locked and limited conversation to collaborators Feb 17, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants