#40656: Use -isystem instead of -I in default configure.cppflags ---------------------------+-------------------------------- Reporter: ryandesign@… | Owner: macports-tickets@… Type: enhancement | Status: reopened Priority: Normal | Milestone: MacPorts Future Component: base | Version: 2.2.0 Resolution: | Keywords: haspatch Port: | ---------------------------+-------------------------------- Comment (by jmr@…): Replying to [comment:17 ryandesign@…]:
I have been using this patch for months without problems. This problem you've mentioned with swig is the first problem I've heard of. Could you be more specific? Could we fix that problem instead?
As per the link, lal fails to build. Let’s be clear: it is a bug when a build system passes externally-supplied CPPFLAGS to the compiler before its own internal include paths. IMO we should be working with upstream to fix such bugs, not applying a workaround globally. Even in cases where there is no upstream, just having a patch is more valuable to others in the larger community who encounter the same problem building the software, compared to having an invisible workaround. But it appears that I’m outvoted, so I’ll go along with reintroducing the change into trunk provided the problem with swig is fixed, and no other problems turn up. -- Ticket URL: <https://trac.macports.org/ticket/40656#comment:19> MacPorts <http://www.macports.org/> Ports system for OS X