From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:37574)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <pbonzini@redhat.com>) id 1WwXhO-0002op-Cd
	for qemu-devel@nongnu.org; Mon, 16 Jun 2014 10:15:55 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <pbonzini@redhat.com>) id 1WwXhI-0004re-VA
	for qemu-devel@nongnu.org; Mon, 16 Jun 2014 10:15:50 -0400
Received: from mx1.redhat.com ([209.132.183.28]:38387)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <pbonzini@redhat.com>) id 1WwXhI-0004rQ-Ld
	for qemu-devel@nongnu.org; Mon, 16 Jun 2014 10:15:44 -0400
Message-ID: <539EFC0A.2070201@redhat.com>
Date: Mon, 16 Jun 2014 16:15:38 +0200
From: Paolo Bonzini <pbonzini@redhat.com>
MIME-Version: 1.0
References: <1402865252-25671-1-git-send-email-sbruno@freebsd.org>	
	<1402865252-25671-2-git-send-email-sbruno@freebsd.org>	
	<CAFEAcA8zhYjm5zpRc8A+2ZrBQUBW8xeb3ch2OiP0WU7RhqjYRg@mail.gmail.com>	
	<20140616091227.GF1943@T430.nay.redhat.com>
	<539EC217.9070909@redhat.com>	 <1402926920.16961.1.camel@bruno>
	<1402927131.16961.2.camel@bruno>
In-Reply-To: <1402927131.16961.2.camel@bruno>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
Subject: Re: [Qemu-devel] [PATCH 1/3 v2] bsd-user: Revert part of update to
 rules.mk
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: sbruno@freebsd.org
Cc: Fam Zheng <famz@redhat.com>, QEMU Developers <qemu-devel@nongnu.org>

Il 16/06/2014 15:58, Sean Bruno ha scritto:
> On Mon, 2014-06-16 at 06:55 -0700, Sean Bruno wrote:
>>>> Sean, could you show the configure command line?
>>>
>>> Also please attach the "make V=1" output and config-host.mak file.
>>>
>>> Paolo
>>>
>>>
>>
>> I'm using:
>>
>>  ./configure --static --target-list="i386-bsd-user sparc-bsd-user
>> sparc64-bsd-user x86_64-bsd-user"
>>
>> Log:
>> http://people.freebsd.org/~sbruno/qemu_build_fail.txt
>>
>> sean
>>
>>
>
> Also, configure output
>
> http://people.freebsd.org/~sbruno/qemu_user_configure.txt

So this is happening because the $(sort) removes the ordering between 
libraries and breaks static linking.  Your patch likely does nothing 
except adding more copies of the libraries, which lets the linker work.

The executable that breaks is qemu-nbd, which I suspect you couldn't 
care less about.

The bug remains, and ought to be fixed, probably like this:

diff --git a/rules.mak b/rules.mak
index dde8e00..50b305e 100644
--- a/rules.mak
+++ b/rules.mak
@@ -22,7 +22,7 @@ QEMU_DGFLAGS += -MMD -MP -MT $@ -MF $(*D)/$(*F).d
  # Same as -I$(SRC_PATH) -I., but for the nested source/object directories
  QEMU_INCLUDES += -I$(<D) -I$(@D)

-extract-libs = $(strip $(sort $(foreach o,$1,$($o-libs))))
+extract-libs = $(foreach o,$1,$($o-libs))
  expand-objs = $(strip $(sort $(filter %.o,$1)) \
                    $(foreach o,$(filter %.mo,$1),$($o-objs)) \
                    $(filter-out %.o %.mo,$1))

but the other question is: do we care about static linking of anything 
except the user-mode emulators?  Should --static automatically imply 
--disable-tools --disable-system?

Paolo