[68792] trunk/dports/lang/python26

[Blair Zajac]

On Jun 13, 2010, at 7:21 PM, Bryan Blackburn wrote:

> On Sun, Jun 13, 2010 at 07:02:33PM -0700, Blair Zajac said:
>>
>> On Jun 13, 2010, at 6:58 PM, Bryan Blackburn wrote:
>>
>>> On Sun, Jun 13, 2010 at 08:41:53PM -0500, Ryan Schmidt said:
>>>>
>>>> On Jun 13, 2010, at 15:49, blb at macports.org wrote:
>>>>
>>>>> Revision: 68792
>>>>>       http://trac.macports.org/changeset/68792
>>>>> Author:   blb at macports.org
>>>>> Date:     2010-06-13 13:49:38 -0700 (Sun, 13 Jun 2010)
>>>>> Log Message:
>>>>> -----------
>>>>> lang/python26 - do not link in X11 when tk is installed with
>>>>> +quartz, ticket #25221
>>>>
>>>>
>>>>> +   # Check for whether tk is installed for X11 or Quartz (to
>>>>> avoid trying
>>>>> +   # to link to X11 in the Quartz case, ticket #25221)
>>>>> +   if {[catch {exec grep "^TK_LIBS.*X11" ${prefix}/lib/
>>>>> tkConfig.sh > /dev/null 2>@1} results options]} {
>>>>> +      # Error from grep means not found, so Quartz
>>>>> +      system "cd ${worksrcpath} && patch -p0 <
>>>>> ${filespath}/patch-setup.py_tkquartz.diff"
>>>>> +   }
>>>>
>>>> That ought to be "2>&1" not "2>@1", right?
>>>
>>> Not in Tcl's exec, where @ is used for file id's instead of &.  See
>>> 'man n
>>> exec' for all the gory details...
>>
>> Well, something isn't working there on 10.5 ppc:
>
> Yup, was using a Tcl 8.5+ feature to catch, fixed in r68800.

Great, thanks!

Blair