Page MenuHomeFreeBSD

devel/mdb: Use clang 7 as compiler on 11.x and 12.x to fix build.
ClosedPublic

Authored by jhb on Nov 21 2019, 7:07 PM.

Details

Summary

clang 8 and later error for duplicate definitions of enum values. mdb
depends on having identical but duplicate definitions. This was fixed
differently in 13 with a change to the kernel, but that change is
disruptive and cannot be easily backported.

Test Plan
  • built on 13 (still used cc) and stable/11 (used clang70)

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

jhb created this revision.Nov 21 2019, 7:07 PM
bdrewery accepted this revision.Nov 21 2019, 7:31 PM
This revision is now accepted and ready to land.Nov 21 2019, 7:31 PM
dch added a subscriber: dch.Nov 21 2019, 7:31 PM
dch added inline comments.
devel/mdb/Makefile
34 ↗(On Diff #64686)

per IRC, guard this? .if ${OPSYS} == FreeBSD && ..

This revision was automatically updated to reflect the committed changes.
linimon retitled this revision from Use clang 7 as compiler on 11.x and 12.x to fix build. to devel/mdb: Use clang 7 as compiler on 11.x and 12.x to fix build..Nov 22 2019, 1:41 AM