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)
Wed, Nov 5, 9:04 AM
Unknown Object (File)
Thu, Oct 30, 1:34 AM
Unknown Object (File)
Fri, Oct 24, 12:04 AM
Unknown Object (File)
Fri, Oct 17, 7:33 PM
Unknown Object (File)
Oct 8 2025, 7:33 AM
Unknown Object (File)
Oct 7 2025, 9:55 PM
Unknown Object (File)
Sep 19 2025, 4:51 PM
Unknown Object (File)
Sep 13 2025, 10:09 AM

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.