[rt.cpan.org #94367] [PATCH] wx-config --version and wxwidgets3.0

[Andreas Voegele via RT]

Sun Aug 17 12:25:07 2014: Request 94367 was acted upon.
Transaction: Correspondence added by andr...@andreasvoegele.com
       Queue: Alien-wxWidgets
     Subject: [PATCH] wx-config --version and wxwidgets3.0
   Broken in: (no value)
    Severity: (no value)
       Owner: Nobody
  Requestors: gre...@debian.org
      Status: new
 Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=94367 >


Am Di 01. Apr 2014, 13:09:12, gre...@debian.org schrieb:
> 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.




--- 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' ;