[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: mdemo ltdl failure

From: Ralf Wildenhues
Subject: Re: mdemo ltdl failure
Date: Thu, 19 Apr 2007 20:24:37 +0200
User-agent: Mutt/1.5.13 (2006-08-11)

* Charles Wilson wrote on Thu, Apr 19, 2007 at 08:08:41PM CEST:
> Ralf Wildenhues wrote:
> >
> >It's a good choice of testing, and should be done again with the final
> >patch.  Plus one test on Solaris with its /bin/sh.  (Just noting this,
> >I can probably do these tests then.)
> will do for linux & mingw, but have no unencumbered access to solaris at 
> present.

I'll do that then.

> >>+            AS_IF([test "$os_major" -gt 
> >>"1"],[lt_cv_sys_argz_works=yes],dnl
> >>+                 [test "$os_minor" -gt 
> >>"5"],[lt_cv_sys_argz_works=yes],dnl
> >>+                 [test "$os_micro" -gt 
> >>"24"],[lt_cv_sys_argz_works=yes],dnl
> >
> >No need for double quoting the literal integers.  And if it can happen
> >that $os_micro is the empty string (which I'll consider quite likely),
> >then double quoting isn't enough precaution against error output, as 
> >"" isn't interpreted by test as a number.
> Okay, but actually, all cygwin releases are guaranteed to have non-empty 
> major, minor, and micro numbers -- and we're inside a $host_os == cygwin 
> block, here.

Oh, I overlooked this fact in my review.  Ignore that comment then.


reply via email to

[Prev in Thread] Current Thread [Next in Thread]