Page MenuHomeFreeBSD

math/glm: make it compile with GCC 7
ClosedPublic

Authored by fernape on Aug 9 2018, 5:44 PM.
Tags
None
Referenced Files
Unknown Object (File)
Nov 7 2024, 4:38 AM
Unknown Object (File)
Oct 15 2024, 5:00 AM
Unknown Object (File)
Oct 12 2024, 11:01 PM
Unknown Object (File)
Oct 12 2024, 10:56 PM
Unknown Object (File)
Oct 6 2024, 9:10 PM
Unknown Object (File)
Oct 5 2024, 4:43 AM
Unknown Object (File)
Sep 30 2024, 1:15 PM
Unknown Object (File)
Sep 23 2024, 6:40 PM
Subscribers

Details

Summary

GCC 7 was set as default in r475856.

cad/openvsp broke because it used some headers from math/glm but try to compile them with GCC. On the other side, math/glm doesn't recognize GCC 7.x as a valid compiler. Import changes from dd48b56e44d699a022c69155c8672caacafd9e8a to update the list of supported compilers.

Test Plan

For both math/glm and cad/openvsp:

  • portlint -AC OK
  • poudriere builds for {10.4,11.1}{amd64,i386}, 12i386 OK

No other ports depending on math/glm try to build with gcc as far as I can see.

Diff Detail

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

Event Timeline

I think you need to bump the port revision. Otherwise the change won't be picked up, and the installed header would be still the unpatched one.

I think you need to bump the port revision. Otherwise the change won't be picked up, and the installed header would be still the unpatched one.

I'm not convinced that is strictly necessary. People building from ports will see the change when updating the tree and they'll be able to build cad/openvsp with the new math/glm (other ports are not affected by this). People using packages, will just install cad/openvsp 3.16.0 normally.

This revision is now accepted and ready to land.Aug 11 2018, 7:02 AM
This revision was automatically updated to reflect the committed changes.