[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[bug#27791] [PATCH] gnu: Add passmenu

From: Ludovic Courtès
Subject: [bug#27791] [PATCH] gnu: Add passmenu
Date: Mon, 16 Oct 2017 15:22:47 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/25.3 (gnu/linux)

Hi Jelle,

Is anything holding this back?

TIA!  :-)


Marius Bakke <address@hidden> skribis:

> Hi Jelle,
> Jelle Licht <address@hidden> writes:
>> Hello guix,
>> Attached is a patch to include passmenu, a dmenu interface to the pass
>> password store.
>> I was not quite sure how to structure this patch, as it basically installs
>> and wraps a shell script from the `password-store' sources. We could
>> instead include it as a separate output of our `password-store' package,
>> but I already had it like this in my GUIX_PACKAGE_PATH and I was not even
>> sure if that approach was in general preferable.
> I don't think wrapping it with dmenu in PATH is necessary. Users of this
> script are expected to have dmenu from before, and may want to use
> another implementation (e.g. rofi), another version, etc.
> Can you try to simply add a phase to the normal password-store package
> that copies this file to out/bin? We can probably avoid the wrapper too
> by giving it the full path to `xdotool`, e.g.:
> (substitute "passmenu"
>   (("xdotool") (string-append (assoc-ref inputs "xdotool")
>                               "/bin/xdotool")))
> Adding 'xdotool' adds ~8MiB to the password-store closure size, so I
> don't think we need a separate output either.
> Thanks!

reply via email to

[Prev in Thread] Current Thread [Next in Thread]