py-swiginac @1.5.1.1 - new port

[mbrethen]

Swiginac is a Python interface to GiNaC, built with SWIG. The aim of swiginac is to make all the functionality of GiNaC accessible from Python as an extension module.

[ryandesign (Ryan Carsten Schmidt)]

Thanks.

The construct "eval ... [glob ...]" is deprecated. It was used for Tcl 8.4 compatibility but now that MacPorts uses a bundled copy of Tcl 8.5 on all systems, the expansion operator {*} should be used instead of eval, as in "... {*}[glob ...]". I've added a lint warning for this in r142588.

[ryandesign (Ryan Carsten Schmidt)]

Replying to ryandesign@…:

I've added a lint warning for this in r142588.

When the next version of MacPorts from trunk is released, the lint warning would look like this:

$ port lint
--->  Verifying Portfile for py-swiginac
Warning: Line 47 should use the expansion operator instead of the eval procedure. Change
Warning:         eval xinstall -m 644 [glob ${worksrcpath}/doc/swiginac*.*] ${docdir}
Warning: to
Warning:         xinstall -m 644 {*}[glob ${worksrcpath}/doc/swiginac*.*] ${docdir}
Warning: Line 49 should use the expansion operator instead of the eval procedure. Change
Warning:         eval xinstall -m 644 [glob ${worksrcpath}/doc/examples/*.py] \
Warning: to
Warning:         xinstall -m 644 {*}[glob ${worksrcpath}/doc/examples/*.py] \
--->  0 errors and 2 warnings found.

[ryandesign (Ryan Carsten Schmidt)]

Please attach the file patch-setup.py.diff referenced in the portfile.

[mbrethen]

The portfile has been updated per your recommendations.

[ryandesign (Ryan Carsten Schmidt)]

Thanks. The original Portfile included this line:

xinstall -m 644 ${worksrcpath}/doc/html4css1.css ${docdir}

This was removed in your revised Portfile. Was that intentional?

[mbrethen]

No, clumsy typing. Fixed.

[ryandesign (Ryan Carsten Schmidt)]

Ok, committed in r142608.