Merge lp:~posulliv/mysql-sandbox/make-source-issue into lp:mysql-sandbox

[Padraig O'Sullivan]

[Padraig O'Sullivan (posulliv)]

I was trying to create a sandbox from a 5.1 source tree and ran in to some issues with the parsing of the configure.in file. It has a different format in 5.1 source trees.

This is my primitive and trivial fix as in it works for me. Thought I'd propose it for merging just in case it might prove useful.

Thanks,
Padraig

[Giuseppe Maxia (giuseppe-maxia)]

Fixed in 3.0.08. (rev 36)
I used a slightly different change to get to make the regex more robust.

    while (my $line = <$CONFIG>) {
        if ($line =~ /
                    (?:AM_INIT_AUTOMAKE
                    |
                    AC_INIT) # either of these macros
                    \D+ # followed by one or more non digit
                    (?:(?i)mysql) # followed by "mysql" in any case
                    \D+ # followed by one or more non digit
                    (\d\.\d.\d+) # capture the version
                    /x) {
            $version = $1;
            last;
        }
    }

So, I disapprove your change (because I fixed it otherwise), but I warmly thank you for noticing it!

review: Disapprove

[Padraig O'Sullivan (posulliv)]

Cool! Thanks for looking at it. I am very unfamiliar with Perl so I figured my fix was a little bit primitive. Thank you for making it more robust and now I learned a little bit of Perl :)

Unmerged revisions

34. By Padraig O'Sullivan on 2010-02-16

Corrected issue with creating sandbox from 5.1 source trees.

=== modified file 'bin/make_sandbox_from_source'
--- bin/make_sandbox_from_source	2010-01-03 22:12:45 +0000
+++ bin/make_sandbox_from_source	2010-02-16 22:17:12 +0000
@@ -134,7 +134,8 @@
         or die "can't find configure.in\n";
     my $version;
     while (my $line = <$CONFIG>) {
-        if ($line =~ /AM_INIT_AUTOMAKE\D+(\d\.\d.\d+)/) {
+        if ($line =~ /AM_INIT_AUTOMAKE\D+(\d\.\d.\d+)/ ||
+            $line =~ /AC_INIT\D+(\d\.\d.\d+)/) {
             $version = $1;
             last;
         }