Page MenuHomeFreeBSD

Sprinkle bits around to make ports build on tier2/gcc targets
ClosedPublic

Authored by linimon on Jul 5 2018, 1:27 AM.
Tags
None
Referenced Files
Unknown Object (File)
Dec 29 2025, 1:58 PM
Unknown Object (File)
Oct 30 2025, 9:37 PM
Unknown Object (File)
Oct 25 2025, 12:46 AM
Unknown Object (File)
Oct 14 2025, 9:11 PM
Unknown Object (File)
Oct 11 2025, 12:17 AM
Unknown Object (File)
Oct 8 2025, 4:59 AM
Unknown Object (File)
Oct 6 2025, 4:01 AM
Unknown Object (File)
Sep 30 2025, 1:45 PM

Details

Reviewers
jhibbits
linimon
Group Reviewers
office
multimedia
gnome
Summary

Most of the ports were missing compiler:* USES flags. A few were
simply missing USE_CSTD=gnu99, as evidenced in the error messages.

Some are just plain broken (math/superlu_mt is a runaway on powerpc64), so mark
as such.

Test Plan

Only test built on powerpc64.

Diff Detail

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

Event Timeline

jbeich added inline comments.
multimedia/libvpx/Makefile
70 โ†—(On Diff #44881)

Why do you need this in addition to USES=compiler:c++11-lib? See rP472311.

jhibbits added inline comments.
multimedia/libvpx/Makefile
70 โ†—(On Diff #44881)

I don't. This was leftovers. I thought I had cleaned up everything during my tree diff move, but apparently some things slipped through. Sorry about that.

jhibbits marked an inline comment as done.

Revert a no-longer-necessary diff against libvpx. There may be more, as I did
the work on one tree and submitted the diff from another tree.

games/apricots/Makefile
25

CXXFLAGS.clang is only supported by default in base system. Try adding the following:

USES=	compiler

CXXFLAGS+=	${CXXFLAGS_${CHOSEN_COMPILER_TYPE}}

Ditto in other ports. Also, keep separator consistent with CXXFLAGS_powerpc64 support in Mk/bsd.port.mk: replace CXXFLAGS.clang (dot) with CXXFLAGS_clang (underscore).

deskutils/zim/Makefile
22

These two changes seem unneccessary.

graphics/gnash/Makefile
29

I will sort these before my testing.

math/superlu_mt/Makefile
23

I have already committed this, as it needed no testing.

graphics/osg/Makefile
7

Why the change here?

textproc/tinyxml2/Makefile
7

Also seems unneeded?

For this run I will also try to add the following conditional to mail/exim/Makefile:

.if ${CHOSEN_COMPILER_TYPE} == gcc
EXIM_WARN_FLAGS?= -Wno-logical-op-parentheses -Wno-macro-redefined -Wno-parentheses -Wno-dangling-else
.endif

Thanks for the review, guys. It's quite obvious now that my patch is more WIP than complete and ready, but I think the bulk is still good.

deskutils/zim/Makefile
22

Yup, I think I had added a compiler: directive in there, then just removed the directive instead of reverting the whole file.

graphics/osg/Makefile
7

Because this can already build on tier2 archs, so it needs to be rebuilt.

textproc/tinyxml2/Makefile
7

Same as above, this is needed to force-rebuild the port.

linimon accepted this revision.
linimon edited reviewers, added: jhibbits; removed: linimon.

Overcome by subsequent events.

This revision is now accepted and ready to land.Dec 21 2018, 8:25 PM

Overcome by subsequent events.