kde4 1.1 PortGroup file adaptation to qt4-mac +concurrent

[RJVB (René Bertin)]

The kde4 1.1 PortGroup file contains hardcoded declarations for the QCA2 include and library directories. Those are inappropriate for qt4-mac +concurrent, and in my testing they are also not required.

[mkae (Marko Käning)]

Sure that committing this won't break anything?

For me it's unclear how to test this without rebuilding whole kdelibs4 and perhaps port:konversation...

[RJVB (René Bertin)]

This change won't require anything to be rebuilt. It removes instructions where to find QCA during a build. When I tried with the standard (traditional, exclusive) port:qt4-mac active those instructions were redundant (= not required), so removing them didn't break anything. For me.

Testing is simple enough: port -v configure foo where foo is any KDE4 port that depends on port:qca (or possibly port:qca-ossl). If that doesn't raise errors or warnings about something qca-related not being found you have your proof. You can also double-check in the generated CMakeCache.txt file if the qca stuff is found in the correct locations.

The reason that I like to have this committed is that PortGroup definitions outside the default repository apparently apply only to the ports in the same repository. Or maybe the rule is that a PortGroup definition in a port repository overrides PortGroup definitions from outside that repository.

[mkae (Marko Käning)]

Since I wasn't sure what you really meant in your last paragraph (w.r.t. default and same), I removed these two lines in my base installation, as well as in the checked out port group definition in the corresponding dports directory and ended up with this:

-----------------------------------------------------------------------------
-- The following external packages were located on your system.
-- This installation will have the extra features provided by these packages.
-----------------------------------------------------------------------------
   * QCA - Qt Cryptographic Architecture

-----------------------------------------------------------------------------
-- Congratulations! All external packages have been found.
-----------------------------------------------------------------------------

meaning that a removal of these two options is possible. :-)

[mkae (Marko Käning)]

BTW, your patch wasn't applicable as is, since the portgroup had changed in the meantime.

[mkae (Marko Käning)]

Done in r133738

[mkae (Marko Käning)]

Thanks, René, for supplying just committed r133799.