On 05/01/2010, Mladen Turk <mt...@apache.org> wrote:
> On 01/05/2010 11:57 AM, rj...@apache.org wrote:
>
> > Author: rjung
> > Get rid of nested version test by formatting
> > version number as one big decimal.
> >
> > +  decimal_apr_version=`printf %02d%02d%03d ${1} ${2} ${3}`
> > +  if test "${decimal_apr_version}" -lt "0101003"; then
> >
>
>  You should not quote the arithmetic ops though.
>  Also 0101003 is exactly what? 1.1.3 ?
For added safety the format could also be used to generate the comparator.

>  And the format should be %d%d%03d
>  APR has 1-1-N digits in formats.
>  There will never be thing like APR 1.11.X :)
>
>
>  Regards
>  --
>  ^TM
>
>
> ---------------------------------------------------------------------
>  To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
>  For additional commands, e-mail: dev-h...@tomcat.apache.org
>
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to