search

TritonDataCenter / illumos-kvm

KVM driver for illumos
Other
119 stars 65 forks source link

OS-8263 KVM Incorrect LDFLAGS for 64-bit link #29

Closed citrus-it closed 3 years ago

citrus-it commented 3 years ago

Having recently fixed the OmniOS toolchain so that -zassert-deflib works properly for 64-bit objects, illumos-kvm fails to build with:

/opt/gcc-7/bin/gcc -Wl,-Bdirect -Wl,-zfatal-warnings -Wl,-zassert-deflib -Wl,-zguidance -m64 -shared -nodefaultlibs -std=gnu89 -Wl,-M/data/omnios-build/omniosorg/bloody/illumos/usr/src/common/mapfiles/common/map.pagealign -Wl,-M/data/omnios-build/omniosorg/bloody/illumos/usr/src/common/mapfiles/common/map.noexdata -Wl,-ztext -Wl,-zdefs -Wl,-zignore -Wl,-M/data/omnios-build/omniosorg/bloody/illumos/usr/src/cmd/mdb/common/modules/conf/mapfile-extern -L/data/omnios-build/omniosorg/bloody/illumos/proto/root_i386/lib -L/data/omnios-build/omniosorg/bloody/illumos/proto/root_i386/usr/lib -o kvm.so kvm_mdb.o -lc
ld: warning: dynamic library found on default search path (/lib/amd64): libc.so
collect2: error: ld returned 1 exit status

The wrong paths are being passed to -L. With this patch, the build is successful.