From owner-svn-ports-all@freebsd.org Mon Dec 19 11:12:34 2016 Return-Path: Delivered-To: svn-ports-all@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 35F10C87AF0; Mon, 19 Dec 2016 11:12:34 +0000 (UTC) (envelope-from tcberner@FreeBSD.org) Received: from repo.freebsd.org (repo.freebsd.org [IPv6:2610:1c1:1:6068::e6a:0]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id E3568DFC; Mon, 19 Dec 2016 11:12:33 +0000 (UTC) (envelope-from tcberner@FreeBSD.org) Received: from repo.freebsd.org ([127.0.1.37]) by repo.freebsd.org (8.15.2/8.15.2) with ESMTP id uBJBCXwl024477; Mon, 19 Dec 2016 11:12:33 GMT (envelope-from tcberner@FreeBSD.org) Received: (from tcberner@localhost) by repo.freebsd.org (8.15.2/8.15.2/Submit) id uBJBCWdt024475; Mon, 19 Dec 2016 11:12:32 GMT (envelope-from tcberner@FreeBSD.org) Message-Id: <201612191112.uBJBCWdt024475@repo.freebsd.org> X-Authentication-Warning: repo.freebsd.org: tcberner set sender to tcberner@FreeBSD.org using -f From: "Tobias C. Berner" Date: Mon, 19 Dec 2016 11:12:32 +0000 (UTC) To: ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: svn commit: r428918 - in head: Mk devel/qt5/files X-SVN-Group: ports-head MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-BeenThere: svn-ports-all@freebsd.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: SVN commit messages for the ports tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 19 Dec 2016 11:12:34 -0000 Author: tcberner Date: Mon Dec 19 11:12:32 2016 New Revision: 428918 URL: https://svnweb.freebsd.org/changeset/ports/428918 Log: Change the way the pkgconfig-file install path is set for Qt5 ports. As noticed in x11-toolkits/qt5-charts we never corrected the way in which qmake sets the pkgconfig path for Qt5-modules. Instead there was a sed-call on the generated Makefiles in bsd.qt.mk in the target qt-pre-install. This diff modifies devel/qmake5 to set QMAKE_PKGCONFIG_DESTDIR to ../libdata/pkgconfig from the default pkgconfig. As we split Qt-base into multiple ports, simply patching devel/qmake5 is not enough, as the other base-ports will use the file from inside the tarball. Therefore add it as an extrapatch for the Qt-base ports. As this affects the same file extrapatch-libtool in devel/qt5 touched, rename this one and add a hunk to do this change too. Note: qt4 ports are unaffected, as they never were converted to used the qt-preinstall target, and still contain the sed-call in their Makefiles. Reviewed by: rakuco Approved by: rakuco (mentor) Differential Revision: https://reviews.freebsd.org/D8713 Added: head/devel/qt5/files/extrapatch-mkspecs_features_qt__module.prf - copied, changed from r428917, head/devel/qt5/files/extrapatch-libtool Deleted: head/devel/qt5/files/extrapatch-libtool Modified: head/Mk/bsd.qt.mk Modified: head/Mk/bsd.qt.mk ============================================================================== --- head/Mk/bsd.qt.mk Mon Dec 19 09:50:20 2016 (r428917) +++ head/Mk/bsd.qt.mk Mon Dec 19 11:12:32 2016 (r428918) @@ -170,13 +170,14 @@ CONFIGURE_ARGS+=-verbose . if ${QT_DIST} == "base" || ${_QT_VERSION:M4*} . if ${_QT_VERSION:M4*} -_EXTRA_PATCHES_QT4= ${.CURDIR:H:H}/devel/${_QT_RELNAME}/files/extrapatch-src-corelib-global-qglobal.h +_EXTRA_PATCHES_QT4= ${.CURDIR:H:H}/devel/${_QT_RELNAME}/files/extrapatch-src-corelib-global-qglobal.h \ + ${.CURDIR:H:H}/devel/${_QT_RELNAME}/files/extrapatch-libtool . else -_EXTRA_PATCHES_QT5= ${.CURDIR:H:H}/devel/${_QT_RELNAME}/files/extrapatch-src_corelib_global_qcompilerdetection.h +_EXTRA_PATCHES_QT5= ${.CURDIR:H:H}/devel/${_QT_RELNAME}/files/extrapatch-src_corelib_global_qcompilerdetection.h \ + ${.CURDIR:H:H}/devel/${_QT_RELNAME}/files/extrapatch-mkspecs_features_qt__module.prf . endif EXTRA_PATCHES?= ${.CURDIR:H:H}/devel/${_QT_RELNAME}/files/extrapatch-configure \ ${.CURDIR:H:H}/devel/${_QT_RELNAME}/files/extrapatch-config.tests-unix-compile.test \ - ${.CURDIR:H:H}/devel/${_QT_RELNAME}/files/extrapatch-libtool \ ${_EXTRA_PATCHES_QT4} ${_EXTRA_PATCHES_QT5} . endif @@ -683,13 +684,6 @@ qt5-pre-configure: # See QTBUG-40825 and ports bugs 194088, 195105 and 198720. ${ECHO_CMD} 'QMAKE_LIBDIR_FLAGS = -L${CONFIGURE_WRKSRC}/lib' >> ${CONFIGURE_WRKSRC}/.qmake.cache -pre-install: qt-pre-install -qt-pre-install: -# Search both in CONFIGURE_WRKSRC and WRKSRC, as the former is not -# a subdirectory of the latter for out-of-source builds. - @${FIND} ${WRKSRC} ${CONFIGURE_WRKSRC} -name "Makefile*" -type f | \ - ${XARGS} ${REINPLACE_CMD} -e 's,${PREFIX}/${QT_LIBDIR_REL}/pkgconfig,${PREFIX}/libdata/pkgconfig,g' - post-install: qt-post-install qt-post-install: . if ${QT_DEFINES:N-*} Copied and modified: head/devel/qt5/files/extrapatch-mkspecs_features_qt__module.prf (from r428917, head/devel/qt5/files/extrapatch-libtool) ============================================================================== --- head/devel/qt5/files/extrapatch-libtool Mon Dec 19 09:50:20 2016 (r428917, copy source) +++ head/devel/qt5/files/extrapatch-mkspecs_features_qt__module.prf Mon Dec 19 11:12:32 2016 (r428918) @@ -1,6 +1,22 @@ ---- mkspecs/features/qt_module.prf.orig 2016-02-24 21:20:22.414067000 +0100 -+++ mkspecs/features/qt_module.prf 2016-02-24 21:20:35.884538000 +0100 -@@ -267,7 +267,6 @@ +The first hunk fixes the install location of the pkconfig files: + FreeBSD installs pkgconfig files to $LOCALBASE/libdata/pkgconfig + qmake uses QMAKE_PKGCONFIG_DESTDIR as relative path to $$QT_LIBDIR + respectively as absolute path. + +The second hunk stops qmake from generating la files. + +--- mkspecs/features/qt_module.prf.orig 2016-09-16 05:49:42 UTC ++++ mkspecs/features/qt_module.prf +@@ -247,7 +247,7 @@ load(qt_targets) + # this builds on top of qt_common + !internal_module:!lib_bundle:if(unix|mingw) { + CONFIG += create_pc +- QMAKE_PKGCONFIG_DESTDIR = pkgconfig ++ QMAKE_PKGCONFIG_DESTDIR = ../libdata/pkgconfig + host_build: \ + QMAKE_PKGCONFIG_LIBDIR = $$[QT_HOST_LIBS] + else: \ +@@ -267,7 +267,6 @@ load(qt_targets) QMAKE_PKGCONFIG_INSTALL_REPLACE += pclib_replace } !lib_bundle:unix {