all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Tobias Geerinckx-Rice <me@tobias.gr>
To: ludo@gnu.org
Cc: 29653-done@debbugs.gnu.org
Subject: bug#29653: [PATCH 2/2] gnu: Add fbcat.
Date: Tue, 12 Dec 2017 05:45:03 +0100	[thread overview]
Message-ID: <85d38495-1ac7-f0d7-c6da-a8a1aa7cee4e@tobias.gr> (raw)
In-Reply-To: <874loxxmwa.fsf@gnu.org>

Ludo',

Ludovic Courtès wrote on 11/12/17 at 17:14:
> Tobias Geerinckx-Rice <me@tobias.gr> skribis:
> 
>> * gnu/packages/linux.scm (fbcat): New public variable.
> 
> [...]
> 
>> +    (inputs
>> +     ;; The ‘fbgrab’ wrapper can use one of several PPM-to-PNG converters.  We
>> +     ;; choose netpbm simply because it's the smallest.  It still adds ~94 MiB
>> +     ;; to an otherwise tiny package, so we put ‘fbgrab’ in its own output.
>> +     `(("pnmtopng" ,netpbm)))
>> +    (outputs (list "out" "fbgrab"))
> 
> That’s quite unusual but why not.

It is. IMO a nicer hack than pseudopackages like ‘fbcat-full’.

If we ever start more aggressively auto-splitting outputs and this would
interfere (‘fbcat:fbgrab:debug’), I'll rewrite it through my tears.

> Otherwise LGTM!

Thanks!

Pushed as c1c825844392d2db2414ec0c07dce0d882b6551d (snapscreenshot)
      and 8d8d227bbbe9bd1981bc63704c7f3fe029001f53 (fbcat).

Kind regards,

T G-R

      reply	other threads:[~2017-12-12  4:43 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-12-10 23:03 [bug#29651] [PATCH 0/2] fbcat and snapscreenshot Tobias Geerinckx-Rice
2017-12-10 23:03 ` [bug#29652] [PATCH 1/2] gnu: Add snapscreenshot Tobias Geerinckx-Rice
2017-12-11 16:12   ` Ludovic Courtès
2017-12-10 23:03 ` [bug#29653] [PATCH 2/2] gnu: Add fbcat Tobias Geerinckx-Rice
2017-12-11 16:14   ` Ludovic Courtès
2017-12-12  4:45     ` Tobias Geerinckx-Rice [this message]

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=85d38495-1ac7-f0d7-c6da-a8a1aa7cee4e@tobias.gr \
    --to=me@tobias.gr \
    --cc=29653-done@debbugs.gnu.org \
    --cc=ludo@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 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.