[80194] trunk/dports/net/wireshark/Portfile

[Ryan Schmidt]

On Jul 6, 2011, at 11:10, ricci at macports.org wrote:

> Revision: 80194
>          http://trac.macports.org/changeset/80194
> Author:   ricci at macports.org
> Date:     2011-07-06 09:10:47 -0700 (Wed, 06 Jul 2011)
> Log Message:
> -----------
> wireshark: upgrade to 1.6.0.  Make the (experimental) python support variants for py25, py26, and py27


> +variant python25 description {use python25 for the experimental python interface} conflicts python26 python27 {
> +	configure.args-append	--with-python=${prefix}
> +	configure.args-delete	--without-python
> +	depends_lib-append	port:python25
> +}
> +
> +variant python26 description {use python26 for the experimental python interface} conflicts python25 python27 {
> +	configure.args-append	--with-python=${prefix}
> +	configure.args-delete	--without-python
> +	depends_lib-append	port:python26
> +}
> +
> +variant python27 description {use python27 for the experimental python interface} conflicts python25 python26 {
> +	configure.args-append	--with-python=${prefix}
> +	configure.args-delete	--without-python
> +	depends_lib-append	port:python27
> +}

These three variants all do the same thing: set --with-python to ${prefix}; surely that's not right? Surely each variant needs to set --with-python to the path to the specific python version being requested?

Also, once you sort this out, can the same changes be made to the wireshark-devel port? wireshark and wireshark-devel should be kept as similar as possible.