Re: bootstrap breakage starting with fec7d87
On 2013-11-19 10:08, Ozkan Sezer wrote: Starting with fec7d87 (funclib.sh: simplify version comparison functions) I am getting the following error from bootstrap: bootstrap: error: 'makeinfo' version == 4.13 is too old bootstrap:'makeinfo' version = 4.8 is required 9fd7b88 is fine. This is with Fedora 16, with grep-2.9-3.fc16.x86_64, sed-4.2.1-9.fc16.x86_64, and bash-4.2.37-1.fc16.x86_64 Will this be fixed anytime soon? Yes, please. I came up with this patch, but I don't know how portable it is, so I would like someone knowledgeable to comment on it before pushing it... Cheers, Peter From a7462c5563e124e06f4f61ce2a9cea26cf8be390 Mon Sep 17 00:00:00 2001 From: Peter Rosin p...@lysator.liu.se Date: Tue, 19 Nov 2013 11:54:27 +0100 Subject: [PATCH] bootstrap: fix version sort Reported by Ozkan Sezer who suffered from makeinfo 4.13 being detected as lesser than the required makeinfo 4.8. * bootstrap (func_sort_ver): Sort numerically on the non-primary keys as well. --- bootstrap |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/bootstrap b/bootstrap index 1b16d95..852efd5 100755 --- a/bootstrap +++ b/bootstrap @@ -1323,7 +1323,7 @@ func_sort_ver () $debug_cmd printf '%s\n%s\n' $1 $2 | -sort -t. -k1n -k1 -k2n -k2 -k3n -k3 -k4n -k4 -k5n -k5 -k6n -k6 -k7n -k7 -k8n -k8 -k9n -k9 +sort -t. -k 1,1n -k 2,2n -k 3,3n -k 4,4n -k 5,5n -k 6,6n -k 7,7n -k 8,8n -k 9,9n } # func_lt_ver PREV CURR -- 1.7.9
Re: bootstrap breakage starting with fec7d87
Hi Ozkan, Petor, On Nov 19, 2013, at 11:57 PM, Peter Rosin p...@lysator.liu.se wrote: On 2013-11-19 10:08, Ozkan Sezer wrote: Starting with fec7d87 (funclib.sh: simplify version comparison functions) I am getting the following error from bootstrap: bootstrap: error: 'makeinfo' version == 4.13 is too old bootstrap:'makeinfo' version = 4.8 is required 9fd7b88 is fine. This is with Fedora 16, with grep-2.9-3.fc16.x86_64, sed-4.2.1-9.fc16.x86_64, and bash-4.2.37-1.fc16.x86_64 Will this be fixed anytime soon? Yes, please. Ouch, sorry about that. I came up with this patch, but I don't know how portable it is, so I would like someone knowledgeable to comment on it before pushing it... I don't have access to any Windows environments, but your patch works correctly for me on various flavours of Mac OS, GNU/Linux, Solaris, HPUX, and AIX -- I no longer have access to Tru64 Unix, SCO Unix or IRIX. Thanks for the quick fix. Assuming it works on cygwin, mingw and other windows variants we support, please go ahead and push. Cheers, -- Gary V. Vaughan (gary AT gnu DOT org) signature.asc Description: Message signed with OpenPGP using GPGMail
Re: bootstrap breakage starting with fec7d87
On Nov 20, 2013, at 12:32 PM, Gary V. Vaughan g...@gnu.org wrote: Hi Ozkan, Petor, ^H^H^H^H^H^H^HPeter, *blush* signature.asc Description: Message signed with OpenPGP using GPGMail
Re: bootstrap breakage starting with fec7d87
Starting with fec7d87 (funclib.sh: simplify version comparison functions) I am getting the following error from bootstrap: bootstrap: error: 'makeinfo' version == 4.13 is too old bootstrap:'makeinfo' version = 4.8 is required 9fd7b88 is fine. This is with Fedora 16, with grep-2.9-3.fc16.x86_64, sed-4.2.1-9.fc16.x86_64, and bash-4.2.37-1.fc16.x86_64 Will this be fixed anytime soon? -- O.S. ___ https://lists.gnu.org/mailman/listinfo/libtool
Re: bootstrap breakage starting with fec7d87
On 2013-11-19 10:08, Ozkan Sezer wrote: Starting with fec7d87 (funclib.sh: simplify version comparison functions) I am getting the following error from bootstrap: bootstrap: error: 'makeinfo' version == 4.13 is too old bootstrap:'makeinfo' version = 4.8 is required 9fd7b88 is fine. This is with Fedora 16, with grep-2.9-3.fc16.x86_64, sed-4.2.1-9.fc16.x86_64, and bash-4.2.37-1.fc16.x86_64 Will this be fixed anytime soon? Yes, please. I came up with this patch, but I don't know how portable it is, so I would like someone knowledgeable to comment on it before pushing it... Cheers, Peter From a7462c5563e124e06f4f61ce2a9cea26cf8be390 Mon Sep 17 00:00:00 2001 From: Peter Rosin p...@lysator.liu.se Date: Tue, 19 Nov 2013 11:54:27 +0100 Subject: [PATCH] bootstrap: fix version sort Reported by Ozkan Sezer who suffered from makeinfo 4.13 being detected as lesser than the required makeinfo 4.8. * bootstrap (func_sort_ver): Sort numerically on the non-primary keys as well. --- bootstrap |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/bootstrap b/bootstrap index 1b16d95..852efd5 100755 --- a/bootstrap +++ b/bootstrap @@ -1323,7 +1323,7 @@ func_sort_ver () $debug_cmd printf '%s\n%s\n' $1 $2 | -sort -t. -k1n -k1 -k2n -k2 -k3n -k3 -k4n -k4 -k5n -k5 -k6n -k6 -k7n -k7 -k8n -k8 -k9n -k9 +sort -t. -k 1,1n -k 2,2n -k 3,3n -k 4,4n -k 5,5n -k 6,6n -k 7,7n -k 8,8n -k 9,9n } # func_lt_ver PREV CURR -- 1.7.9 ___ https://lists.gnu.org/mailman/listinfo/libtool
Re: bootstrap breakage starting with fec7d87
Hi Ozkan, Petor, On Nov 19, 2013, at 11:57 PM, Peter Rosin p...@lysator.liu.se wrote: On 2013-11-19 10:08, Ozkan Sezer wrote: Starting with fec7d87 (funclib.sh: simplify version comparison functions) I am getting the following error from bootstrap: bootstrap: error: 'makeinfo' version == 4.13 is too old bootstrap:'makeinfo' version = 4.8 is required 9fd7b88 is fine. This is with Fedora 16, with grep-2.9-3.fc16.x86_64, sed-4.2.1-9.fc16.x86_64, and bash-4.2.37-1.fc16.x86_64 Will this be fixed anytime soon? Yes, please. Ouch, sorry about that. I came up with this patch, but I don't know how portable it is, so I would like someone knowledgeable to comment on it before pushing it... I don't have access to any Windows environments, but your patch works correctly for me on various flavours of Mac OS, GNU/Linux, Solaris, HPUX, and AIX -- I no longer have access to Tru64 Unix, SCO Unix or IRIX. Thanks for the quick fix. Assuming it works on cygwin, mingw and other windows variants we support, please go ahead and push. Cheers, -- Gary V. Vaughan (gary AT gnu DOT org) signature.asc Description: Message signed with OpenPGP using GPGMail ___ https://lists.gnu.org/mailman/listinfo/libtool
Re: bootstrap breakage starting with fec7d87
On Nov 20, 2013, at 12:32 PM, Gary V. Vaughan g...@gnu.org wrote: Hi Ozkan, Petor, ^H^H^H^H^H^H^HPeter, *blush* signature.asc Description: Message signed with OpenPGP using GPGMail ___ https://lists.gnu.org/mailman/listinfo/libtool
Re: bootstrap breakage starting with fec7d87
On 2013-11-20 00:32, Gary V. Vaughan wrote: Hi Ozkan, Petor, On Nov 19, 2013, at 11:57 PM, Peter Rosin p...@lysator.liu.se wrote: On 2013-11-19 10:08, Ozkan Sezer wrote: Starting with fec7d87 (funclib.sh: simplify version comparison functions) I am getting the following error from bootstrap: bootstrap: error: 'makeinfo' version == 4.13 is too old bootstrap:'makeinfo' version = 4.8 is required 9fd7b88 is fine. This is with Fedora 16, with grep-2.9-3.fc16.x86_64, sed-4.2.1-9.fc16.x86_64, and bash-4.2.37-1.fc16.x86_64 Will this be fixed anytime soon? Yes, please. Ouch, sorry about that. I came up with this patch, but I don't know how portable it is, so I would like someone knowledgeable to comment on it before pushing it... I don't have access to any Windows environments, but your patch works correctly for me on various flavours of Mac OS, GNU/Linux, Solaris, HPUX, and AIX -- I no longer have access to Tru64 Unix, SCO Unix or IRIX. Thanks for the quick fix. Assuming it works on cygwin, mingw and other windows variants we support, please go ahead and push. Works on Cygwin, so pushed. During patch development I tested various multi-dotted version numbers with various difficulties of the 4.82.620 vs 4.82.2000, 2.82 vs 10.7 and 4.10 vs 4.100 kind, as well as some simple ones of the 5 vs 8 and 4.3 vs 4.8 variety. *checks some more* Hmmm, just checked 1.4.4a vs 1.4-p3a, and the new sort thinks 1.4-p3a is lesser, which is good I suppose. But the function comment is no longer accurate... I don't know what to write instead in that comment, but with the new sort 1.4-p12a 1.4-p3a, which looks about as bad as 1.4.4a 1.4-p3a. Maybe like this: diff --git a/bootstrap b/bootstrap index 852efd5..ca2bb11 100755 --- a/bootstrap +++ b/bootstrap @@ -1312,9 +1312,9 # func_sort_ver VER1 VER2 # --- # 'sort -V' is not generally available. # Note this deviates from the version comparison in automake -# in that it treats 1.5 1.5.0, and treats 1.4.4a 1.4-p3a +# in that it treats 1.5 1.5.0, and treats 1.4-p12a 1.4-p3a # but this should suffice as we won't be specifying old # version formats or redundant trailing .0 in bootstrap.conf. # If we did want full compatibility then we should probably # use m4_version_compare from autoconf. But I'm not sure if m4_version_compare handles p12a vs p3a, so I'll leave this for Gary to clean up. Cheers, Peter ___ https://lists.gnu.org/mailman/listinfo/libtool
bootstrap breakage starting with fec7d87
Starting with fec7d87 (funclib.sh: simplify version comparison functions) I am getting the following error from bootstrap: bootstrap: error: 'makeinfo' version == 4.13 is too old bootstrap:'makeinfo' version = 4.8 is required 9fd7b88 is fine. This is with Fedora 16, with grep-2.9-3.fc16.x86_64, sed-4.2.1-9.fc16.x86_64, and bash-4.2.37-1.fc16.x86_64 -- O.S. ___ https://lists.gnu.org/mailman/listinfo/libtool