python3Packages.click: 8.1.8 -> 8.2.1

[dotlambda]

The previous PR was reverted because it breaks click-repl.
This PR circumvents that problem by removing from python3Packages.celery the functionality that depends on click-repl.
In order still give access to the affected AMQP REPL, we introduce a top-level attribute celery which uses an overridden Python package set. This however violates #421201.
I also don't like the number of packages that have to be rebuilt just to build celery. Maybe we shouldn't add the top-level attribute and let our users override click if they need the AMQP REPL?

fixes #438287
cc @MultisampledNight @caniko @RaitoBezarius @frogamic

This breaks at least:

• python3Packages.greynoise @mbalatsko
• python3Packages.typer @winpat
• todoman test_sane_config fails with Click 8.3.0: TypeError: 'Sentinel' object is not iterable pimutils/todoman#596 @leenaars @antonmosich

Things done

• Built on platform:
  • x86_64-linux
  • aarch64-linux
  • x86_64-darwin
  • aarch64-darwin
• Tested, as applicable:
  • NixOS tests in nixos/tests.
  • Package tests at passthru.tests.
  • Tests in lib/tests or pkgs/test for functions and "core" functionality.
• Ran nixpkgs-review on this PR. See nixpkgs-review usage.
• Tested basic functionality of all binary files, usually in ./result/bin/.
• Nixpkgs Release Notes
  • Package update: when the change is major or breaking.
• NixOS Release Notes
  • Module addition: when adding a new NixOS module.
  • Module update: when the change is significant.
• Fits CONTRIBUTING.md, pkgs/README.md, maintainers/README.md and other READMEs.

---

Add a 👍 reaction to pull requests you find important.

[MultisampledNight]

@caniko would you mind taking the pyproject-nix suggestion to a more sync channel like #python:nixos.org on matrix? not sure how this pull would benefit from the discussion this would spark

[dotlambda]

I also don't like the number of packages that have to be rebuilt just to build celery. Maybe we shouldn't add the top-level attribute and let our users override click if they need the AMQP REPL?

I went for that. We can always add back 44c514f if we need a top-level celery.

[antonmosich]

todoman also encounters the error from test_sane_config when starting the resulting executable with no arguments/subcommand (with my config at least). I don't have the time to fix this right now though, sorry.

[dotlambda]

todoman also encounters the error from test_sane_config when starting the resulting executable with no arguments/subcommand (with my config at least). I don't have the time to fix this right now though, sorry.

In that case let's stick with version 8.2.1 for now.

[dotlambda]

Weirdly, greynoise's tests fail with click 8.2.1 but not 8.3.0.
Honestly, I don't care and I'll merge this anyway if everything else looks good enough.

[dotlambda]

todoman also encounters the error from test_sane_config when starting the resulting executable with no arguments/subcommand (with my config at least). I don't have the time to fix this right now though, sorry.

In that case let's stick with version 8.2.1 for now.

8.3.1 should fix that issue: #454434