all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: ludo@gnu.org (Ludovic Courtès)
To: Marius Bakke <mbakke@fastmail.com>
Cc: 32155@debbugs.gnu.org
Subject: [bug#32155] [PATCH 0/4] sfml updates
Date: Tue, 17 Jul 2018 23:16:03 +0200	[thread overview]
Message-ID: <87zhyp7ecs.fsf@gnu.org> (raw)
In-Reply-To: <20180714144121.30906-1-mbakke@fastmail.com> (Marius Bakke's message of "Sat, 14 Jul 2018 16:41:21 +0200")

Hello Marius,

Marius Bakke <mbakke@fastmail.com> skribis:

> This series updates sfml to the latest release, and adds some of the
> "stb" <https://github.com/nothings/stb> libraries in order to unbundle
> all dependencies.
>
> Marius Bakke (4):
>   gnu: sfml: Update to 2.5.0.
>   gnu: Add stb-image.
>   gnu: Add stb-image-write.
>   gnu: sfml: Remove all bundled dependencies.

It all LGTM!

> +(define (make-stb-header-package name version description)
> +  (package
> +    (inherit stb)
> +    (name name)
> +    (version version)
> +    (source #f)
> +    (inputs `(("stb" ,stb)))
> +    (build-system trivial-build-system)
> +    (arguments
> +     `(#:modules ((guix build utils))
> +       #:builder (begin
> +                   (use-modules (guix build utils))
> +                   (let ((stb (assoc-ref %build-inputs "stb"))
> +                         (lib (string-join (string-split ,name #\-) "_"))
> +                         (out (assoc-ref %outputs "out")))
> +                     (install-file (string-append stb "/" lib ".h")
> +                                   (string-append out "/include"))
> +                     #t))))
> +    (description description)))
> +
> +(define-public stb-image
> +  (make-stb-header-package
> +   "stb-image" "2.19"
> +   "stb-image is a small and self-contained library for image loading or
> +decoding from file or memory.  A variety of formats are supported."))

Not sure if it really matters here, but note that the description here
won’t be subject to translation.

To make it translatable, it would have to be embedded in a (description
"…") form.  I address that with phony macros as in
‘define-word-list-dictionary’ in aspell.scm.

Thanks,
Ludo’.

  parent reply	other threads:[~2018-07-17 21:17 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-07-14 14:41 [bug#32155] [PATCH 0/4] sfml updates Marius Bakke
2018-07-14 14:43 ` [bug#32155] [PATCH 1/4] gnu: sfml: Update to 2.5.0 Marius Bakke
2018-07-14 14:43   ` [bug#32155] [PATCH 2/4] gnu: Add stb-image Marius Bakke
2018-07-14 14:43   ` [bug#32155] [PATCH 3/4] gnu: Add stb-image-write Marius Bakke
2018-07-14 14:43   ` [bug#32155] [PATCH 4/4] gnu: sfml: Remove all bundled dependencies Marius Bakke
2018-07-14 14:47     ` Marius Bakke
2018-07-17 21:16 ` Ludovic Courtès [this message]
2018-07-17 23:18   ` bug#32155: [PATCH 0/4] sfml updates Marius Bakke

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

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

  git send-email \
    --in-reply-to=87zhyp7ecs.fsf@gnu.org \
    --to=ludo@gnu.org \
    --cc=32155@debbugs.gnu.org \
    --cc=mbakke@fastmail.com \
    /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 external index

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

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.