From: "Mike Ralphson" <mike.ralphson@gmail.com>
To: "Michal Rokos" <michal.rokos@nextsoft.cz>
Cc: GIT <git@vger.kernel.org>
Subject: Re: [PATCH v3] autoconf: Test FREAD_READS_DIRECTORIES
Date: Tue, 4 Mar 2008 15:32:49 +0000 [thread overview]
Message-ID: <e2b179460803040732k222651c2vdb04e553b8137891@mail.gmail.com> (raw)
In-Reply-To: <200803041514.42783.michal.rokos@nextsoft.cz>
On 04/03/2008, Michal Rokos <michal.rokos@nextsoft.cz> wrote:
> Add test for FREAD_READS_DIRECTORIES to configure script.
>
> When "." could be opened and fread reads more than 0 items,
> FREAD_READS_DIRECTORIES will be set.
On AIX 5.3 this appears to work correctly, i.e. it defines
FREAD_READS_DIRECTORIES
I've negatively tested it just by swapping the return values.
It is susceptible to failing for other reasons than those expected
(i.e. the configure script gets the libsocket test wrong on my
platform if I forget to amend CFLAGS because libg is not available,
and then various tests including this one fail because libsocket is
erroneously linked in and the test programs fail to compile at all),
but that is I think a wider issue with autoconf tests in general.
So, with those caveats:
Tested-by: Mike Ralphson <mike@abacus.co.uk>
Patch to set this option by default on AIX for the non-autoconf
Makefile to follow.
next prev parent reply other threads:[~2008-03-04 15:33 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-03-04 14:14 [PATCH v3] autoconf: Test FREAD_READS_DIRECTORIES Michal Rokos
2008-03-04 14:59 ` Johannes Schindelin
2008-03-04 15:30 ` Michal Rokos
2008-03-04 16:01 ` Johannes Schindelin
2008-03-04 15:32 ` Mike Ralphson [this message]
2008-03-04 16:22 ` Fwd: " Mike Ralphson
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=e2b179460803040732k222651c2vdb04e553b8137891@mail.gmail.com \
--to=mike.ralphson@gmail.com \
--cc=git@vger.kernel.org \
--cc=michal.rokos@nextsoft.cz \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).