From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:43403) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1e2beo-0007kk-EE for guix-patches@gnu.org; Thu, 12 Oct 2017 07:28:07 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1e2bel-0005VW-5q for guix-patches@gnu.org; Thu, 12 Oct 2017 07:28:06 -0400 Received: from debbugs.gnu.org ([208.118.235.43]:54482) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1e2bel-0005VQ-1x for guix-patches@gnu.org; Thu, 12 Oct 2017 07:28:03 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1e2bej-0002P4-UJ for guix-patches@gnu.org; Thu, 12 Oct 2017 07:28:01 -0400 Subject: [bug#28719] [PATCH 5/6] gnu: qscintilla: Adjust installation directory layout. Resent-Message-ID: From: iyzsong@member.fsf.org (=?UTF-8?Q?=E5=AE=8B=E6=96=87=E6=AD=A6?=) References: <20171006120214.29511-1-h.goebel@crazy-compilers.com> <20171006120214.29511-5-h.goebel@crazy-compilers.com> <87efqb1cwn.fsf@member.fsf.org> <41f1882d-41a5-4992-bb7d-d0b5481c6ca5@crazy-compilers.com> Date: Thu, 12 Oct 2017 19:27:22 +0800 In-Reply-To: <41f1882d-41a5-4992-bb7d-d0b5481c6ca5@crazy-compilers.com> (Hartmut Goebel's message of "Tue, 10 Oct 2017 17:39:03 +0200") Message-ID: <87y3ogip2d.fsf@member.fsf.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-patches-bounces+kyle=kyleam.com@gnu.org Sender: "Guix-patches" To: Hartmut Goebel Cc: 28719@debbugs.gnu.org Hartmut Goebel writes: > Am 10.10.2017 um 13:05 schrieb =E5=AE=8B=E6=96=87=E6=AD=A6: >> Idelly, With the previous "Fix install paths" patch, this substitute for >> qscintilla.pro could be removed. Can you check it? Thanks! > > Found time to test it earlier :-) > > I tested it and this substitute is still needed. The reason AFAICT is > that QScintilla - is referring to variables defined in qmake, while the > other packages are using some definition-file in qtbase. The variables > are set as follows: > > $ /gnu/store/=E2=80=A6-qtbase-5.9.1/bin/qmake -query > QT_SYSROOT: > QT_INSTALL_PREFIX:/gnu/store/=E2=80=A6-qtbase-5.9.1 > QT_INSTALL_ARCHDATA:/gnu/store/=E2=80=A6-qtbase-5.9.1/lib/qt5 > =E2=80=A6 > Oh, my bad. I forget that those variables will only be correct when qmake is using the customized qt.conf (like qtsvg), you can make qscintilla inherit from qtsvg to use its 'configure-qmake' phase.