[43663] trunk/dports/x11/fox/Portfile

[Ryan Schmidt]

On Dec 13, 2008, at 01:24, jeremyhu at macports.org wrote:

> Revision: 43663
>           http://trac.macports.org/changeset/43663
> Author:   jeremyhu at macports.org
> Date:     2008-12-12 23:24:42 -0800 (Fri, 12 Dec 2008)
> Log Message:
> -----------
> fox: Removed Xcode 3.0.0 libGL linking bug workaround since it's  
> been fixed for 3 versions now.

Do you mean this problem doesn't occur anymore in the last 3 versions  
of Xcode (3.1.2, 3.1.1, 3.1.0)? If so, that's (good) news to me. But  
the Apple Developer Connection article on the problem doesn't mention  
that:

http://developer.apple.com/qa/qa2007/qa1567.html


If we're going to remove that bit from portfiles, we might want to in  
its place put a check to ensure the user has at least Xcode 3.1, e.g.:

pre-fetch {
     if {${os.platform} == "darwin" && ${os.major} == 9} {
         set minimum_xcodeversion 3.1
         set current_xcodeversion [exec defaults read /Developer/ 
Applications/Xcode.app/Contents/Info CFBundleShortVersionString]
         if {[rpm-vercomp ${current_xcodeversion} $ 
{minimum_xcodeversion}] < 0} {
             return -code error "You have Xcode $ 
{current_xcodeversion}. Please update to at least Xcode $ 
{minimum_xcodeversion}."
         }
     }
}

I do something similar in cairo to ensure the user has at least Xcode  
2.4.1 on Tiger.


> Modified Paths:
> --------------
>     trunk/dports/x11/fox/Portfile
>
> Modified: trunk/dports/x11/fox/Portfile
> ===================================================================
> --- trunk/dports/x11/fox/Portfile	2008-12-13 07:18:48 UTC (rev 43662)
> +++ trunk/dports/x11/fox/Portfile	2008-12-13 07:24:42 UTC (rev 43663)
> @@ -25,9 +25,6 @@
>  	lib:libXrandr.2:xorg-libXrandr
>
>  configure.cppflags-append	"-I${prefix}/include -I${x11prefix}/ 
> include"
> -platform darwin 9 {
> -configure.ldflags-append	"-dylib_file /System/Library/Frameworks/ 
> OpenGL.framework/Versions/A/Libraries/libGL.dylib:/System/Library/ 
> Frameworks/OpenGL.framework/Versions/A/Libraries/libGL.dylib"
> -}
>
>  configure.pre_args  --prefix=${prefix} --mandir=${prefix}/share/man
>  configure.args      --with-x --with-xft --with-opengl --enable-cups