[kde-freebsd] More changes needed for building net/kdenetwork4 version 4.1.85

[Max Brazhnikov]

On Sat, 27 Dec 2008 21:31:36 -0600, Matt wrote:
> Here's a start at it.  Options have been added for Meanwhile, GMP,
> oRTP, Decibel and XMMS and the plist has been updated with extra
> optional files that are built with the Meanwhile and GMP options.  It
> builds OK in tinderbox, but I haven't run-tested it yet.
>
> Some observations from this configuration:
>
> - I added an explicit dependency on the OpenSSL port because of a
> problem I've been having since version 4.1.3.  If that dependency is
> not there and I have the OpenSSL port installed, Kopete core dumps
> when I connect to my GroupWise chat server (which is an SSL
> connection).  I'm sure there is an underlying cause for this -
> suggestions welcome.
It shouldn't be set in ports Makefile. Instead put WITH_OPENSSL_PORT=yes (or 
WITH_OPENSSL_BASE=yes) into your make.conf

> - Only the Meanwhile option is accompanied by a corresponding cmake
> value in the src cmake files, which is why it's the only one that I've
> added a CMAKE_ARGS line for in the Makefile.

> - Without an explicit option to disable the GMP dependency at
> configure and build time, the plist generation will be incorrect if a
> user does not enable the GMP option but has the math/libgmp4 port
> installed when they run a make on the kdenetwork4 port.  This is
> because the GMP test is on by default in the kdenetwork4 configure
> script and if it finds the library it will include the option in the
> build resulting in two extra files being installed.  Any good ways
> around this?
make GMP support non-optional. I prefer Meanwhile plugin to be non-optional 
too. The better approach would be to create separate ports for plugins.

> - The oRTP, Decibel and XMMS detection works the same way as GMP, but
> these options are not resulting in extra files being installed.
ok

> - The oRTP detection in cmake/modules/FindLiboRTP.cmake is not working
> correctly.  The patch shown below works around the issue, but it
> doesn't seem like the "right" thing to do.
It doesn't respect even $CC :( .

> --- ../cmake/modules/FindLiboRTP.cmake.orig     2008-12-26
> 22:20:02.000000000 -0600
> +++ ../cmake/modules/FindLiboRTP.cmake  2008-12-26 22:22:03.000000000 -0600
> @@ -14,7 +14,7 @@
>  SET( LIBORTP_FOUND FALSE )
>
>  IF( LIBORTP_INCLUDE_DIR AND LIBORTP_LIBRARY )
> -  EXECUTE_PROCESS( COMMAND gcc -o ortpversion -lortp
> -L${LIBORTP_INCLUDE_DIR}
> ${CMAKE_SOURCE_DIR}/cmake/modules/ortpversion.c OUTPUT_QUIET
> ERROR_QUIET RESULT_VARIABLE compileErr )
> +  EXECUTE_PROCESS( COMMAND gcc -I/usr/local/include -o ortpversion
> -lortp -L/usr/local/lib
> ${CMAKE_SOURCE_DIR}/cmake/modules/ortpversion.c OUTPUT_QUIET
> ERROR_QUIET RESULT_VARIABLE compileErr )
>    if ( NOT compileErr )
>      EXECUTE_PROCESS(COMMAND ./ortpversion 0 13 RESULT_VARIABLE resultErr )
>      if ( NOT resultErr )

Max