mbox series

[bug#67537,0/4] Update weirdware packages.

Message ID cover.1701293236.git.sarg@sarg.org.ru
Headers show
Series Update weirdware packages. | expand

Message

Sergey Trofimov Nov. 29, 2023, 9:31 p.m. UTC 
Hello guix,

I've noticed that recently added weirdware emacs packages use `inputs` to
declare dependencies. I think `propagated-inputs` make more sense here as
these are required in runtime.

Sergey Trofimov (4):
  gnu: emacs-dnt: Propagate inputs.
  gnu: emacs-debase: Remove unused inputs.
  gnu: emacs-discomfort: Propagate inputs.
  gnu: emacs-hyperspace: Propagated inputs.

 gnu/packages/emacs-xyz.scm | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)


base-commit: bdbb9dc27a590b08651d058f06a42caa26e04abb

Comments

Liliana Marie Prikler Nov. 29, 2023, 9:55 p.m. UTC | #1 
Am Mittwoch, dem 29.11.2023 um 22:31 +0100 schrieb Sergey Trofimov:
> Hello guix,
> 
> I've noticed that recently added weirdware emacs packages use
> `inputs` to
> declare dependencies. I think `propagated-inputs` make more sense
> here as
> these are required in runtime.
> 
> Sergey Trofimov (4):
>   gnu: emacs-dnt: Propagate inputs.
>   gnu: emacs-debase: Remove unused inputs.
>   gnu: emacs-discomfort: Propagate inputs.
>   gnu: emacs-hyperspace: Propagated inputs.
Nice catch.  LGTM.
Nicolas Goaziou Jan. 4, 2024, 11:38 a.m. UTC | #2 
Hello,

Liliana Marie Prikler <liliana.prikler@gmail.com> writes:

> Am Mittwoch, dem 29.11.2023 um 22:31 +0100 schrieb Sergey Trofimov:
>> Hello guix,
>> 
>> I've noticed that recently added weirdware emacs packages use
>> `inputs` to
>> declare dependencies. I think `propagated-inputs` make more sense
>> here as
>> these are required in runtime.
>> 
>> Sergey Trofimov (4):
>>   gnu: emacs-dnt: Propagate inputs.
>>   gnu: emacs-debase: Remove unused inputs.
>>   gnu: emacs-discomfort: Propagate inputs.
>>   gnu: emacs-hyperspace: Propagated inputs.
> Nice catch.  LGTM.

Applied. Thanks.

Regards,