On Tue, 12 Mar 2013 12:49:03 +0000 Max Brazhnikov wrote:
> SVN commit 9076 by makc:
> 
> bsd.cmake.mk
> - Deprecate CMAKE_OUTSOURCE in favour of 'USE_CMAKE=  outsource'.
> - Add guards to ensure bsd.cmake.mk is included only once.
> - Remove trivial comments
> - Enable verbosity/disable colouring when PACKAGE_BUILDING is set.
>   Although tinderbox and pointyhat set BATCH as well, the PACKAGE_BUILDING
>   is more appropriate for this check. I think the check for BATCH can be
>   removed from here, but I leave it for now.
> 
> bsd.kde4.mk
> - Use 'USE_CMAKE=     outsource' for KDE4_BUILDENV

Actually, I don't any reason why we should keep KDE4_BUILDENV:
de facto it means USE_LDCONFIG=yes and USE_CMAKE=outsource now.
The first only pollutes numerous ports, that don't install libraries, the 
second can replace KDE4_BUILDENV in ports Makefiles.

KDE4_VERSION in PLIST_SUB and redefined CMAKE_BUILD_TYPE won't hunt non KDE SC 
ports. Any objection against removing it?

Max
Index: bsd.kde4.mk
===================================================================
--- bsd.kde4.mk	(revision 9076)
+++ bsd.kde4.mk	(working copy)
@@ -102,7 +102,8 @@
 . endif
 .endif
 
-PLIST_SUB+=				KDE4_PREFIX="${KDE4_PREFIX}"
+PLIST_SUB+=				KDE4_PREFIX="${KDE4_PREFIX}" \
+						KDE4_VERSION="${KDE4_VERSION}"
 
 # Keep in sync with cmake/modules/PythonMacros.cmake
 _PYTHON_SHORT_VER=	${PYTHON_VERSION:S/^python//:S/.//}
@@ -116,24 +117,10 @@
 		PYO_SUFFIX=pyo
 .endif
 
-# The following definitions are not intended for usage in KDE4-dependent ports.
-.if defined(KDE4_BUILDENV)
-
-USE_CMAKE=				outsource
-USE_LDCONFIG=			yes
-
-. if ${KDE4_BRANCH} == "unstable"
-WITH_DEBUG=				yes
-. endif
-
 . if defined(WITH_DEBUG)
 CMAKE_BUILD_TYPE=		DebugFull
 . endif
 
-PLIST_SUB+=				KDE4_VERSION="${KDE4_VERSION}"
-
-.endif # defined(KDE4_BUILDENV)
-
 .endif # !defined(_POSTMKINCLUDED) && !defined(Kde_Pre_Include)
 
 .if defined(_POSTMKINCLUDED) && !defined(Kde_Post_Include)
_______________________________________________
kde-freebsd mailing list
kde-freebsd@kde.org
https://mail.kde.org/mailman/listinfo/kde-freebsd
See also http://freebsd.kde.org/ for latest information

Reply via email to