python3Packages.pygeoip: drop

[Sigmanificient]

Description of changes

• python3Packages: get rid of uses of nose #326513

Things done

• Built on platform(s)
  • x86_64-linux
  • aarch64-linux
  • x86_64-darwin
  • aarch64-darwin
• For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
  • sandbox = relaxed
  • sandbox = true
• Tested, as applicable:
  • NixOS test(s) (look inside nixos/tests)
  • and/or package tests
  • or, for functions and "core" functionality, tests in lib/tests or pkgs/test
  • made sure NixOS tests are linked to the relevant packages
• Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
• Tested basic functionality of all binary files (usually in ./result/bin/)
• 24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)
  • (Package updates) Added a release notes entry if the change is major or breaking
  • (Module updates) Added a release notes entry if the change is significant
  • (Module addition) Added a release notes entry if adding a new NixOS module
• Fits CONTRIBUTING.md.

---

Add a 👍 reaction to pull requests you find important.

[Sigmanificient]

@emilazy @natsukium Having trouble with infinite recursion (sopel -> sopel-help -> sopel -> ...)

Help would be welcome

[Sigmanificient]

@mog (sopel maintainer)

[natsukium]

Please remove sopel-help from sopel's dependencies, since sopel doesn't directly depend on sopel-help.
https://github.com/search?q=repo%3Asopel-irc%2Fsopel+sopel_help&type=code

[Sigmanificient]

Please remove sopel-help from sopel's dependencies, since sopel doesn't directly depend on sopel-help. github.com/search?q=repo%3Asopel-irc%2Fsopel+sopel_help&type=code

I added it because of:

Executing pythonRuntimeDepsCheck
Checking runtime dependencies for sopel-8.0.0-py3-none-any.whl
  - sopel-help not installed

[emilazy]

This seems reasonable but it’d probably be best to split out the substantive sopel changes into another PR, as they don’t seem necessary to drop this package and I’m not entirely sure how we should handle the circular dependency here.

[natsukium]

Result of nixpkgs-review pr 330509 at 5f4744f0 run on x86_64-linux 1

2 packages failed to build:

• python311Packages.sopel-help
• python312Packages.sopel-help

3 packages built successfully:

• nixpkgs-manual
• python311Packages.sopel
• python312Packages.sopel

[natsukium]

I think the current hack is fine for the circular dependency, but sopel-help tests don't pass.

[emilazy]

I was thinking we’d probably want a standard sopel installation to include sopel-help, since that seems to be the intention of the upstream, but I don’t know how that would be best accomplished (a wrapper package?). In any case it should be handled in another PR, I think.

[natsukium]

We are working on separating build and runtime inputs for python packages in the future, and it will be resolved then. #272178

[Sigmanificient]

Just removed sopel-help since is it no longer require, will open pr if I get it to build

[emilazy]

@ofborg build python3Packages.sopel

[SuperSandro2000]

@ofborg build python312Packages.sopel