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)
Mon, Apr 22, 7:22 PM
Unknown Object (File)
Mon, Apr 22, 7:22 PM
Unknown Object (File)
Mon, Apr 22, 7:22 PM
Unknown Object (File)
Mon, Apr 22, 7:22 PM
Unknown Object (File)
Mon, Apr 22, 7:09 PM
Unknown Object (File)
Mon, Apr 22, 10:34 AM
Unknown Object (File)
Sat, Apr 20, 12:57 PM
Unknown Object (File)
Mon, Apr 15, 7:07 AM
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
Lint Not Applicable
Unit
Tests Not Applicable

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.