Page MenuHomeFreeBSD

converters/lua-iconv: fix build on armv7
ClosedPublic

Authored by pkubaj on Aug 24 2019, 12:56 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Dec 15, 11:39 PM
Unknown Object (File)
Nov 18 2025, 3:57 AM
Unknown Object (File)
Nov 5 2025, 9:04 AM
Unknown Object (File)
Oct 30 2025, 1:34 AM
Unknown Object (File)
Oct 24 2025, 12:04 AM
Unknown Object (File)
Oct 17 2025, 7:33 PM
Unknown Object (File)
Oct 8 2025, 7:33 AM
Unknown Object (File)
Oct 7 2025, 9:55 PM

Details

Summary

armv7 requires -fPIC too.

PR: 239623

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Approved. IMHO there should also be an entry for armv6.

This revision is now accepted and ready to land.Aug 31 2019, 11:13 AM

Why isn't this just CFLAGS+=-fPIC ?

Because e.g. powerpc64 doesn't need it.

This revision was automatically updated to reflect the committed changes.

Why isn't this just CFLAGS+=-fPIC ?

Because e.g. powerpc64 doesn't need it.

What that says to me is that we need a standard ${CFLAGS_PIC} variable or some such, which is defined to -fPIC on most platforms and empty on powerpc64, rather than pollute every port makefile with an ever-changing list of which arches need -fPIC and which don't.