From: David Bremner <david@tethera.net>
To: David Bremner <david@tethera.net>, notmuch@notmuchmail.org
Subject: [PATCH 1/2] configure: detect 64 bit time_t
Date: Wed, 24 Jun 2020 11:32:33 -0300 [thread overview]
Message-ID: <20200624143234.540526-1-david@tethera.net> (raw)
In-Reply-To: <87lfkcocvi.fsf@tethera.net>
Certain tests involving timestamps > 32bits cannot pass with the
current libnotmuch API. We will avoid this isssue for now by disabling
those tests on "old" architectures with 32bit time_t.
---
configure | 33 ++++++++++++++++++++++++++++++++-
1 file changed, 32 insertions(+), 1 deletion(-)
diff --git a/configure b/configure
index 05ade05b..ecb0bdb7 100755
--- a/configure
+++ b/configure
@@ -1045,6 +1045,34 @@ else
fi
rm -f compat/have_timegm
+cat <<EOF > _time_t.c
+#include <time.h>
+#include <stdio.h>
+int main(int argc, char**argv) {
+ printf("%d\n",sizeof(time_t) >= 8);
+}
+EOF
+
+printf "Checking for 64 bit time_t... "
+if ${CC} _time_t.c -o _time_t > /dev/null 2>&1;
+then
+ have_64bit_time_t=$(./_time_t)
+ if [ $have_64bit_time_t -eq 1 ]
+ then
+ printf "Yes.\n"
+ else
+ printf "No.\n"
+ fi
+else
+ cat <<EOF
+
+*** Error: compilation failed. Please try running configure again in
+a clean environment, and if the problem persists, report a bug.
+EOF
+ rm -f _time_t _time_t.c exit 1
+fi
+
+
printf "Checking for dirent.d_type... "
if ${CC} -o compat/have_d_type "$srcdir"/compat/have_d_type.c > /dev/null 2>&1
then
@@ -1128,7 +1156,7 @@ for flag in -Wmissing-declarations; do
done
printf "\n\t%s\n" "${WARN_CFLAGS}"
-rm -f minimal minimal.c _libversion.c _libversion _libversion.sh _check_session_keys.c _check_session_keys _check_x509_validity.c _check_x509_validity
+rm -f minimal minimal.c _time_t _time_t.c _libversion.c _libversion _libversion.sh _check_session_keys.c _check_session_keys _check_x509_validity.c _check_x509_validity
# construct the Makefile.config
cat > Makefile.config <<EOF
@@ -1429,6 +1457,9 @@ NOTMUCH_HAVE_MAN=$((have_sphinx))
NOTMUCH_HAVE_BASH=${have_bash}
NOTMUCH_BASH_ABSOLUTE=${bash_absolute}
+# Whether time_t is 64 bits (or more)
+NOTMUCH_HAVE_64BIT_TIME_T=${have_64bit_time_t}
+
# Whether perl exists, and if so where
NOTMUCH_HAVE_PERL=${have_perl}
NOTMUCH_PERL_ABSOLUTE=${perl_absolute}
--
2.27.0
next prev parent reply other threads:[~2020-06-24 14:32 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-06-22 14:18 test failures on 32 bit architectures David Bremner
2020-06-22 23:12 ` David Bremner
2020-06-24 0:39 ` David Bremner
2020-06-24 11:04 ` [PATCH] lib: avoid casting gint64 to time_t David Bremner
2020-06-24 12:40 ` David Bremner
2020-06-24 14:32 ` David Bremner [this message]
2020-06-24 14:32 ` [PATCH 2/2] test: mark two tests broken on machines with 32 bit time_t David Bremner
2020-06-27 1:25 ` David Bremner
2020-06-24 19:38 ` [PATCH 1/2] configure: detect 64 " Tomi Ollila
2020-06-27 1:25 ` David Bremner
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
List information: https://notmuchmail.org/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20200624143234.540526-1-david@tethera.net \
--to=david@tethera.net \
--cc=notmuch@notmuchmail.org \
/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.
Code repositories for project(s) associated with this public inbox
https://yhetil.org/notmuch.git/
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 read-only IMAP folder(s) and NNTP newsgroup(s).