Hello Jeroen,

I am aware of the issue related to the CI failure. Tests runs fine on my pc, 
also the nextdns module is imported without any errors when tested locally.  
The error is most likely due to the absence of tenacity in the host itself. Let 
me know your thoughts on it.

Thanks,

Sourav


Oct 11, 2024, 14:02 by j...@debian.org:

> Control: tags -1 moreinfo
>
> On Fri, 11 Oct 2024 10:03:44 +0530
> Sourav Bhattacharya <souravbhattacha...@tutamail.com> wrote:
>
>> I am looking for a sponsor for my package python-nextdns:
>>
>
> hi again Sourav,
>
> nice to see the improvements made since yesterday's review. Couple of
> issues remaining before I can upload the package:
>
> * control: most teams require team packages to sit in their team
>  repository. Have you contacted the LEGO team?
>
> * rules: pybuild name -if set at all- should be whatever is used to
>  import the module in Python, i.e. "nextdns".
>
> * CI failure (autopkgtest [1]) on the salsa repo indicates a serious
>  problem with the binary package (one or more missing dependencies).
>  Please make a habit of inspecting CI results prior to filing a
>  sponsorship request.
>
> That missing dependency is probably caused by an upstream mistake in
> setup.py, which only mentions the aiohttp module in "install_requires"
> while requirements.txt also lists the tenacity module (as it should,
> given the latter gets imported in nextdns/__init__.py:12).
>
> Either add a patch for setup.py or manually add the missing
> dependency to the binary package for now. Please also file a bug
> about this with the upstream developer(s).
>
>
> [1]https://salsa.debian.org/sxctic-54l/python-nextdns/-/jobs/6412294
>

Reply via email to