* [PATCH XFSPROGS 1/3] configure.in: Declare a requirement for at least autoconf 2.50
@ 2011-07-21 23:50 ` Theodore Ts'o
0 siblings, 0 replies; 30+ messages in thread
From: Theodore Ts'o @ 2011-07-21 23:50 UTC (permalink / raw)
To: Ext4 Developers List, xfs; +Cc: Theodore Ts'o
On Debian/Ubuntu systems, if autoconf version 2.13 is installed,
autoconf will try to automatically figure out whether autoconf 2.13 or
something more modern is required (since the autoconf maintainers,
curses be upon them, didn't bother to maintain compatibility between
autoconf 2.13 and 2.50). Unfortunately, the hueristics aren't
perfect, and although the configure.in file looks superficially like
it will be compatible with autoconf 2.13, it isn't. You will end up
with a number of very subtle compilation failures if you use autoconf
2.13.
So declare a requirement for autoconf 2.50 using AC_PREREQ(2.50).
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
---
configure.in | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/configure.in b/configure.in
index 10c4c61..664c0e9 100644
--- a/configure.in
+++ b/configure.in
@@ -1,4 +1,5 @@
AC_INIT(include/libxfs.h)
+AC_PREREQ(2.50)
AC_CONFIG_AUX_DIR([.])
AC_CONFIG_MACRO_DIR([m4])
AC_CONFIG_HEADER(include/platform_defs.h)
--
1.7.4.1.22.gec8e1.dirty
^ permalink raw reply related [flat|nested] 30+ messages in thread
* [PATCH XFSPROGS 1/3] configure.in: Declare a requirement for at least autoconf 2.50
@ 2011-07-21 23:50 ` Theodore Ts'o
0 siblings, 0 replies; 30+ messages in thread
From: Theodore Ts'o @ 2011-07-21 23:50 UTC (permalink / raw)
To: Ext4 Developers List, xfs; +Cc: Theodore Ts'o
On Debian/Ubuntu systems, if autoconf version 2.13 is installed,
autoconf will try to automatically figure out whether autoconf 2.13 or
something more modern is required (since the autoconf maintainers,
curses be upon them, didn't bother to maintain compatibility between
autoconf 2.13 and 2.50). Unfortunately, the hueristics aren't
perfect, and although the configure.in file looks superficially like
it will be compatible with autoconf 2.13, it isn't. You will end up
with a number of very subtle compilation failures if you use autoconf
2.13.
So declare a requirement for autoconf 2.50 using AC_PREREQ(2.50).
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
---
configure.in | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/configure.in b/configure.in
index 10c4c61..664c0e9 100644
--- a/configure.in
+++ b/configure.in
@@ -1,4 +1,5 @@
AC_INIT(include/libxfs.h)
+AC_PREREQ(2.50)
AC_CONFIG_AUX_DIR([.])
AC_CONFIG_MACRO_DIR([m4])
AC_CONFIG_HEADER(include/platform_defs.h)
--
1.7.4.1.22.gec8e1.dirty
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply related [flat|nested] 30+ messages in thread
* [PATCH XFSPROGS 2/3] Build without enabling DEBUG by default
2011-07-21 23:50 ` Theodore Ts'o
@ 2011-07-21 23:50 ` Theodore Ts'o
-1 siblings, 0 replies; 30+ messages in thread
From: Theodore Ts'o @ 2011-07-21 23:50 UTC (permalink / raw)
To: Ext4 Developers List, xfs; +Cc: Theodore Ts'o
The tree currently will blow out with a build failure if -DDEBUG is
specified.
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
---
m4/package_globals.m4 | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/m4/package_globals.m4 b/m4/package_globals.m4
index e469671..d2a3e2f 100644
--- a/m4/package_globals.m4
+++ b/m4/package_globals.m4
@@ -17,7 +17,7 @@ AC_DEFUN([AC_PACKAGE_GLOBALS],
test -z "$BUILD_VERSION" || pkg_release="$BUILD_VERSION"
AC_SUBST(pkg_release)
- DEBUG=${DEBUG:-'-DDEBUG'} dnl -DNDEBUG
+ DEBUG=${DEBUG:-'-DNDEBUG'} dnl -DNDEBUG
debug_build="$DEBUG"
AC_SUBST(debug_build)
--
1.7.4.1.22.gec8e1.dirty
^ permalink raw reply related [flat|nested] 30+ messages in thread* [PATCH XFSPROGS 2/3] Build without enabling DEBUG by default
@ 2011-07-21 23:50 ` Theodore Ts'o
0 siblings, 0 replies; 30+ messages in thread
From: Theodore Ts'o @ 2011-07-21 23:50 UTC (permalink / raw)
To: Ext4 Developers List, xfs; +Cc: Theodore Ts'o
The tree currently will blow out with a build failure if -DDEBUG is
specified.
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
---
m4/package_globals.m4 | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/m4/package_globals.m4 b/m4/package_globals.m4
index e469671..d2a3e2f 100644
--- a/m4/package_globals.m4
+++ b/m4/package_globals.m4
@@ -17,7 +17,7 @@ AC_DEFUN([AC_PACKAGE_GLOBALS],
test -z "$BUILD_VERSION" || pkg_release="$BUILD_VERSION"
AC_SUBST(pkg_release)
- DEBUG=${DEBUG:-'-DDEBUG'} dnl -DNDEBUG
+ DEBUG=${DEBUG:-'-DNDEBUG'} dnl -DNDEBUG
debug_build="$DEBUG"
AC_SUBST(debug_build)
--
1.7.4.1.22.gec8e1.dirty
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply related [flat|nested] 30+ messages in thread* Re: [PATCH XFSPROGS 2/3] Build without enabling DEBUG by default
2011-07-21 23:50 ` Theodore Ts'o
@ 2011-07-22 14:17 ` Christoph Hellwig
-1 siblings, 0 replies; 30+ messages in thread
From: Christoph Hellwig @ 2011-07-22 14:17 UTC (permalink / raw)
To: Theodore Ts'o; +Cc: Ext4 Developers List, xfs
On Thu, Jul 21, 2011 at 07:50:36PM -0400, Theodore Ts'o wrote:
> The tree currently will blow out with a build failure if -DDEBUG is
> specified.
This still doesn't make sense to me. libxfs explicily overrides
DEBUG in the makefile:
# don't try linking xfs_repair with a debug libxfs.
DEBUG = -DNDEBUG
we really need to figure out why your enviroment doesn't pick this
up, especially as we've never seen that report from anybody else.
^ permalink raw reply [flat|nested] 30+ messages in thread
* Re: [PATCH XFSPROGS 2/3] Build without enabling DEBUG by default
@ 2011-07-22 14:17 ` Christoph Hellwig
0 siblings, 0 replies; 30+ messages in thread
From: Christoph Hellwig @ 2011-07-22 14:17 UTC (permalink / raw)
To: Theodore Ts'o; +Cc: Ext4 Developers List, xfs
On Thu, Jul 21, 2011 at 07:50:36PM -0400, Theodore Ts'o wrote:
> The tree currently will blow out with a build failure if -DDEBUG is
> specified.
This still doesn't make sense to me. libxfs explicily overrides
DEBUG in the makefile:
# don't try linking xfs_repair with a debug libxfs.
DEBUG = -DNDEBUG
we really need to figure out why your enviroment doesn't pick this
up, especially as we've never seen that report from anybody else.
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply [flat|nested] 30+ messages in thread
* Re: [PATCH XFSPROGS 2/3] Build without enabling DEBUG by default
2011-07-22 14:17 ` Christoph Hellwig
@ 2011-07-22 18:30 ` Ted Ts'o
-1 siblings, 0 replies; 30+ messages in thread
From: Ted Ts'o @ 2011-07-22 18:30 UTC (permalink / raw)
To: Christoph Hellwig; +Cc: Ext4 Developers List, xfs
On Fri, Jul 22, 2011 at 10:17:53AM -0400, Christoph Hellwig wrote:
> On Thu, Jul 21, 2011 at 07:50:36PM -0400, Theodore Ts'o wrote:
> > The tree currently will blow out with a build failure if -DDEBUG is
> > specified.
>
> This still doesn't make sense to me. libxfs explicily overrides
> DEBUG in the makefile:
>
> # don't try linking xfs_repair with a debug libxfs.
> DEBUG = -DNDEBUG
I added some debugging statements in libxfs/Makefile, and found this
(see below).
It looks like the problem is that GNU make 3.81 is doing an immediate
expansion of this line in builddefs:
CFLAGS += $(FCFLAGS) $(GCFLAGS) $(PCFLAGS) $(LCFLAGS)
- Ted
LTCOMPILE is /bin/sh ../libtool --quiet --tag=CC --mode=compile gcc -I/kbuild/xfstests/bld/include -g -O2 -DDEBUG -DVERSION="3.1.5" -DLOCALEDIR="//share/locale" -DPACKAGE="xfsprogs" -I./include -DENABLE_GETTEXT -D_GNU_SOURCE -D_XOPEN_SOURCE=500 -D_FILE_OFFSET_BITS=64 -funsigned-char -fno-strict-aliasing -Wall -g -O2 -DDEBUG -DVERSION="3.1.5" -DLOCALEDIR="//share/locale" -DPACKAGE="xfsprogs" -I./include -DENABLE_GETTEXT -D_GNU_SOURCE -D_XOPEN_SOURCE=500 -D_FILE_OFFSET_BITS=64 -funsigned-char -fno-strict-aliasing -Wall -I. -g -O2 -DNDEBUG -DVERSION="3.1.5" -DLOCALEDIR="//share/locale" -DPACKAGE="xfsprogs" -I../include -DENABLE_GETTEXT -D_GNU_SOURCE -D_XOPEN_SOURCE=500 -D_FILE_OFFSET_BITS=64 -funsigned-char -fno-strict-aliasing -Wall
FCFLAGS is -I.
GCFLAGS is -g -O2 -DNDEBUG -DVERSION="3.1.5" -DLOCALEDIR="//share/locale" -DPACKAGE="xfsprogs" -I../include -DENABLE_GETTEXT
PCFLAGS is -D_GNU_SOURCE -D_XOPEN_SOURCE=500 -D_FILE_OFFSET_BITS=64 -funsigned-char -fno-strict-aliasing -Wall
CFLAGS is -I/kbuild/xfstests/bld/include -g -O2 -DDEBUG -DVERSION="3.1.5" -DLOCALEDIR="//share/locale" -DPACKAGE="xfsprogs" -I./include -DENABLE_GETTEXT -D_GNU_SOURCE -D_XOPEN_SOURCE=500 -D_FILE_OFFSET_BITS=64 -funsigned-char -fno-strict-aliasing -Wall -g -O2 -DDEBUG -DVERSION="3.1.5" -DLOCALEDIR="//share/locale" -DPACKAGE="xfsprogs" -I./include -DENABLE_GETTEXT -D_GNU_SOURCE -D_XOPEN_SOURCE=500 -D_FILE_OFFSET_BITS=64 -funsigned-char -fno-strict-aliasing -Wall -I. -g -O2 -DNDEBUG -DVERSION="3.1.5" -DLOCALEDIR="//share/locale" -DPACKAGE="xfsprogs" -I../include -DENABLE_GETTEXT -D_GNU_SOURCE -D_XOPEN_SOURCE=500 -D_FILE_OFFSET_BITS=64 -funsigned-char -fno-strict-aliasing -Wall
CCF is gcc -I/kbuild/xfstests/bld/include -g -O2 -DDEBUG -DVERSION="3.1.5" -DLOCALEDIR="//share/locale" -DPACKAGE="xfsprogs" -I./include -DENABLE_GETTEXT -D_GNU_SOURCE -D_XOPEN_SOURCE=500 -D_FILE_OFFSET_BITS=64 -funsigned-char -fno-strict-aliasing -Wall -g -O2 -DDEBUG -DVERSION="3.1.5" -DLOCALEDIR="//share/locale" -DPACKAGE="xfsprogs" -I./include -DENABLE_GETTEXT -D_GNU_SOURCE -D_XOPEN_SOURCE=500 -D_FILE_OFFSET_BITS=64 -funsigned-char -fno-strict-aliasing -Wall -I. -g -O2 -DNDEBUG -DVERSION="3.1.5" -DLOCALEDIR="//share/locale" -DPACKAGE="xfsprogs" -I../include -DENABLE_GETTEXT -D_GNU_SOURCE -D_XOPEN_SOURCE=500 -D_FILE_OFFSET_BITS=64 -funsigned-char -fno-strict-aliasing -Wall
^ permalink raw reply [flat|nested] 30+ messages in thread
* Re: [PATCH XFSPROGS 2/3] Build without enabling DEBUG by default
@ 2011-07-22 18:30 ` Ted Ts'o
0 siblings, 0 replies; 30+ messages in thread
From: Ted Ts'o @ 2011-07-22 18:30 UTC (permalink / raw)
To: Christoph Hellwig; +Cc: Ext4 Developers List, xfs
On Fri, Jul 22, 2011 at 10:17:53AM -0400, Christoph Hellwig wrote:
> On Thu, Jul 21, 2011 at 07:50:36PM -0400, Theodore Ts'o wrote:
> > The tree currently will blow out with a build failure if -DDEBUG is
> > specified.
>
> This still doesn't make sense to me. libxfs explicily overrides
> DEBUG in the makefile:
>
> # don't try linking xfs_repair with a debug libxfs.
> DEBUG = -DNDEBUG
I added some debugging statements in libxfs/Makefile, and found this
(see below).
It looks like the problem is that GNU make 3.81 is doing an immediate
expansion of this line in builddefs:
CFLAGS += $(FCFLAGS) $(GCFLAGS) $(PCFLAGS) $(LCFLAGS)
- Ted
LTCOMPILE is /bin/sh ../libtool --quiet --tag=CC --mode=compile gcc -I/kbuild/xfstests/bld/include -g -O2 -DDEBUG -DVERSION="3.1.5" -DLOCALEDIR="//share/locale" -DPACKAGE="xfsprogs" -I./include -DENABLE_GETTEXT -D_GNU_SOURCE -D_XOPEN_SOURCE=500 -D_FILE_OFFSET_BITS=64 -funsigned-char -fno-strict-aliasing -Wall -g -O2 -DDEBUG -DVERSION="3.1.5" -DLOCALEDIR="//share/locale" -DPACKAGE="xfsprogs" -I./include -DENABLE_GETTEXT -D_GNU_SOURCE -D_XOPEN_SOURCE=500 -D_FILE_OFFSET_BITS=64 -funsigned-char -fno-strict-aliasing -Wall -I. -g -O2 -DNDEBUG -DVERSION="3.1.5" -DLOCALEDIR="//share/locale" -DPACKAGE="xfsprogs" -I../include -DENABLE_GETTEXT -D_GNU_SOURCE -D_XOPEN_SOURCE=500 -D_FILE_OFFSET_BITS=64 -funsigned-char -fno-strict-aliasing -Wall
FCFLAGS is -I.
GCFLAGS is -g -O2 -DNDEBUG -DVERSION="3.1.5" -DLOCALEDIR="//share/locale" -DPACKAGE="xfsprogs" -I../include -DENABLE_GETTEXT
PCFLAGS is -D_GNU_SOURCE -D_XOPEN_SOURCE=500 -D_FILE_OFFSET_BITS=64 -funsigned-char -fno-strict-aliasing -Wall
CFLAGS is -I/kbuild/xfstests/bld/include -g -O2 -DDEBUG -DVERSION="3.1.5" -DLOCALEDIR="//share/locale" -DPACKAGE="xfsprogs" -I./include -DENABLE_GETTEXT -D_GNU_SOURCE -D_XOPEN_SOURCE=500 -D_FILE_OFFSET_BITS=64 -funsigned-char -fno-strict-aliasing -Wall -g -O2 -DDEBUG -DVERSION="3.1.5" -DLOCALEDIR="//share/locale" -DPACKAGE="xfsprogs" -I./include -DENABLE_GETTEXT -D_GNU_SOURCE -D_XOPEN_SOURCE=500 -D_FILE_OFFSET_BITS=64 -funsigned-char -fno-strict-aliasing -Wall -I. -g -O2 -DNDEBUG -DVERSION="3.1.5" -DLOCALEDIR="//share/locale" -DPACKAGE="xfsprogs" -I../include -DENABLE_GETTEXT -D_GNU_SOURCE -D_XOPEN_SOURCE=500 -D_FILE_OFFSET_BITS=64 -funsigned-char -fno-strict-aliasing -Wall
CCF is gcc -I/kbuild/xfstests/bld/include -g -O2 -DDEBUG -DVERSION="3.1.5" -DLOCALEDIR="//share/locale" -DPACKAGE="xfsprogs" -I./include -DENABLE_GETTEXT -D_GNU_SOURCE -D_XOPEN_SOURCE=500 -D_FILE_OFFSET_BITS=64 -funsigned-char -fno-strict-aliasing -Wall -g -O2 -DDEBUG -DVERSION="3.1.5" -DLOCALEDIR="//share/locale" -DPACKAGE="xfsprogs" -I./include -DENABLE_GETTEXT -D_GNU_SOURCE -D_XOPEN_SOURCE=500 -D_FILE_OFFSET_BITS=64 -funsigned-char -fno-strict-aliasing -Wall -I. -g -O2 -DNDEBUG -DVERSION="3.1.5" -DLOCALEDIR="//share/locale" -DPACKAGE="xfsprogs" -I../include -DENABLE_GETTEXT -D_GNU_SOURCE -D_XOPEN_SOURCE=500 -D_FILE_OFFSET_BITS=64 -funsigned-char -fno-strict-aliasing -Wall
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply [flat|nested] 30+ messages in thread
* Re: [PATCH XFSPROGS 2/3] Build without enabling DEBUG by default
2011-07-22 18:30 ` Ted Ts'o
@ 2011-07-22 18:38 ` Christoph Hellwig
-1 siblings, 0 replies; 30+ messages in thread
From: Christoph Hellwig @ 2011-07-22 18:38 UTC (permalink / raw)
To: Ted Ts'o; +Cc: Christoph Hellwig, Ext4 Developers List, xfs
On Fri, Jul 22, 2011 at 02:30:48PM -0400, Ted Ts'o wrote:
> I added some debugging statements in libxfs/Makefile, and found this
> (see below).
You can get most of this by simply using make V=1, not just in xfsprogs,
but in most projects using silent make output (including those using
automake)
> It looks like the problem is that GNU make 3.81 is doing an immediate
> expansion of this line in builddefs:
I'm having GNU make 3.81 on most of my test systems as well, so that
alone can't be the whole issue.
> CFLAGS += $(FCFLAGS) $(GCFLAGS) $(PCFLAGS) $(LCFLAGS)
>
> LTCOMPILE is /bin/sh ../libtool --quiet --tag=CC --mode=compile gcc -I/kbuild/xfstests/bld/include -g -O2 -DDEBUG -DVERSION="3.1.5" -DLOCALEDIR="//share/locale" -DPACKAGE="xfsprogs" -I./include -DENABLE_GETTEXT -D_GNU_SOURCE -D_XOPEN_SOURCE=500 -D_FILE_OFFSET_BITS=64 -funsigned-char -fno-strict-aliasing -Wall -g -O2 -DDEBUG -DVERSION="3.1.5" -DLOCALEDIR="//share/locale" -DPACKAGE="xfsprogs" -I./include -DENABLE_GETTEXT -D_GNU_SOURCE -D_XOPEN_SOURCE=500 -D_FILE_OFFSET_BITS=64 -funsigned-char -fno-strict-aliasing -Wall -I. -g -O2 -DNDEBUG -DVERSION="3.1.5" -DLOCALEDIR="//share/locale" -DPACKAGE="xfsprogs" -I../include -DENABLE_GETTEXT -D_GNU_SOURCE -D_XOPEN_SOURCE=500 -D_FILE_OFFSET_BITS=64 -funsigned-char -fno-strict-aliasing -Wall
A typical V=1 compile line looks like:
/bin/bash ../libtool --quiet --tag=CC --mode=compile gcc -I. -g -O2 -DNDEBUG -DVERSION=\"3.1.5\" -DLOCALEDIR=\"/usr/share/locale\" -DPACKAGE=\"xfsprogs\" -I../include -DENABLE_GETTEXT -D_GNU_SOURCE -D_XOPEN_SOURCE=500 -D_FILE_OFFSET_BITS=64 -funsigned-char -fno-strict-aliasing -Wall -c xfs_dir2_sf.c
for me. The biggest thing is that you have another directory in your
include path. How did you add that one? Did you add anything else
special to the ./configure or make command lines?
^ permalink raw reply [flat|nested] 30+ messages in thread
* Re: [PATCH XFSPROGS 2/3] Build without enabling DEBUG by default
@ 2011-07-22 18:38 ` Christoph Hellwig
0 siblings, 0 replies; 30+ messages in thread
From: Christoph Hellwig @ 2011-07-22 18:38 UTC (permalink / raw)
To: Ted Ts'o; +Cc: Christoph Hellwig, Ext4 Developers List, xfs
On Fri, Jul 22, 2011 at 02:30:48PM -0400, Ted Ts'o wrote:
> I added some debugging statements in libxfs/Makefile, and found this
> (see below).
You can get most of this by simply using make V=1, not just in xfsprogs,
but in most projects using silent make output (including those using
automake)
> It looks like the problem is that GNU make 3.81 is doing an immediate
> expansion of this line in builddefs:
I'm having GNU make 3.81 on most of my test systems as well, so that
alone can't be the whole issue.
> CFLAGS += $(FCFLAGS) $(GCFLAGS) $(PCFLAGS) $(LCFLAGS)
>
> LTCOMPILE is /bin/sh ../libtool --quiet --tag=CC --mode=compile gcc -I/kbuild/xfstests/bld/include -g -O2 -DDEBUG -DVERSION="3.1.5" -DLOCALEDIR="//share/locale" -DPACKAGE="xfsprogs" -I./include -DENABLE_GETTEXT -D_GNU_SOURCE -D_XOPEN_SOURCE=500 -D_FILE_OFFSET_BITS=64 -funsigned-char -fno-strict-aliasing -Wall -g -O2 -DDEBUG -DVERSION="3.1.5" -DLOCALEDIR="//share/locale" -DPACKAGE="xfsprogs" -I./include -DENABLE_GETTEXT -D_GNU_SOURCE -D_XOPEN_SOURCE=500 -D_FILE_OFFSET_BITS=64 -funsigned-char -fno-strict-aliasing -Wall -I. -g -O2 -DNDEBUG -DVERSION="3.1.5" -DLOCALEDIR="//share/locale" -DPACKAGE="xfsprogs" -I../include -DENABLE_GETTEXT -D_GNU_SOURCE -D_XOPEN_SOURCE=500 -D_FILE_OFFSET_BITS=64 -funsigned-char -fno-strict-aliasing -Wall
A typical V=1 compile line looks like:
/bin/bash ../libtool --quiet --tag=CC --mode=compile gcc -I. -g -O2 -DNDEBUG -DVERSION=\"3.1.5\" -DLOCALEDIR=\"/usr/share/locale\" -DPACKAGE=\"xfsprogs\" -I../include -DENABLE_GETTEXT -D_GNU_SOURCE -D_XOPEN_SOURCE=500 -D_FILE_OFFSET_BITS=64 -funsigned-char -fno-strict-aliasing -Wall -c xfs_dir2_sf.c
for me. The biggest thing is that you have another directory in your
include path. How did you add that one? Did you add anything else
special to the ./configure or make command lines?
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply [flat|nested] 30+ messages in thread
* Re: [PATCH XFSPROGS 2/3] Build without enabling DEBUG by default
2011-07-22 18:38 ` Christoph Hellwig
@ 2011-07-22 22:30 ` Alex Elder
-1 siblings, 0 replies; 30+ messages in thread
From: Alex Elder @ 2011-07-22 22:30 UTC (permalink / raw)
To: Christoph Hellwig; +Cc: Ted Ts'o, Ext4 Developers List, xfs
On Fri, 2011-07-22 at 14:38 -0400, Christoph Hellwig wrote:
> On Fri, Jul 22, 2011 at 02:30:48PM -0400, Ted Ts'o wrote:
. . .
> A typical V=1 compile line looks like:
>
> /bin/bash ../libtool --quiet --tag=CC --mode=compile gcc -I. -g -O2
> -DNDEBUG -DVERSION=\"3.1.5\" -DLOCALEDIR=\"/usr/share/locale\"
> -DPACKAGE=\"xfsprogs\" -I../include -DENABLE_GETTEXT -D_GNU_SOURCE
> -D_XOPEN_SOURCE=500 -D_FILE_OFFSET_BITS=64 -funsigned-char
> -fno-strict-aliasing -Wall -c xfs_dir2_sf.c
FWIW, I get a compile line for xfs_dir2_sf.c identical to
what Christoph supplied.
> for me. The biggest thing is that you have another directory in your
> include path. How did you add that one? Did you add anything else
> special to the ./configure or make command lines?
^ permalink raw reply [flat|nested] 30+ messages in thread
* Re: [PATCH XFSPROGS 2/3] Build without enabling DEBUG by default
@ 2011-07-22 22:30 ` Alex Elder
0 siblings, 0 replies; 30+ messages in thread
From: Alex Elder @ 2011-07-22 22:30 UTC (permalink / raw)
To: Christoph Hellwig; +Cc: Ext4 Developers List, Ted Ts'o, xfs
On Fri, 2011-07-22 at 14:38 -0400, Christoph Hellwig wrote:
> On Fri, Jul 22, 2011 at 02:30:48PM -0400, Ted Ts'o wrote:
. . .
> A typical V=1 compile line looks like:
>
> /bin/bash ../libtool --quiet --tag=CC --mode=compile gcc -I. -g -O2
> -DNDEBUG -DVERSION=\"3.1.5\" -DLOCALEDIR=\"/usr/share/locale\"
> -DPACKAGE=\"xfsprogs\" -I../include -DENABLE_GETTEXT -D_GNU_SOURCE
> -D_XOPEN_SOURCE=500 -D_FILE_OFFSET_BITS=64 -funsigned-char
> -fno-strict-aliasing -Wall -c xfs_dir2_sf.c
FWIW, I get a compile line for xfs_dir2_sf.c identical to
what Christoph supplied.
> for me. The biggest thing is that you have another directory in your
> include path. How did you add that one? Did you add anything else
> special to the ./configure or make command lines?
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply [flat|nested] 30+ messages in thread
* Re: [PATCH XFSPROGS 2/3] Build without enabling DEBUG by default
2011-07-22 18:38 ` Christoph Hellwig
@ 2011-07-22 22:34 ` Ted Ts'o
-1 siblings, 0 replies; 30+ messages in thread
From: Ted Ts'o @ 2011-07-22 22:34 UTC (permalink / raw)
To: Christoph Hellwig; +Cc: Ext4 Developers List, xfs
On Fri, Jul 22, 2011 at 02:38:30PM -0400, Christoph Hellwig wrote:
>
> You can get most of this by simply using make V=1, not just in xfsprogs,
> but in most projects using silent make output (including those using
> automake)
Yeah, I know about V=1 (and I have been using it), but I wanted to
know which of the many Makefile variables which finally make up
$(CFLAGS) was getting set to which value, and V=1 doesn't do this.
> for me. The biggest thing is that you have another directory in your
> include path. How did you add that one? Did you add anything else
> special to the ./configure or make command lines?
Yes I needed it because I wanted to do a hermetic build; that is, I
didn't want to depend on the version of the acl, attr, dmapi, etc.,
shipped by the distribution. (This is actually more important when
building xfstests, since I wanted to make sure we used the version of
the xfsprogs headers shipped by the very tip of xfsprogs-dev, since in
the past I've had problems because the distro-shipped header files
were too old.)
The way I did that was by setting the environment CFLAGS to
-I/kbuild/xfstests/bld/include. And that seems to be the problem. If
CFLAGS is set as an environment variable, then it's set as a :=, which
means that CFLAGS is set right away, the first time buildmacros is
included.
How about this instead? A patch like this so that CFLAGS can be
remembered at configure time:
diff --git a/include/builddefs.in b/include/builddefs.in
index 4c8ec64..81ebfcd 100644
--- a/include/builddefs.in
+++ b/include/builddefs.in
@@ -25,6 +25,7 @@ OPTIMIZER = @opt_build@
MALLOCLIB = @malloc_lib@
LOADERFLAGS = @LDFLAGS@
LTLDFLAGS = @LDFLAGS@
+CFLAGS = @CFLAGS@
LIBRT = @librt@
LIBUUID = @libuuid@
Then I don't have to set an environment variable when running make; I
just have to pass it to configure.
- Ted
^ permalink raw reply related [flat|nested] 30+ messages in thread
* Re: [PATCH XFSPROGS 2/3] Build without enabling DEBUG by default
@ 2011-07-22 22:34 ` Ted Ts'o
0 siblings, 0 replies; 30+ messages in thread
From: Ted Ts'o @ 2011-07-22 22:34 UTC (permalink / raw)
To: Christoph Hellwig; +Cc: Ext4 Developers List, xfs
On Fri, Jul 22, 2011 at 02:38:30PM -0400, Christoph Hellwig wrote:
>
> You can get most of this by simply using make V=1, not just in xfsprogs,
> but in most projects using silent make output (including those using
> automake)
Yeah, I know about V=1 (and I have been using it), but I wanted to
know which of the many Makefile variables which finally make up
$(CFLAGS) was getting set to which value, and V=1 doesn't do this.
> for me. The biggest thing is that you have another directory in your
> include path. How did you add that one? Did you add anything else
> special to the ./configure or make command lines?
Yes I needed it because I wanted to do a hermetic build; that is, I
didn't want to depend on the version of the acl, attr, dmapi, etc.,
shipped by the distribution. (This is actually more important when
building xfstests, since I wanted to make sure we used the version of
the xfsprogs headers shipped by the very tip of xfsprogs-dev, since in
the past I've had problems because the distro-shipped header files
were too old.)
The way I did that was by setting the environment CFLAGS to
-I/kbuild/xfstests/bld/include. And that seems to be the problem. If
CFLAGS is set as an environment variable, then it's set as a :=, which
means that CFLAGS is set right away, the first time buildmacros is
included.
How about this instead? A patch like this so that CFLAGS can be
remembered at configure time:
diff --git a/include/builddefs.in b/include/builddefs.in
index 4c8ec64..81ebfcd 100644
--- a/include/builddefs.in
+++ b/include/builddefs.in
@@ -25,6 +25,7 @@ OPTIMIZER = @opt_build@
MALLOCLIB = @malloc_lib@
LOADERFLAGS = @LDFLAGS@
LTLDFLAGS = @LDFLAGS@
+CFLAGS = @CFLAGS@
LIBRT = @librt@
LIBUUID = @libuuid@
Then I don't have to set an environment variable when running make; I
just have to pass it to configure.
- Ted
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply related [flat|nested] 30+ messages in thread
* Re: [PATCH XFSPROGS 2/3] Build without enabling DEBUG by default
2011-07-22 22:34 ` Ted Ts'o
@ 2011-07-23 1:32 ` Christoph Hellwig
-1 siblings, 0 replies; 30+ messages in thread
From: Christoph Hellwig @ 2011-07-23 1:32 UTC (permalink / raw)
To: Ted Ts'o; +Cc: Christoph Hellwig, Ext4 Developers List, xfs
On Fri, Jul 22, 2011 at 06:34:58PM -0400, Ted Ts'o wrote:
> Yes I needed it because I wanted to do a hermetic build; that is, I
> didn't want to depend on the version of the acl, attr, dmapi, etc.,
> shipped by the distribution. (This is actually more important when
> building xfstests, since I wanted to make sure we used the version of
> the xfsprogs headers shipped by the very tip of xfsprogs-dev, since in
> the past I've had problems because the distro-shipped header files
> were too old.)
Given that you are on a Debian-ish environment, did you consider simply
using pbuilder? I use that regularly to test clean package builds,
without all these weird overrides.
> The way I did that was by setting the environment CFLAGS to
> -I/kbuild/xfstests/bld/include. And that seems to be the problem. If
> CFLAGS is set as an environment variable, then it's set as a :=, which
> means that CFLAGS is set right away, the first time buildmacros is
> included.
>
> How about this instead? A patch like this so that CFLAGS can be
> remembered at configure time:
>
> diff --git a/include/builddefs.in b/include/builddefs.in
> index 4c8ec64..81ebfcd 100644
> --- a/include/builddefs.in
> +++ b/include/builddefs.in
> @@ -25,6 +25,7 @@ OPTIMIZER = @opt_build@
> MALLOCLIB = @malloc_lib@
> LOADERFLAGS = @LDFLAGS@
> LTLDFLAGS = @LDFLAGS@
> +CFLAGS = @CFLAGS@
>
> LIBRT = @librt@
> LIBUUID = @libuuid@
>
> Then I don't have to set an environment variable when running make; I
> just have to pass it to configure.
This looks much better, and is indeed similar to some other fixes we
recently had to put in, probably due to changing make behaviour.
If you add a signoff and a proper description I'll put it into all the
repositories.
^ permalink raw reply [flat|nested] 30+ messages in thread
* Re: [PATCH XFSPROGS 2/3] Build without enabling DEBUG by default
@ 2011-07-23 1:32 ` Christoph Hellwig
0 siblings, 0 replies; 30+ messages in thread
From: Christoph Hellwig @ 2011-07-23 1:32 UTC (permalink / raw)
To: Ted Ts'o; +Cc: Christoph Hellwig, Ext4 Developers List, xfs
On Fri, Jul 22, 2011 at 06:34:58PM -0400, Ted Ts'o wrote:
> Yes I needed it because I wanted to do a hermetic build; that is, I
> didn't want to depend on the version of the acl, attr, dmapi, etc.,
> shipped by the distribution. (This is actually more important when
> building xfstests, since I wanted to make sure we used the version of
> the xfsprogs headers shipped by the very tip of xfsprogs-dev, since in
> the past I've had problems because the distro-shipped header files
> were too old.)
Given that you are on a Debian-ish environment, did you consider simply
using pbuilder? I use that regularly to test clean package builds,
without all these weird overrides.
> The way I did that was by setting the environment CFLAGS to
> -I/kbuild/xfstests/bld/include. And that seems to be the problem. If
> CFLAGS is set as an environment variable, then it's set as a :=, which
> means that CFLAGS is set right away, the first time buildmacros is
> included.
>
> How about this instead? A patch like this so that CFLAGS can be
> remembered at configure time:
>
> diff --git a/include/builddefs.in b/include/builddefs.in
> index 4c8ec64..81ebfcd 100644
> --- a/include/builddefs.in
> +++ b/include/builddefs.in
> @@ -25,6 +25,7 @@ OPTIMIZER = @opt_build@
> MALLOCLIB = @malloc_lib@
> LOADERFLAGS = @LDFLAGS@
> LTLDFLAGS = @LDFLAGS@
> +CFLAGS = @CFLAGS@
>
> LIBRT = @librt@
> LIBUUID = @libuuid@
>
> Then I don't have to set an environment variable when running make; I
> just have to pass it to configure.
This looks much better, and is indeed similar to some other fixes we
recently had to put in, probably due to changing make behaviour.
If you add a signoff and a proper description I'll put it into all the
repositories.
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply [flat|nested] 30+ messages in thread
* Re: [PATCH XFSPROGS 2/3] Build without enabling DEBUG by default
2011-07-23 1:32 ` Christoph Hellwig
@ 2011-07-26 15:10 ` Christoph Hellwig
-1 siblings, 0 replies; 30+ messages in thread
From: Christoph Hellwig @ 2011-07-26 15:10 UTC (permalink / raw)
To: Ted Ts'o; +Cc: Christoph Hellwig, Ext4 Developers List, xfs
On Fri, Jul 22, 2011 at 09:32:14PM -0400, Christoph Hellwig wrote:
> > diff --git a/include/builddefs.in b/include/builddefs.in
> > index 4c8ec64..81ebfcd 100644
> > --- a/include/builddefs.in
> > +++ b/include/builddefs.in
> > @@ -25,6 +25,7 @@ OPTIMIZER = @opt_build@
> > MALLOCLIB = @malloc_lib@
> > LOADERFLAGS = @LDFLAGS@
> > LTLDFLAGS = @LDFLAGS@
> > +CFLAGS = @CFLAGS@
> >
> > LIBRT = @librt@
> > LIBUUID = @libuuid@
> >
> > Then I don't have to set an environment variable when running make; I
> > just have to pass it to configure.
>
> This looks much better, and is indeed similar to some other fixes we
> recently had to put in, probably due to changing make behaviour.
>
> If you add a signoff and a proper description I'll put it into all the
> repositories.
Can you at least provide the signoff? I can take care of the rest if
this means I can get it in ASAP.
^ permalink raw reply [flat|nested] 30+ messages in thread
* Re: [PATCH XFSPROGS 2/3] Build without enabling DEBUG by default
@ 2011-07-26 15:10 ` Christoph Hellwig
0 siblings, 0 replies; 30+ messages in thread
From: Christoph Hellwig @ 2011-07-26 15:10 UTC (permalink / raw)
To: Ted Ts'o; +Cc: Christoph Hellwig, Ext4 Developers List, xfs
On Fri, Jul 22, 2011 at 09:32:14PM -0400, Christoph Hellwig wrote:
> > diff --git a/include/builddefs.in b/include/builddefs.in
> > index 4c8ec64..81ebfcd 100644
> > --- a/include/builddefs.in
> > +++ b/include/builddefs.in
> > @@ -25,6 +25,7 @@ OPTIMIZER = @opt_build@
> > MALLOCLIB = @malloc_lib@
> > LOADERFLAGS = @LDFLAGS@
> > LTLDFLAGS = @LDFLAGS@
> > +CFLAGS = @CFLAGS@
> >
> > LIBRT = @librt@
> > LIBUUID = @libuuid@
> >
> > Then I don't have to set an environment variable when running make; I
> > just have to pass it to configure.
>
> This looks much better, and is indeed similar to some other fixes we
> recently had to put in, probably due to changing make behaviour.
>
> If you add a signoff and a proper description I'll put it into all the
> repositories.
Can you at least provide the signoff? I can take care of the rest if
this means I can get it in ASAP.
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply [flat|nested] 30+ messages in thread
* Re: [PATCH XFSPROGS 2/3] Build without enabling DEBUG by default
2011-07-26 15:10 ` Christoph Hellwig
@ 2011-07-30 16:55 ` Christoph Hellwig
-1 siblings, 0 replies; 30+ messages in thread
From: Christoph Hellwig @ 2011-07-30 16:55 UTC (permalink / raw)
To: Ted Ts'o; +Cc: Christoph Hellwig, Ext4 Developers List, xfs
On Tue, Jul 26, 2011 at 11:10:33AM -0400, Christoph Hellwig wrote:
> > This looks much better, and is indeed similar to some other fixes we
> > recently had to put in, probably due to changing make behaviour.
> >
> > If you add a signoff and a proper description I'll put it into all the
> > repositories.
>
> Can you at least provide the signoff? I can take care of the rest if
> this means I can get it in ASAP.
Ted,
can you please give me a signoff for this patch? I could probably just
put it in anyway given how trivial it is. But let's just stick to the
proper protocol if we can.
^ permalink raw reply [flat|nested] 30+ messages in thread
* Re: [PATCH XFSPROGS 2/3] Build without enabling DEBUG by default
@ 2011-07-30 16:55 ` Christoph Hellwig
0 siblings, 0 replies; 30+ messages in thread
From: Christoph Hellwig @ 2011-07-30 16:55 UTC (permalink / raw)
To: Ted Ts'o; +Cc: Christoph Hellwig, Ext4 Developers List, xfs
On Tue, Jul 26, 2011 at 11:10:33AM -0400, Christoph Hellwig wrote:
> > This looks much better, and is indeed similar to some other fixes we
> > recently had to put in, probably due to changing make behaviour.
> >
> > If you add a signoff and a proper description I'll put it into all the
> > repositories.
>
> Can you at least provide the signoff? I can take care of the rest if
> this means I can get it in ASAP.
Ted,
can you please give me a signoff for this patch? I could probably just
put it in anyway given how trivial it is. But let's just stick to the
proper protocol if we can.
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply [flat|nested] 30+ messages in thread
* [PATCH] Build using CFLAGS passed in at configure
2011-07-30 16:55 ` Christoph Hellwig
@ 2011-08-01 21:58 ` Theodore Ts'o
-1 siblings, 0 replies; 30+ messages in thread
From: Theodore Ts'o @ 2011-08-01 21:58 UTC (permalink / raw)
To: hch; +Cc: xfs, Ext4 Developers List, Theodore Ts'o
In order to build xfsprogs in a hermetic build, we need be able to
pass in -I and -L flags to the compiler and linker, respectively.
This needs to be used by the configure script, but we also need to
make sure these flags are used by the Makefiles as well.
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
---
include/builddefs.in | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/include/builddefs.in b/include/builddefs.in
index 4c8ec64..81ebfcd 100644
--- a/include/builddefs.in
+++ b/include/builddefs.in
@@ -25,6 +25,7 @@ OPTIMIZER = @opt_build@
MALLOCLIB = @malloc_lib@
LOADERFLAGS = @LDFLAGS@
LTLDFLAGS = @LDFLAGS@
+CFLAGS = @CFLAGS@
LIBRT = @librt@
LIBUUID = @libuuid@
--
1.7.4.1.22.gec8e1.dirty
^ permalink raw reply related [flat|nested] 30+ messages in thread
* [PATCH] Build using CFLAGS passed in at configure
@ 2011-08-01 21:58 ` Theodore Ts'o
0 siblings, 0 replies; 30+ messages in thread
From: Theodore Ts'o @ 2011-08-01 21:58 UTC (permalink / raw)
To: hch; +Cc: Ext4 Developers List, Theodore Ts'o, xfs
In order to build xfsprogs in a hermetic build, we need be able to
pass in -I and -L flags to the compiler and linker, respectively.
This needs to be used by the configure script, but we also need to
make sure these flags are used by the Makefiles as well.
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
---
include/builddefs.in | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/include/builddefs.in b/include/builddefs.in
index 4c8ec64..81ebfcd 100644
--- a/include/builddefs.in
+++ b/include/builddefs.in
@@ -25,6 +25,7 @@ OPTIMIZER = @opt_build@
MALLOCLIB = @malloc_lib@
LOADERFLAGS = @LDFLAGS@
LTLDFLAGS = @LDFLAGS@
+CFLAGS = @CFLAGS@
LIBRT = @librt@
LIBUUID = @libuuid@
--
1.7.4.1.22.gec8e1.dirty
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply related [flat|nested] 30+ messages in thread
* Re: [PATCH] Build using CFLAGS passed in at configure
2011-08-01 21:58 ` Theodore Ts'o
@ 2011-08-02 10:03 ` Christoph Hellwig
-1 siblings, 0 replies; 30+ messages in thread
From: Christoph Hellwig @ 2011-08-02 10:03 UTC (permalink / raw)
To: Theodore Ts'o; +Cc: hch, Ext4 Developers List, xfs
On Mon, Aug 01, 2011 at 05:58:24PM -0400, Theodore Ts'o wrote:
> In order to build xfsprogs in a hermetic build, we need be able to
> pass in -I and -L flags to the compiler and linker, respectively.
> This needs to be used by the configure script, but we also need to
> make sure these flags are used by the Makefiles as well.
Thanks a lot a Ted!
I've added this to xfsprogs, xfsdump and dmapi. xfstests actually
already has it, sent in in 2009 by Arek for the same reason you want
it in xfsprogs. Wish we would have been a bit better to propagate build
system fixes.
^ permalink raw reply [flat|nested] 30+ messages in thread
* Re: [PATCH] Build using CFLAGS passed in at configure
@ 2011-08-02 10:03 ` Christoph Hellwig
0 siblings, 0 replies; 30+ messages in thread
From: Christoph Hellwig @ 2011-08-02 10:03 UTC (permalink / raw)
To: Theodore Ts'o; +Cc: hch, Ext4 Developers List, xfs
On Mon, Aug 01, 2011 at 05:58:24PM -0400, Theodore Ts'o wrote:
> In order to build xfsprogs in a hermetic build, we need be able to
> pass in -I and -L flags to the compiler and linker, respectively.
> This needs to be used by the configure script, but we also need to
> make sure these flags are used by the Makefiles as well.
Thanks a lot a Ted!
I've added this to xfsprogs, xfsdump and dmapi. xfstests actually
already has it, sent in in 2009 by Arek for the same reason you want
it in xfsprogs. Wish we would have been a bit better to propagate build
system fixes.
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply [flat|nested] 30+ messages in thread
* [PATCH XFSPROGS 3/3] mkfs: Link against libuuid after liblkid
2011-07-21 23:50 ` Theodore Ts'o
@ 2011-07-21 23:50 ` Theodore Ts'o
-1 siblings, 0 replies; 30+ messages in thread
From: Theodore Ts'o @ 2011-07-21 23:50 UTC (permalink / raw)
To: Ext4 Developers List, xfs; +Cc: Theodore Ts'o
The order in which libraries are searched matters if you are using
static libraries. Since libblkid uses some functions from libuuid, it
needs to come before libuuid in the link line.
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
---
mkfs/Makefile | 7 +++----
1 files changed, 3 insertions(+), 4 deletions(-)
diff --git a/mkfs/Makefile b/mkfs/Makefile
index 8b61c2f..75da633 100644
--- a/mkfs/Makefile
+++ b/mkfs/Makefile
@@ -11,10 +11,6 @@ FSTYP = fstyp
HFILES = xfs_mkfs.h
CFILES = maxtrres.c proto.c xfs_mkfs.c
-LLDLIBS = $(LIBXFS) $(LIBUUID) $(LIBRT) $(LIBPTHREAD)
-LTDEPENDENCIES = $(LIBXFS)
-LLDFLAGS = -static
^ permalink raw reply related [flat|nested] 30+ messages in thread
* [PATCH XFSPROGS 3/3] mkfs: Link against libuuid after liblkid
@ 2011-07-21 23:50 ` Theodore Ts'o
0 siblings, 0 replies; 30+ messages in thread
From: Theodore Ts'o @ 2011-07-21 23:50 UTC (permalink / raw)
To: Ext4 Developers List, xfs; +Cc: Theodore Ts'o
The order in which libraries are searched matters if you are using
static libraries. Since libblkid uses some functions from libuuid, it
needs to come before libuuid in the link line.
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
---
mkfs/Makefile | 7 +++----
1 files changed, 3 insertions(+), 4 deletions(-)
diff --git a/mkfs/Makefile b/mkfs/Makefile
index 8b61c2f..75da633 100644
--- a/mkfs/Makefile
+++ b/mkfs/Makefile
@@ -11,10 +11,6 @@ FSTYP = fstyp
HFILES = xfs_mkfs.h
CFILES = maxtrres.c proto.c xfs_mkfs.c
-LLDLIBS = $(LIBXFS) $(LIBUUID) $(LIBRT) $(LIBPTHREAD)
-LTDEPENDENCIES = $(LIBXFS)
-LLDFLAGS = -static
-
ifeq ($(ENABLE_BLKID),yes)
LLDLIBS += $(LIBBLKID)
CFLAGS += -DENABLE_BLKID
@@ -23,6 +19,9 @@ LLDLIBS += $(LIBDISK)
LTDEPENDENCIES += $(LIBDISK)
endif
+LLDLIBS += $(LIBXFS) $(LIBUUID) $(LIBRT) $(LIBPTHREAD)
+LTDEPENDENCIES += $(LIBXFS)
+LLDFLAGS = -static
LSRCFILES = $(FSTYP).c
LDIRT = $(FSTYP)
--
1.7.4.1.22.gec8e1.dirty
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply related [flat|nested] 30+ messages in thread
* Re: [PATCH XFSPROGS 3/3] mkfs: Link against libuuid after liblkid
2011-07-21 23:50 ` Theodore Ts'o
@ 2011-07-22 14:29 ` Christoph Hellwig
-1 siblings, 0 replies; 30+ messages in thread
From: Christoph Hellwig @ 2011-07-22 14:29 UTC (permalink / raw)
To: Theodore Ts'o; +Cc: Ext4 Developers List, xfs
On Thu, Jul 21, 2011 at 07:50:37PM -0400, Theodore Ts'o wrote:
> The order in which libraries are searched matters if you are using
> static libraries. Since libblkid uses some functions from libuuid, it
> needs to come before libuuid in the link line.
Thanks, applied.
^ permalink raw reply [flat|nested] 30+ messages in thread
* Re: [PATCH XFSPROGS 1/3] configure.in: Declare a requirement for at least autoconf 2.50
2011-07-21 23:50 ` Theodore Ts'o
@ 2011-07-22 14:28 ` Christoph Hellwig
-1 siblings, 0 replies; 30+ messages in thread
From: Christoph Hellwig @ 2011-07-22 14:28 UTC (permalink / raw)
To: Theodore Ts'o; +Cc: Ext4 Developers List, xfs
On Thu, Jul 21, 2011 at 07:50:35PM -0400, Theodore Ts'o wrote:
> On Debian/Ubuntu systems, if autoconf version 2.13 is installed,
> autoconf will try to automatically figure out whether autoconf 2.13 or
> something more modern is required (since the autoconf maintainers,
> curses be upon them, didn't bother to maintain compatibility between
> autoconf 2.13 and 2.50). Unfortunately, the hueristics aren't
> perfect, and although the configure.in file looks superficially like
> it will be compatible with autoconf 2.13, it isn't. You will end up
> with a number of very subtle compilation failures if you use autoconf
> 2.13.
>
> So declare a requirement for autoconf 2.50 using AC_PREREQ(2.50).
Thanks a lot, I've applied our xfsprogs and xfstests patches, and also
ported the change to the xfsdump and dmapi repositories.
^ permalink raw reply [flat|nested] 30+ messages in thread
* Re: [PATCH XFSPROGS 1/3] configure.in: Declare a requirement for at least autoconf 2.50
@ 2011-07-22 14:28 ` Christoph Hellwig
0 siblings, 0 replies; 30+ messages in thread
From: Christoph Hellwig @ 2011-07-22 14:28 UTC (permalink / raw)
To: Theodore Ts'o; +Cc: Ext4 Developers List, xfs
On Thu, Jul 21, 2011 at 07:50:35PM -0400, Theodore Ts'o wrote:
> On Debian/Ubuntu systems, if autoconf version 2.13 is installed,
> autoconf will try to automatically figure out whether autoconf 2.13 or
> something more modern is required (since the autoconf maintainers,
> curses be upon them, didn't bother to maintain compatibility between
> autoconf 2.13 and 2.50). Unfortunately, the hueristics aren't
> perfect, and although the configure.in file looks superficially like
> it will be compatible with autoconf 2.13, it isn't. You will end up
> with a number of very subtle compilation failures if you use autoconf
> 2.13.
>
> So declare a requirement for autoconf 2.50 using AC_PREREQ(2.50).
Thanks a lot, I've applied our xfsprogs and xfstests patches, and also
ported the change to the xfsdump and dmapi repositories.
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply [flat|nested] 30+ messages in thread
end of thread, other threads:[~2011-08-02 10:04 UTC | newest]
Thread overview: 30+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-07-21 23:50 [PATCH XFSPROGS 1/3] configure.in: Declare a requirement for at least autoconf 2.50 Theodore Ts'o
2011-07-21 23:50 ` Theodore Ts'o
2011-07-21 23:50 ` [PATCH XFSPROGS 2/3] Build without enabling DEBUG by default Theodore Ts'o
2011-07-21 23:50 ` Theodore Ts'o
2011-07-22 14:17 ` Christoph Hellwig
2011-07-22 14:17 ` Christoph Hellwig
2011-07-22 18:30 ` Ted Ts'o
2011-07-22 18:30 ` Ted Ts'o
2011-07-22 18:38 ` Christoph Hellwig
2011-07-22 18:38 ` Christoph Hellwig
2011-07-22 22:30 ` Alex Elder
2011-07-22 22:30 ` Alex Elder
2011-07-22 22:34 ` Ted Ts'o
2011-07-22 22:34 ` Ted Ts'o
2011-07-23 1:32 ` Christoph Hellwig
2011-07-23 1:32 ` Christoph Hellwig
2011-07-26 15:10 ` Christoph Hellwig
2011-07-26 15:10 ` Christoph Hellwig
2011-07-30 16:55 ` Christoph Hellwig
2011-07-30 16:55 ` Christoph Hellwig
2011-08-01 21:58 ` [PATCH] Build using CFLAGS passed in at configure Theodore Ts'o
2011-08-01 21:58 ` Theodore Ts'o
2011-08-02 10:03 ` Christoph Hellwig
2011-08-02 10:03 ` Christoph Hellwig
2011-07-21 23:50 ` [PATCH XFSPROGS 3/3] mkfs: Link against libuuid after liblkid Theodore Ts'o
2011-07-21 23:50 ` Theodore Ts'o
2011-07-22 14:29 ` Christoph Hellwig
2011-07-22 14:29 ` Christoph Hellwig
2011-07-22 14:28 ` [PATCH XFSPROGS 1/3] configure.in: Declare a requirement for at least autoconf 2.50 Christoph Hellwig
2011-07-22 14:28 ` Christoph Hellwig
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.