Mailing-List: contact dev-help@apr.apache.org; run by ezmlm
Precedence: bulk
Received-SPF: pass (nike.apache.org: domain of jorton@redhat.com designates
 66.187.233.31 as permitted sender)
Date: Mon, 29 Oct 2007 19:32:34 +0000
From: Joe Orton <jorton@redhat.com>
To: dev@apr.apache.org
Subject: Re: svn commit: r589583 - /apr/apr/trunk/configure.in
Message-ID: <20071029193234.GA5096@redhat.com>
Mail-Followup-To: dev@apr.apache.org
References: <20071029111903.9CD171A9832@eris.apache.org>
 <47262C60.9080007@rowe-clan.net>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
In-Reply-To: <47262C60.9080007@rowe-clan.net>
User-Agent: Mutt/1.5.14 (2007-02-12)
Organization: Red Hat UK Ltd, Amberley Place, 107-111 Peascod Street, Windsor,
 Berkshire, SL4 1TE, United Kingdom.
  Registered in UK and Wales under Company Registration No. 03798903
  Directors: Michael Cunningham (USA), Brendan Lane (Ireland),
 Matt Parson (USA), Charlie Peters (USA)

On Mon, Oct 29, 2007 at 01:54:24PM -0500, William Rowe wrote:
> jorton@apache.org wrote:
> >Author: jorton
> >Date: Mon Oct 29 04:19:02 2007
> >New Revision: 589583
...
> >+# breaking ABI.
> >+ino_t_value=ino_t
> >+if test "$ac_cv_sizeof_long" = "4"; then
> >+    APR_CHECK_TYPES_COMPATIBLE(ino_t, unsigned long, 
> >+                               ino_t_value="unsigned long")
> >+fi
> >+AC_MSG_NOTICE([using $ino_t_value for ino_t])
> 
> This doesn't look right for 32 platforms built without 64 bit off's, and
> consider they might initially build on one which doesn't support the newer
> types, but then might compile add ins on an updated kernel, eh?

I can't parse that.

> Shouldn't we continue to lock this /down/ to unsigned int where we know this
> is true?  In fact, I'd add unsigned short to that list for safety.

No, unless you know of a platform where ino_t is defined to be anything 
other than a 32-bit unsigned long *and* varies by _FILE_OFFSET_BITS?  
Otherwise configure is just testing for hypothetical platforms, which is 
a waste of cycles.

joe