Page MenuHomeFreeBSD

games/fs2open: fix linking on 12.0 and higher when using LLVM
ClosedPublic

Authored by pkubaj on May 15 2019, 10:21 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Apr 9, 3:03 PM
Unknown Object (File)
Mon, Apr 8, 4:52 AM
Unknown Object (File)
Sun, Mar 31, 2:17 PM
Unknown Object (File)
Mar 16 2024, 10:32 PM
Unknown Object (File)
Feb 19 2024, 3:41 PM
Unknown Object (File)
Feb 19 2024, 3:41 PM
Unknown Object (File)
Feb 19 2024, 11:29 AM
Unknown Object (File)
Feb 19 2024, 11:25 AM
Subscribers

Details

Summary

This port was tested on 11.2-RELEASE (the oldest supported version) on amd64 and 12.0-RELEASE on powerpc64. Since 12.0-RELEASE amd64 uses lld.

This port doesn't link with lld. Fix that (LLD_UNSAFE doesn't work) and remove additional -march=native.

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 24294
Build 23120: arc lint + arc unit

Event Timeline

Though, the last two patches should be excluded, nothing in them changes really.

This revision is now accepted and ready to land.May 16 2019, 6:58 AM
tobik added inline comments.
games/fs2open/files/patch-cmake_toolchain-clang.cmake
19

FreeBSD 13.0 does not have ld.bfd in base anymore, so committing this would break the build there. Either add something like

.if !exists(/usr/bin/ld.bfd)
USE_BINUTILS=	yes
.endif

or make sure the build respects LDFLAGS so that LLD_UNSAFE=yes can actually work.

This revision now requires review to proceed.May 17 2019, 10:32 PM

Looks good to me and builds fine on 13.0/amd64, thanks.

I'd bump PORTREVISION too to trigger a rebuild since the build is without -march=native now and because CFLAGS, CXXFLAGS, LDFLAGS are actually respected by the build now.

games/fs2open/files/patch-freespace2_CMakeLists.txt
1โ€“3

Remove this patch from the lot, no functional changes. (I am looking at fixing makepatch to avoid keeping those patches.)

games/fs2open/files/patch-freespace2_CMakeLists.txt
1โ€“3

Fixed in rP502102.

Remove patch that doesn't change anything.

This revision is now accepted and ready to land.Jun 8 2019, 5:19 PM
This revision was automatically updated to reflect the committed changes.