diff mbox

btrfs-progs: allow CFLAGS specification in new build

Message ID 54D3D31D.40802@redhat.com (mailing list archive)
State New, archived
Headers show

Commit Message

Eric Sandeen Feb. 5, 2015, 8:31 p.m. UTC
Disclaimer: I am not an auto$FOO expert by any means.

But I could find no way to specify additional CFLAGS, and I think
that this is because they are hard-coded in configure.ac.

The below works for me, but I don't know if it's the right solution...

Signed-off-by: Eric Sandeen <sandeen@redhat.com>
---


--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Comments

Eric Sandeen Feb. 5, 2015, 9:08 p.m. UTC | #1
On 2/5/15 2:31 PM, Eric Sandeen wrote:
> Disclaimer: I am not an auto$FOO expert by any means.
> 
> But I could find no way to specify additional CFLAGS, and I think
> that this is because they are hard-coded in configure.ac.
> 
> The below works for me, but I don't know if it's the right solution...

gah, looking at the wrong branch in git.  Never mind.

-Eric

--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/Makefile.in b/Makefile.in
index 93b6a3c..4a36f6a 100644
--- a/Makefile.in
+++ b/Makefile.in
@@ -12,6 +12,7 @@  DISABLE_BTRFSCONVERT = @DISABLE_BTRFSCONVERT@
 
 # Non-static compilation flags
 CFLAGS = @CFLAGS@ \
+	 -g -O1 \
 	 -include config.h -Wall \
 	 -D_FILE_OFFSET_BITS=64 -DBTRFS_FLAT_INCLUDES \
 	 -D_XOPEN_SOURCE=700  \
diff --git a/configure.ac b/configure.ac
index cbad099..f6ebb7d 100644
--- a/configure.ac
+++ b/configure.ac
@@ -8,8 +8,6 @@  LIBBTRFS_MAJOR=0
 LIBBTRFS_MINOR=1
 LIBBTRFS_PATCHLEVEL=1
 
-CFLAGS="-g -O1"
-
 AC_PREREQ([2.60])
 
 AC_CONFIG_AUX_DIR([config])