Page MenuHomeFreeBSD

devel/freebsd-gcc6: update to GCC 6.5.0.
ClosedPublic

Authored by jhb on Fri, Nov 22, 8:24 PM.

Details

Summary

Take maintainership.

Test Plan
  • built mips64, aarch64, and i386 worlds with cross toolchain

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 27721
Build 25920: arc lint + arc unit

Event Timeline

jhb created this revision.Fri, Nov 22, 8:24 PM
lwhsu accepted this revision.Sat, Nov 23, 5:00 AM
This revision is now accepted and ready to land.Sat, Nov 23, 5:00 AM
lwhsu added a comment.Sat, Nov 23, 5:07 AM

BTW, I would suggest doing similar improvements like devel/binutils to have "native" FLAVOR, now the default is aarch64.

ref: https://lists.freebsd.org/pipermail/freebsd-toolchain/2019-November/005074.html

linimon retitled this revision from Update to GCC 6.5.0. to devel/freebsd-gcc6: update to GCC 6.5.0..Sat, Nov 23, 12:48 PM
jhb added a comment.Thu, Nov 28, 4:39 PM

BTW, I would suggest doing similar improvements like devel/binutils to have "native" FLAVOR, now the default is aarch64.
ref: https://lists.freebsd.org/pipermail/freebsd-toolchain/2019-November/005074.html

A native flavor doesn't make sense for this port though, that would mean we would have a 'freebsd-gcc6' package, but we have that already in base/gcc which installs to /usr, etc. Perhaps we can merge base/gcc into this someday, but that may not work well. In particular, you can't build base/gcc in poudriere because it writes to /usr which fails since the jail is mounted read-only and only permits writes to LOCALBASE. Also, base/gcc uses an extra patch and has some other settings to support cross-building, etc. I could perhaps make the default flavor be ${ARCH} instead of the first flavor in the list so that by default you get the 'amd64-gcc' package, but that would just mean that 'freebsd-gcc6' would alias 'freebsd-gcc6@amd64'. Note that this is always the case with flavors that the default flavor aliases with the base package name.

This revision was automatically updated to reflect the committed changes.