unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
* [PATCH] gnu: qtbase: Add search path specification for 'QMAKEPATH'.
@ 2016-08-14  6:14 宋文武
  2016-08-19  9:17 ` Andreas Enge
  0 siblings, 1 reply; 2+ messages in thread
From: 宋文武 @ 2016-08-14  6:14 UTC (permalink / raw)
  To: guix-devel; +Cc: 宋文武

* gnu/packages/qt.scm (qtbase)[native-search-paths]: New field.
[arguments]: Add phase to patch 'qt_config.prf'.
---
 gnu/packages/qt.scm | 31 ++++++++++++++++++++++++++++++-
 1 file changed, 30 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/qt.scm b/gnu/packages/qt.scm
index a35c337..a12c526 100644
--- a/gnu/packages/qt.scm
+++ b/gnu/packages/qt.scm
@@ -445,7 +445,36 @@ developers using C++ or QML, a CSS & JavaScript like language.")
                        "-no-avx"
                        "-no-avx2"
                        "-no-mips_dsp"
-                       "-no-mips_dspr2"))))))))
+                       "-no-mips_dspr2")))))
+         (add-after 'install 'patch-qt_config.prf
+           (lambda* (#:key outputs #:allow-other-keys)
+             (let* ((out (assoc-ref outputs "out"))
+                    (qt_config.prf (string-append
+                                    out "/mkspecs/features/qt_config.prf")))
+               ;; For each Qt module, let `qmake' uses search paths in the
+               ;; module directory instead of all in QT_INSTALL_PREFIX.
+               (substitute* qt_config.prf
+                 (("\\$\\$\\[QT_INSTALL_HEADERS\\]")
+                  "$$replace(dir, mkspecs/modules, include)")
+                 (("\\$\\$\\[QT_INSTALL_LIBS\\]")
+                  "$$replace(dir, mkspecs/modules, lib)")
+                 (("\\$\\$\\[QT_HOST_LIBS\\]")
+                  "$$replace(dir, mkspecs/modules, lib)")
+                 (("\\$\\$\\[QT_INSTALL_PLUGINS\\]")
+                  "$$replace(dir, mkspecs/modules, plugins)")
+                 (("\\$\\$\\[QT_INSTALL_LIBEXECS\\]")
+                  "$$replace(dir, mkspecs/modules, libexec)")
+                 (("\\$\\$\\[QT_INSTALL_BINS\\]")
+                  "$$replace(dir, mkspecs/modules, bin)")
+                 (("\\$\\$\\[QT_INSTALL_IMPORTS\\]")
+                  "$$replace(dir, mkspecs/modules, imports)")
+                 (("\\$\\$\\[QT_INSTALL_QML\\]")
+                  "$$replace(dir, mkspecs/modules, qml)"))
+               #t))))))
+    (native-search-paths
+     (list (search-path-specification
+            (variable "QMAKEPATH")
+            (files '("")))))
     (home-page "https://www.qt.io/")
     (synopsis "Cross-platform GUI library")
     (description "Qt is a cross-platform application and UI framework for
-- 
2.8.4

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] gnu: qtbase: Add search path specification for 'QMAKEPATH'.
  2016-08-14  6:14 [PATCH] gnu: qtbase: Add search path specification for 'QMAKEPATH' 宋文武
@ 2016-08-19  9:17 ` Andreas Enge
  0 siblings, 0 replies; 2+ messages in thread
From: Andreas Enge @ 2016-08-19  9:17 UTC (permalink / raw)
  To: 宋文武; +Cc: guix-devel

Hello 宋文武,

thanks a lot for solving this blocking problem!

On Sun, Aug 14, 2016 at 02:14:11PM +0800, 宋文武 wrote:
> +               ;; For each Qt module, let `qmake' uses search paths in the
> +               ;; module directory instead of all in QT_INSTALL_PREFIX.
> +               (substitute* qt_config.prf
> +                 (("\\$\\$\\[QT_INSTALL_HEADERS\\]")
> +                  "$$replace(dir, mkspecs/modules, include)")
> +                 (("\\$\\$\\[QT_INSTALL_LIBS\\]")
> +                  "$$replace(dir, mkspecs/modules, lib)")
> +                 (("\\$\\$\\[QT_HOST_LIBS\\]")
> +                  "$$replace(dir, mkspecs/modules, lib)")
> +                 (("\\$\\$\\[QT_INSTALL_PLUGINS\\]")
> +                  "$$replace(dir, mkspecs/modules, plugins)")
> +                 (("\\$\\$\\[QT_INSTALL_LIBEXECS\\]")
> +                  "$$replace(dir, mkspecs/modules, libexec)")
> +                 (("\\$\\$\\[QT_INSTALL_BINS\\]")
> +                  "$$replace(dir, mkspecs/modules, bin)")
> +                 (("\\$\\$\\[QT_INSTALL_IMPORTS\\]")
> +                  "$$replace(dir, mkspecs/modules, imports)")
> +                 (("\\$\\$\\[QT_INSTALL_QML\\]")
> +                  "$$replace(dir, mkspecs/modules, qml)"))

What do you think, would it make sense to file a bug upstream to suggest
the modifications above?

Andreas

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2016-08-19  9:17 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-08-14  6:14 [PATCH] gnu: qtbase: Add search path specification for 'QMAKEPATH' 宋文武
2016-08-19  9:17 ` Andreas Enge

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).