* [bug#50231] [PATCH] gnu: emacs-desktop-environment: Include runtime dependencies
@ 2021-08-27 18:36 Reily Siegel
2021-09-06 9:17 ` Nicolas Goaziou
0 siblings, 1 reply; 6+ messages in thread
From: Reily Siegel @ 2021-08-27 18:36 UTC (permalink / raw)
To: 50231
* gnu/packages/emacs-xyz.scm (emacs-desktop-environment): Include runtime
dependencies.
---
gnu/packages/emacs-xyz.scm | 9 +++++++++
1 file changed, 9 insertions(+)
diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index 7672168a0e..ed18692b29 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -160,6 +160,7 @@
#:use-module (gnu packages networking)
#:use-module (gnu packages python)
#:use-module (gnu packages python-xyz)
+ #:use-module (gnu packages suckless)
#:use-module (gnu packages telephony)
#:use-module (gnu packages terminals)
#:use-module (gnu packages tex)
@@ -20259,6 +20260,14 @@ autosuggestions with:
(base32
"0s33jncjg8c2xlr90qjk1ishwmwxm9yg0z6n3kzilawcilpxidsh"))))
(build-system emacs-build-system)
+ (propagated-inputs
+ `(("alsa-utils" ,alsa-utils)
+ ("brightnessctl" ,brightnessctl)
+ ("scrot" ,scrot)
+ ("slock" ,slock)
+ ("upower" ,upower)
+ ("tlp" ,tlp)
+ ("playerctl" ,playerctl)))
(home-page "https://gitlab.petton.fr/DamienCassou/desktop-environment")
(synopsis "Control your GNU/Linux desktop environment from Emacs")
(description
--
2.32.0
^ permalink raw reply related [flat|nested] 6+ messages in thread* [bug#50231] [PATCH] gnu: emacs-desktop-environment: Include runtime dependencies
2021-08-27 18:36 [bug#50231] [PATCH] gnu: emacs-desktop-environment: Include runtime dependencies Reily Siegel
@ 2021-09-06 9:17 ` Nicolas Goaziou
2021-09-07 19:19 ` Reily Siegel
0 siblings, 1 reply; 6+ messages in thread
From: Nicolas Goaziou @ 2021-09-06 9:17 UTC (permalink / raw)
To: Reily Siegel; +Cc: 50231
Hello,
Reily Siegel <mail@reilysiegel.com> writes:
> * gnu/packages/emacs-xyz.scm (emacs-desktop-environment): Include runtime
> dependencies.
Thank you.
> + (propagated-inputs
> + `(("alsa-utils" ,alsa-utils)
> + ("brightnessctl" ,brightnessctl)
> + ("scrot" ,scrot)
> + ("slock" ,slock)
> + ("upower" ,upower)
> + ("tlp" ,tlp)
> + ("playerctl" ,playerctl)))
I don't think we should propagate all of them. It's up to the user to
provide its own tools, which are not necessarily those.
WDYT?
Regards,
--
Nicolas Goaziou
^ permalink raw reply [flat|nested] 6+ messages in thread* [bug#50231] [PATCH] gnu: emacs-desktop-environment: Include runtime dependencies
2021-09-06 9:17 ` Nicolas Goaziou
@ 2021-09-07 19:19 ` Reily Siegel
2021-09-07 20:15 ` Nicolas Goaziou
0 siblings, 1 reply; 6+ messages in thread
From: Reily Siegel @ 2021-09-07 19:19 UTC (permalink / raw)
To: Nicolas Goaziou; +Cc: 50231
Hello,
Nicolas Goaziou <mail@nicolasgoaziou.fr> writes:
> I don't think we should propagate all of them. It's up to the user to
> provide its own tools, which are not necessarily those.
The emacs package can be configured to use other tools. However, unless
the package is specifically configured otherwise, it will not work when
installed. Without either these propagated inputs or special
configuration, this package cannot perform any of its functions. Perhaps
these could be included in a separate output?
Reily Siegel
^ permalink raw reply [flat|nested] 6+ messages in thread
* [bug#50231] [PATCH] gnu: emacs-desktop-environment: Include runtime dependencies
2021-09-07 19:19 ` Reily Siegel
@ 2021-09-07 20:15 ` Nicolas Goaziou
2021-09-08 9:33 ` zimoun
2021-09-24 11:56 ` Ludovic Courtès
0 siblings, 2 replies; 6+ messages in thread
From: Nicolas Goaziou @ 2021-09-07 20:15 UTC (permalink / raw)
To: Reily Siegel; +Cc: 50231
Hello,
Reily Siegel <mail@reilysiegel.com> writes:
> The emacs package can be configured to use other tools. However, unless
> the package is specifically configured otherwise, it will not work when
> installed. Without either these propagated inputs or special
> configuration, this package cannot perform any of its functions.
I know, but this Emacs library is not supposed to provide the features
itself. I assume you use it on top of installed executables, as
explained in the package description:
The package depends on the availability of shell commands to do the
hard work for us.
Pulling all of them from an Emacs package sounds backward.
> Perhaps these could be included in a separate output?
Outputs are distinct part of the package output. Here, propagated
executables are not part of the Emacs library, so the suggestion does
not hold.
Maybe Guix is missing a "suggested depedencies" flag, somehow.
Anyhow, I think the status quo is fine. WDY (and others) T?
Regards,
--
Nicolas Goaziou
^ permalink raw reply [flat|nested] 6+ messages in thread
* [bug#50231] [PATCH] gnu: emacs-desktop-environment: Include runtime dependencies
2021-09-07 20:15 ` Nicolas Goaziou
@ 2021-09-08 9:33 ` zimoun
2021-09-24 11:56 ` Ludovic Courtès
1 sibling, 0 replies; 6+ messages in thread
From: zimoun @ 2021-09-08 9:33 UTC (permalink / raw)
To: Nicolas Goaziou, Reily Siegel; +Cc: 50231
Hi,
On Tue, 07 Sep 2021 at 22:15, Nicolas Goaziou <mail@nicolasgoaziou.fr> wrote:
>> The emacs package can be configured to use other tools. However, unless
>> the package is specifically configured otherwise, it will not work when
>> installed. Without either these propagated inputs or special
>> configuration, this package cannot perform any of its functions.
Indeed. However, “guix install emacs-foo” does not install the package
’emacs’, i.e., ’emacs-foo’ cannot perform any of its functions. ;-)
> I know, but this Emacs library is not supposed to provide the features
> itself. I assume you use it on top of installed executables, as
> explained in the package description:
>
> The package depends on the availability of shell commands to do the
> hard work for us.
>
> Pulling all of them from an Emacs package sounds backward.
On the other side, I remember discussions about ’git’ as inputs in
’emacs-magit’. Or ’the-silver-searcher’ for ’emacs-ag’. Maybe it is
worth to improve section about Emacs packages in the manual. Rough
draft:
«The external tools required by the package are not included as inputs,
except if they are required at compile or test time.»
Well, I am sure someone will find a better wording. ;-)
> Maybe Guix is missing a "suggested depedencies" flag, somehow.
Yeah, maybe.
One way should to add a ’properties’ as ’suggest’. Well, from my
experience with Debian, I am not convinced that this suggest field
really helps. Maybe for discovery.
The easiest way is to provide a sentence in the ’description’ field
about such.
Cheers,
simon
^ permalink raw reply [flat|nested] 6+ messages in thread
* [bug#50231] [PATCH] gnu: emacs-desktop-environment: Include runtime dependencies
2021-09-07 20:15 ` Nicolas Goaziou
2021-09-08 9:33 ` zimoun
@ 2021-09-24 11:56 ` Ludovic Courtès
1 sibling, 0 replies; 6+ messages in thread
From: Ludovic Courtès @ 2021-09-24 11:56 UTC (permalink / raw)
To: Nicolas Goaziou; +Cc: Reily Siegel, 50231
Hi,
Nicolas Goaziou <mail@nicolasgoaziou.fr> skribis:
> Reily Siegel <mail@reilysiegel.com> writes:
>
>> The emacs package can be configured to use other tools. However, unless
>> the package is specifically configured otherwise, it will not work when
>> installed. Without either these propagated inputs or special
>> configuration, this package cannot perform any of its functions.
>
> I know, but this Emacs library is not supposed to provide the features
> itself. I assume you use it on top of installed executables, as
> explained in the package description:
>
> The package depends on the availability of shell commands to do the
> hard work for us.
>
> Pulling all of them from an Emacs package sounds backward.
How about:
1. Providing only those tools for which there’s no choice, like
‘alsa-utils’ and ‘brightnessclt’.
2. Patching file names instead of propagating.
Thoughts?
Ludo’.
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2021-09-24 11:57 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-08-27 18:36 [bug#50231] [PATCH] gnu: emacs-desktop-environment: Include runtime dependencies Reily Siegel
2021-09-06 9:17 ` Nicolas Goaziou
2021-09-07 19:19 ` Reily Siegel
2021-09-07 20:15 ` Nicolas Goaziou
2021-09-08 9:33 ` zimoun
2021-09-24 11:56 ` Ludovic Courtès
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).