Bug #94367 for Alien-wxWidgets: [PATCH] wx-config --version and wxwidgets3.0

Tue Apr 01 13:09:12 2014 gregoa [...] debian.org - Ticket created

Subject:	[PATCH] wx-config --version and wxwidgets3.0
Date:	Tue, 1 Apr 2014 19:08:51 +0200
To:	bug-alien-wxwidgets [...] rt.cpan.org
From:	gregor herrmann <gregoa [...] debian.org>

In Debian we are currently applying the attached patch to Alien-wxWidgets. We thought you might be interested in it, too. Description: adjust Build.PL to wxwidgets3.0 allow ^3 as an answer of `wx-config --version' for system wxwidgets Origin: vendor Author: gregor herrmann <gregoa@debian.org> Last-Update: 2014-04-01 Thanks in advance, gregor herrmann, Debian Perl Group

Message body is not shown because sender requested not to inline it.

Sun Aug 17 12:25:07 2014 andreas [...] andreasvoegele.com - Correspondence added

From:

andreas [...] andreasvoegele.com

Am Di 01. Apr 2014, 13:09:12, gregoa@debian.org schrieb: Show quoted text

> In Debian we are currently applying the attached patch to Alien-wxWidgets. > [...]

And shouldn't Build.PL also respect the WX_CONFIG environment variable mentioned in the README.TXT file? The attached patch includes the patch from Debian and my fix as both changes affect the same part of Build.PL.

Subject:

wx-config-version-and-env.patch

--- Build.PL.orig 2014-03-23 15:48:03.000000000 +0100 +++ Build.PL 2014-08-17 17:43:36.000000000 +0200 @@ -150,11 +150,11 @@ EOP } else { require My::Build::Base; - my $wx_config = My::Build::Base->awx_path_search( 'wx-config' ); + my $wx_config = $ENV{WX_CONFIG} || My::Build::Base->awx_path_search( 'wx-config' ); if( $wx_config ) { - my $ans = `wx-config --version`; - if( $ans =~ /^2\./ ) { - my $prefix = `wx-config --prefix`; + my $ans = `$wx_config --version`; + if( $ans =~ /^[23]\./ ) { + my $prefix = `$wx_config --prefix`; chomp foreach $ans, $prefix; if( _check_installed_widgets_ok($ans, $prefix) ) { $build_wx_dflt = 'no' ;

Sun Aug 17 12:25:07 2014 The RT System itself - Status changed from 'new' to 'open'