./configure compares non-numbers with -gt

[Paul Jarc]

I get this output from ./configure for 2.05a:
checking version of installed readline library... ./configure: test: 2a: 
integer expression expected
4.2a

(I had to run the configure script with bash to get even this; pdksh
was giving me "Memory fault" and dying from SIGSEGV, although I
haven't been able to reproduce it outside bash's configure script.)

The relevant part of the configure script (line 3238, apparently
generated by line 389 of configure.in):
elif test $RL_MAJOR -gt 4 || { test $RL_MAJOR = 4 && test $RL_MINOR -gt 2 ; } ; 
then

In my case, $RL_MINOR is 2a.  Maybe this test should be changed to
something like this:
case $RL_MINOR in
  *[!0-9]*)
    case $RL_MINOR in
      [2-9]*)   true;;
      1[!0-9]*) false;;
      *)        true;;
    esac
  *) test $RL_MINOR -gt 2;;
esac

Or maybe bash and readline should use purely numeric versions, since
they'd be easier to compare.


paul