[112776] trunk/dports/lang/clisp/Portfile

[Daniel J. Luke]

On Nov 11, 2013, at 9:26 AM, Ryan Schmidt <ryandesign at macports.org> wrote:
> 
> On Nov 11, 2013, at 08:20, Mark Evenson wrote:
>> Sure: my TCL fu is not that great, so I always a little to hesitant to move outside the patterns that I know work in Portfiles, but getting the right conditionalized stanza that is somewhat future proof is certainly the right thing to do 
> 
> It should just be:
> 
> if {${os.platform} eq "darwin" && ${os.major} >= 11} {
>    configure.cflags-append -Wl,-no_pie
> }

why not 

platform darwin {
	if {${os.major} >= 11} {
		configure.cflags-append -Wl,-no_pie
	}
}

which doesn't loose the platform block? [aside: didn't we have a proposal for more flexible platform descriptions at some point so you could just write this in a normal platform block?]

--
Daniel J. Luke                                                                   
+========================================================+                        
| *---------------- dluke at geeklair.net ----------------* |                          
| *-------------- http://www.geeklair.net -------------* |                          
+========================================================+                        
|   Opinions expressed are mine and do not necessarily   |                          
|          reflect the opinions of my employer.          |                          
+========================================================+