[68792] trunk/dports/lang/python26

[Bryan Blackburn]

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.

Bryan

[...]
> 
> Regards,
> Blair
>