guix-patches
[Top][All Lists]
Advanced

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

[bug#42390] [PATCH] gnu: Add python-mamba.


From: Ludovic Courtès
Subject: [bug#42390] [PATCH] gnu: Add python-mamba.
Date: Wed, 22 Jul 2020 12:38:05 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/26.3 (gnu/linux)

Howdy,

(+Cc: Hartmut.)

Tanguy Le Carrour <tanguy@bioneland.org> skribis:

> Le 07/21, Ludovic Courtès a écrit :
>> Tanguy Le Carrour <tanguy@bioneland.org> skribis:
>> > * gnu/packages/python-xyz.scm (python-mamba): New variable.
>> […]
>> 
>> There’s a lint warning though I don’t know if it’s applicable:
>> 
>> --8<---------------cut here---------------start------------->8---
>> gnu/packages/python-xyz.scm:14542:2: python-mamba@0.11.0: 'python-coverage' 
>> should probably be a native input
>> --8<---------------cut here---------------end--------------->8---
>> 
>> If ‘python-coverage’ is only used at build time, then it should
>> definitely be a native input.
>> 
>> WDYT?
>
> I saw the warning (should I have mentioned it in the bug report?),
> but `python-coverage` is definitively a run time dependency.
> If I remove it, I get:
>
> ```
> pkg_resources.DistributionNotFound: The 'coverage' distribution was not found 
> and is required by mamba
> ```

Ah, I see.

> Is there a way to know why the linter is reporting it?!

I don’t know, it comes from commit
e442246a2f5f73b2484adb340b53d3a0018636b1 by Hartmut (Cc’d.)

Hartmut, it looks like there are cases like this where python-coverage
is used as a library and thus should not be in ‘native-inputs’.  Should
we remove it from the list?

Thanks,
Ludo’.





reply via email to

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