GNU bug report logs - #25991
[PATCH] gnu: surf: Add 'dmenu' and 'xprop' to 'inputs'.

Previous Next

Package: guix-patches;

Reported by: Mekeor Melire <mekeor.melire <at> gmail.com>

Date: Mon, 6 Mar 2017 01:56:02 UTC

Severity: normal

Tags: fixed, patch

Done: ludo <at> gnu.org (Ludovic Courtès)

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Kei Kebreau <kei <at> openmailbox.org>
To: Danny Milosavljevic <dannym <at> scratchpost.org>
Cc: Mekeor Melire <mekeor.melire <at> gmail.com>, 25991 <at> debbugs.gnu.org
Subject: bug#25991: [PATCH] gnu: surf: Add 'dmenu' and 'xprop' to 'inputs'.
Date: Sun, 05 Mar 2017 22:23:48 -0500
[Message part 1 (text/plain, inline)]
Danny Milosavljevic <dannym <at> scratchpost.org> writes:

> Hi,
>
> On Mon, 06 Mar 2017 02:55:01 +0100
> Mekeor Melire <mekeor.melire <at> gmail.com> wrote:
>
>> This is meant to fix bug#25759 “Should 'surf' depend on 'xprop' and
>> 'dmenu'?“:
>> 
>> https://lists.gnu.org/archive/html/bug-guix/2017-02/msg00054.html
>> 
>> I'm not sure if we really want this patch to be merged because there was
>> not much of a discussion on this proposal on the mailing list.
>
> It's not finished. http://git.suckless.org/surf/tree/config.def.h#n38
> invokes xprop and dmenu - and these places would have to be
> substituted by store references, like:
>
> (substitute* "tree/config.def.h"
>   (("xprop") (string-append (assoc-ref inputs "xprop") "/bin/xprop"))
>   (("dmenu") (string-append (assoc-ref inputs "dmenu") "/bin/dmenu")))
>
> I'm not sure whether Guix will retain the store references on the next
> GC, though.

I'm sure Guix does, considering we use this trick for all sorts of
packages, including important ones like shadow.
[signature.asc (application/pgp-signature, inline)]

This bug report was last modified 8 years and 170 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.