Btrfs devel - Nov 2011 - [btrfs-progs: PATCH] scrub: fix build failure by restoring proper library ordering

From: Sergei Trofimovich <slyfox@gentoo.org>

$ LDFLAGS=-static make
gcc -lpthread -g -O0 -o btrfs btrfs.o btrfs_cmds.o scrub.o \
        ctree.o disk-io.o radix-tree.o extent-tree.o print-tree.o root-tree.o
dir-item.o file-item.o inode-item.o inode-map.o crc32c.o rbtree.o extent-cache.o
extent_io.o volumes.o utils.o btrfs-list.o btrfslabel.o -static -luuid
scrub.o: In function `scrub_write_progress'':
/home/slyfox/linux-2.6-um-x86_64-fs/btrfs-progs-unstable/scrub.c:752: undefined
reference to `pthread_mutex_lock''
/home/slyfox/linux-2.6-um-x86_64-fs/btrfs-progs-unstable/scrub.c:758: undefined
reference to `pthread_setcancelstate''
/home/slyfox/linux-2.6-um-x86_64-fs/btrfs-progs-unstable/scrub.c:783: undefined
reference to `pthread_mutex_unlock''
/home/slyfox/linux-2.6-um-x86_64-fs/btrfs-progs-unstable/scrub.c:787: undefined
reference to `pthread_setcancelstate''

Signed-off-by: Sergei Trofimovich <slyfox@gentoo.org>

---
 Makefile |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/Makefile b/Makefile
index edee1a0..4d498b9 100644
--- a/Makefile
+++ b/Makefile
@@ -36,8 +36,8 @@ version:
 	bash version.sh
 
 btrfs: $(objects) btrfs.o btrfs_cmds.o scrub.o
-	$(CC) -lpthread $(CFLAGS) -o btrfs btrfs.o btrfs_cmds.o scrub.o \
-		$(objects) $(LDFLAGS) $(LIBS)
+	$(CC) $(CFLAGS) -o btrfs btrfs.o btrfs_cmds.o scrub.o \
+		$(objects) $(LDFLAGS) $(LIBS) -lpthread
 
 btrfsctl: $(objects) btrfsctl.o
 	$(CC) $(CFLAGS) -o btrfsctl btrfsctl.o $(objects) $(LDFLAGS) $(LIBS)
-- 
1.7.3.4

--
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

On Mon, Nov 07, 2011 at 10:19:33AM +0300, slyich@gmail.com
wrote:
>  btrfs: $(objects) btrfs.o btrfs_cmds.o scrub.o
> -	$(CC) -lpthread $(CFLAGS) -o btrfs btrfs.o btrfs_cmds.o scrub.o \
> -		$(objects) $(LDFLAGS) $(LIBS)
> +	$(CC) $(CFLAGS) -o btrfs btrfs.o btrfs_cmds.o scrub.o \
> +		$(objects) $(LDFLAGS) $(LIBS) -lpthread
No, it''s "-pthread", adds both preprocessor and linker flags.


david
--
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

Hi David,

On 07.11.2011 13:45, David Sterba wrote:
> On Mon, Nov 07, 2011 at 10:19:33AM +0300, slyich@gmail.com wrote:
>>  btrfs: $(objects) btrfs.o btrfs_cmds.o scrub.o
>> -	$(CC) -lpthread $(CFLAGS) -o btrfs btrfs.o btrfs_cmds.o scrub.o \
>> -		$(objects) $(LDFLAGS) $(LIBS)
>> +	$(CC) $(CFLAGS) -o btrfs btrfs.o btrfs_cmds.o scrub.o \
>> +		$(objects) $(LDFLAGS) $(LIBS) -lpthread
> 
> No, it''s "-pthread", adds both preprocessor and linker
flags.
It seems to do the right thing on x86_64, although...

If I''m reading my gcc manpage right, it is telling me to use -pthread
only for ia64 on HPUX, powerpc and sparc. (gcc 4.4.5 that is)

-Jan
--
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