[Buildroot] [PATCH 2/2] kvmtool: bump to f77d646ba0
Matthew Weber
matthew.weber at rockwellcollins.com
Wed Dec 6 14:29:32 UTC 2017
Thomas,
On Wed, Dec 6, 2017 at 7:46 AM, Matthew Weber
<matthew.weber at rockwellcollins.com> wrote:
> Thomas,
>
> On Wed, Dec 6, 2017 at 1:59 AM, Thomas Petazzoni
> <thomas.petazzoni at free-electrons.com> wrote:
>>
>> Hello,
>>
>> On Tue, 5 Dec 2017 20:05:42 -0600, Matthew Weber wrote:
>>
>> > Gustavo, the latest kvmtool switched to doing the final link with gcc
>> > vs originally ld. I believe this is the right fix.
>> >
>> > 3 --- a/package/kvmtool/kvmtool.mk
>> > 4 +++ b/package/kvmtool/kvmtool.mk
>> > 5 @@ -33,7 +33,7 @@ KVMTOOL_LICENSE_FILES = COPYING
>> > 6 # If more packages need this (unlikely) an ld wrapper might be a better
>> > 7 # solution, using gcc -dumpspecs information.
>> > 8 KVMTOOL_EXTRA_LDFLAGS = \
>> > 9 - $(if $(BR2_x86_64),-m elf_x86_64)
>> > 10 + $(if $(BR2_x86_64),-m64)
>>
>> If kvmtool is now building with gcc, do we need -m64 at all ?
>>
>
> That's true, gcc would already be assuming m64 based on the target's
> arch vs the linker wouldn't have been able to infer that. I'll remove
> and kickoff a test.
>
Verified, test-pkg now passes and so does your test config. Will send a v2.
Matt
More information about the buildroot
mailing list