unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Ricardo Wurmus <rekado@elephly.net>
To: David Craven <david@craven.ch>
Cc: guix-devel@gnu.org
Subject: Re: [PATCH 03/31] build: Add wrap-qt-program.
Date: Sat, 27 Aug 2016 08:30:32 +0200	[thread overview]
Message-ID: <87poouoh9z.fsf@elephly.net> (raw)
In-Reply-To: <20160826215300.12913-4-david@craven.ch>


David Craven <david@craven.ch> writes:

> * guix/build/qt-utils.scm (wrap-qt-program): New file.

Please also mention the change to MODULES in the Makefile.am.  Other
than that it looks good to me.

~~ Ricardo

> ---
>  Makefile.am             |  1 +
>  guix/build/qt-utils.scm | 40 ++++++++++++++++++++++++++++++++++++++++
>  2 files changed, 41 insertions(+)
>  create mode 100644 guix/build/qt-utils.scm
>
> diff --git a/Makefile.am b/Makefile.am
> index 8bae85e..165dfe9 100644
> --- a/Makefile.am
> +++ b/Makefile.am
> @@ -107,6 +107,7 @@ MODULES =					\
>    guix/build/emacs-utils.scm			\
>    guix/build/graft.scm				\
>    guix/build/bournish.scm			\
> +  guix/build/qt-utils.scm			\
>    guix/search-paths.scm				\
>    guix/packages.scm				\
>    guix/import/utils.scm				\
> diff --git a/guix/build/qt-utils.scm b/guix/build/qt-utils.scm
> new file mode 100644
> index 0000000..48a3267
> --- /dev/null
> +++ b/guix/build/qt-utils.scm
> @@ -0,0 +1,40 @@
> +;;; GNU Guix --- Functional package management for GNU
> +;;; Copyright © 2016 David Craven <david@craven.ch>
> +;;;
> +;;; This file is part of GNU Guix.
> +;;;
> +;;; GNU Guix is free software; you can redistribute it and/or modify it
> +;;; under the terms of the GNU General Public License as published by
> +;;; the Free Software Foundation; either version 3 of the License, or (at
> +;;; your option) any later version.
> +;;;
> +;;; GNU Guix is distributed in the hope that it will be useful, but
> +;;; WITHOUT ANY WARRANTY; without even the implied warranty of
> +;;; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> +;;; GNU General Public License for more details.
> +;;;
> +;;; You should have received a copy of the GNU General Public License
> +;;; along with GNU Guix.  If not, see <http://www.gnu.org/licenses/>.
> +
> +(define-module (guix build qt-utils)
> +  #:use-module (guix build utils)
> +  #:export (wrap-qt-program))
> +
> +(define (wrap-qt-program out program)
> +  (define (suffix env-var path)
> +    (let ((env-val (getenv env-var)))
> +      (if env-val (string-append env-val ":" path) path)))
> +
> +  (let ((qml-path        (suffix "QML2_IMPORT_PATH"
> +                                 (string-append out "/qml")))
> +        (plugin-path     (suffix "QT_PLUGIN_PATH"
> +                                 (string-append out "/plugins")))
> +        (xdg-data-path   (suffix "XDG_DATA_DIRS"
> +                                 (string-append out "/share")))
> +        (xdg-config-path (suffix "XDG_CONFIG_DIRS"
> +                                 (string-append out "/etc/xdg"))))
> +    (wrap-program (string-append out "/bin/" program)
> +      `("QML2_IMPORT_PATH" = (,qml-path))
> +      `("QT_PLUGIN_PATH" = (,plugin-path))
> +      `("XDG_DATA_DIRS" = (,xdg-data-path))
> +      `("XDG_CONFIG_DIRS" = (,xdg-config-path)))))

  reply	other threads:[~2016-08-27  6:30 UTC|newest]

Thread overview: 47+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-08-26 21:52 [PATCH 00/31] KDE Tier 3 Packages David Craven
2016-08-26 21:52 ` [PATCH 01/31] gnu: qtbase: Add native search paths David Craven
2016-08-27  6:21   ` Ricardo Wurmus
2016-08-28  5:36     ` Efraim Flashner
2016-08-28 13:16       ` David Craven
2016-08-26 21:52 ` [PATCH 02/31] gnu: qt: Import licenses using #:prefix license: David Craven
2016-08-27  6:25   ` Ricardo Wurmus
2016-08-26 21:52 ` [PATCH 03/31] build: Add wrap-qt-program David Craven
2016-08-27  6:30   ` Ricardo Wurmus [this message]
2016-08-26 21:52 ` [PATCH 04/31] gnu: Add baloo David Craven
2016-08-27  6:32   ` Ricardo Wurmus
2016-08-29 11:21     ` David Craven
2016-08-30 20:43       ` David Craven
2016-08-26 21:52 ` [PATCH 05/31] gnu: Add kactivities David Craven
2016-08-27  6:38   ` Ricardo Wurmus
2016-08-28  9:38   ` Efraim Flashner
2016-08-28 10:14     ` David Craven
2016-08-26 21:52 ` [PATCH 06/31] gnu: Add kactivities-stats David Craven
2016-08-27  6:41   ` Ricardo Wurmus
2016-08-27  8:41     ` David Craven
2016-08-26 21:52 ` [PATCH 07/31] gnu: Add kbookmarks David Craven
2016-08-26 21:52 ` [PATCH 08/31] gnu: Add kcmutils David Craven
2016-08-26 21:52 ` [PATCH 09/31] gnu: Add kconfigwidgets David Craven
2016-08-26 21:52 ` [PATCH 10/31] gnu: Add kdeclarative David Craven
2016-08-26 21:52 ` [PATCH 11/31] gnu: Add kded David Craven
2016-08-26 21:52 ` [PATCH 12/31] gnu: Add kdesignerplugin David Craven
2016-08-26 21:52 ` [PATCH 13/31] gnu: Add kdesu David Craven
2016-08-26 21:52 ` [PATCH 14/31] gnu: Add kemoticons David Craven
2016-08-26 21:52 ` [PATCH 15/31] gnu: Add kglobalaccel David Craven
2016-08-26 21:52 ` [PATCH 16/31] gnu: Add kiconthemes David Craven
2016-08-26 21:52 ` [PATCH 17/31] gnu: Add kinit David Craven
2016-08-26 21:52 ` [PATCH 18/31] gnu: Add kio David Craven
2016-08-26 21:52 ` [PATCH 19/31] gnu: Add knewstuff David Craven
2016-08-26 21:52 ` [PATCH 20/31] gnu: Add knotifyconfig David Craven
2016-08-26 21:52 ` [PATCH 21/31] gnu: Add kparts David Craven
2016-08-26 21:52 ` [PATCH 22/31] gnu: Add kpeople David Craven
2016-08-26 21:52 ` [PATCH 23/31] gnu: Add krunner David Craven
2016-08-26 21:52 ` [PATCH 24/31] gnu: Add kservice David Craven
2016-08-26 21:52 ` [PATCH 25/31] gnu: Add ktexteditor David Craven
2016-08-26 21:52 ` [PATCH 26/31] gnu: Add ktextwidgets David Craven
2016-08-26 21:52 ` [PATCH 27/31] gnu: Add kwallet David Craven
2016-08-26 21:52 ` [PATCH 28/31] gnu: Add kxmlgui David Craven
2016-08-26 21:52 ` [PATCH 29/31] gnu: Add kxmlrpcclient David Craven
2016-08-26 21:52 ` [PATCH 30/31] gnu: Add plasma-framework David Craven
2016-08-26 21:53 ` [PATCH 31/31] gnu: Add kpmcore David Craven
2016-08-29  7:56 ` [PATCH 00/31] KDE Tier 3 Packages Hartmut Goebel
2016-08-31 20:29   ` Ludovic Courtès

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://guix.gnu.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87poouoh9z.fsf@elephly.net \
    --to=rekado@elephly.net \
    --cc=david@craven.ch \
    --cc=guix-devel@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).