In the svn version (11014 at the time off this post), bootsrap-sh tells that automake version 1.10 is older than 1.5. With this patch the version number will be compare like version number not like string.
Actually, the patch doesn't work for me. I uninstalled automake1.9 to test and I got the same error as I do in the unpatched file. I don't think it's an svn issue since the file has been unchanged for 16 months.
EDIT: Apparently this is due to an issue with the debian package for automake1.10. When I have automake1.9 installed and run the current bootstrap-sh in SVN, it runs fine. With the patch it gives me the version error instead.
I don't think I can commit this patch to the trunk since it appears to break the bootstrap script for users of debian [me] and debian-based distributions such as ubuntu. This was a good effort though- please let me know if you find a way to fix this issue for most people without breaking compatibility with debian.
It could be a Debian issue... I'm also using debian... but I'm pretty sure when I looked, it had the proper version of 1.10.
I know that the existing problem is due to a lexographic compare rather than a numerical compare ( "<" vs. -lt ) (1 < 10 < 2 < 9 while 1 -lt 2 -lt 9 -lt 10 )
This should cause problems on any version of Linux, but it depends on the quantity of autoconf's installed (or their version numbers), which is where distributions like Debian work differently.
This can be fixed by splitting up the strings using cut or something else, so that the "1.nn" gets stipped down to the minor number "nn"... Then, I can use the "-lt" option of test for numerical compare.
The AWK script looks like a more complicated way of doing it... I'll try to commit a fix to this using shell commands tonight....
I also prefer the use of basic unix tools (cut, paste, shell logic) in place of languages like AWK when possible.
While you're patching it, do you think you can fix the missing GLUT bug as well?
The patch work on my ubuntu. Strange it did not work on debian, maybe awk compatibility thing.
I know there a lot of of awk variant (classic unix, posix, gawk addition) and it can cause a lot of problem.
I thought it was not big deal because awk was already used in the bootstrap-sh script and I was lazy. I'm not an experience shell script programmer, I didn't know how to manipulate string in shell script and isolate the 10 in 1.10.
At least it was useful for me, sorry it wasn't for you halleck.
I will look at ace123 fix, I'll probably learn something.
No worries, thanks again for taking the time to at least try to fix that gargantuan and scary script.
Hopefully ace will have better (more portable) luck.
I'm trying to test this but I think there are whitespace problems in the diff due to copy-and-paste across browsers... I can try testing it again if you upload a patch file.
Running aclocal
/usr/share/aclocal/smpeg.m4:13: warning: underquoted definition of AM_PATH_SMPEG/usr/share/aclocal/smpeg.m4:13: run info '(automake)Extending aclocal'
/usr/share/aclocal/smpeg.m4:13: or see http://sources.redhat.com/automake/automake.html#Extending-aclocal
Running autoheader
Running autoconf
Running automake
Makefile.am:1099: `%'-style pattern rules are a GNU make extension
Bootstrap is complete. Run ./configure to configure the build system.
Running aclocal
/usr/share/aclocal/smpeg.m4:13: warning: underquoted definition of AM_PATH_SMPEG run info '(automake)Extending aclocal'
or see http://sources.redhat.com/automake/automake.html#Extending-aclocal
Running autoheader
Running autoconf
Running automake
Bootstrap is complete. Run ./configure to configure the build system.
Which appears identical to the unpatched output.
This is certainly a step in the right direction... although I think I should try a complete build too. Let's see what ace wants to do to the script as well.
I just committed your fix. Sorry for taking so long.
I don't see any reason that it would not work... it now treats them like numbers. I tested it with versions below the ver_needed, and it failed in the appropriate places.