[MacPorts] #49595: [KDE4]: move headerfiles to a dedicated TLD
MacPorts
noreply at macports.org
Thu Nov 5 07:16:50 PST 2015
#49595: [KDE4]: move headerfiles to a dedicated TLD
-------------------------+--------------------------------
Reporter: rjvbertin@… | Owner: macports-tickets@…
Type: defect | Status: new
Priority: Normal | Milestone:
Component: ports | Version:
Keywords: haspatch | Port: kdelibs4
-------------------------+--------------------------------
As already mentioned on macports-dev, it turns out to be preferable to
store the KDE4 headerfiles under a prefix directory where there is less
chance they'll clash with KF5 headerfiles. Otherwise, conflicts will arise
when building KF5 frameworks or applications (I've already witnessed such
conflicts that were resolved by the attached fix, which is why I label
this ticket a defect).
The attached patchfile accomplishes that. It defines the new path,
`${prefix}/include/KDE4` in the KDE4 PortGroup, and applies it the
kdelibs4 Portfile.
For the (presumable vast) majority of dependent ports this is all that is
required, as kdelibs4 installs cmake scripts that will set the new include
path. There are a few exceptions like Phonon and Attica, but as far as I
can tell now they will not lead to clashes so their headers can remain
where they are now.
I was afraid that all dependent ports might require a revbump because of
this, but it turns out that that is not strictly required. The current
patch does '''''not''''' cause binary incompatibilities, and the change to
the header search path in the cmake scripts does not cause errors finding
headers in their old locations. That change only ensures that the headers
can be found in their new location. (Ex: I could rebuild `port:kdenlive`
without having to rebuild `port:kde4-runtime` first, and `port:nepomuk-
widgets` without having to rebuild `port:nepomuk-core` first.)
--
Ticket URL: <https://trac.macports.org/ticket/49595>
MacPorts <https://www.macports.org/>
Ports system for OS X
More information about the macports-tickets
mailing list