From: Theodore Ts'o <tytso@mit.edu>
To: linux-f2fs-devel@lists.sourceforge.net
Cc: Helmut Grohne <helmut@subdivi.de>, Theodore Ts'o <tytso@mit.edu>
Subject: [PATCH] f2fs-tools: Fix cross-build problem caused by AC_CHECK_FILE abuse
Date: Mon, 27 Aug 2018 20:29:41 -0400 [thread overview]
Message-ID: <20180828002941.7507-1-tytso@mit.edu> (raw)
From: Helmut Grohne <helmut@subdivi.de>
f2fs-tools fails to cross build from source, because its upstream
build system wrongly uses AC_CHECK_FILE. The purpose of the macro is
checking for files on the host system. However here it is used to
check whether a .git directory exists, which is a property of the
build place. A simple "test -d" should be used here. After doing so,
f2fs-tools cross builds successfully.
Addresses-Debian-Bug: #896909
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
---
This was a FTCBFS problem reported by a Debian user:
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=896909
configure.ac | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/configure.ac b/configure.ac
index 62c18eb..f3e12d8 100644
--- a/configure.ac
+++ b/configure.ac
@@ -20,14 +20,14 @@ AC_DEFINE([F2FS_MINOR_VERSION], m4_bpatsubst(f2fs_tools_version,
[\([0-9]*\).\([0-9]*\)\(\w\|\W\)*], [\2]),
[Minor version for f2fs-tools])
-AC_CHECK_FILE(.git,
+AS_IF([test -d .git],[
AC_DEFINE([F2FS_TOOLS_DATE],
"m4_bpatsubst(f2fs_tools_gitdate,
[\([0-9-]*\)\(\w\|\W\)*], [\1])",
- [f2fs-tools date based on Git commits]),
+ [f2fs-tools date based on Git commits])],[
AC_DEFINE([F2FS_TOOLS_DATE],
"f2fs_tools_date",
- [f2fs-tools date based on Source releases]))
+ [f2fs-tools date based on Source releases])])
AC_CONFIG_SRCDIR([config.h.in])
AC_CONFIG_HEADER([config.h])
--
2.18.0.rc0
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
next reply other threads:[~2018-08-28 0:30 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-08-28 0:29 Theodore Ts'o [this message]
2018-08-28 3:44 ` [PATCH] f2fs-tools: Fix cross-build problem caused by AC_CHECK_FILE abuse Chao Yu
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=20180828002941.7507-1-tytso@mit.edu \
--to=tytso@mit.edu \
--cc=helmut@subdivi.de \
--cc=linux-f2fs-devel@lists.sourceforge.net \
/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).