Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revert "python3Packages.pika: add gevent to checkInputs" #122317

Closed
wants to merge 1 commit into from

Conversation

dotlambda
Copy link
Member

Reverts #122229
Earlier PRs have priority: #122170

@r-rmcgibbo
Copy link

r-rmcgibbo commented May 9, 2021

Result of nixpkgs-review pr 122317 at 304f146 run on aarch64-linux 1

4 packages failed to build:
1 suggestion:
  • warning: maintainers-missing

    Package does not have a maintainer. Consider adding yourself?

    Near pkgs/development/python-modules/pika/default.nix:21:3:

       |
    21 |   meta = with lib; {
       |   ^
    

Note that build failures may predate this PR, and could be nondeterministic or hardware dependent.
Please exercise your independent judgement. Does something look off? Please file an issue or reach out on IRC.


Result of nixpkgs-review pr 122317 at 304f146 run on x86_64-linux 1

4 packages failed to build:
1 suggestion:
  • warning: maintainers-missing

    Package does not have a maintainer. Consider adding yourself?

    Near pkgs/development/python-modules/pika/default.nix:21:3:

       |
    21 |   meta = with lib; {
       |   ^
    

Note that build failures may predate this PR, and could be nondeterministic or hardware dependent.
Please exercise your independent judgement. Does something look off? Please file an issue or reach out on IRC.

@prusnak
Copy link
Member

prusnak commented May 15, 2021

Closing this one in favor of #122170 which has been rebased on top of the current master.

Merging this one will cause yet another hassle in #122170

@prusnak prusnak closed this May 15, 2021
@SuperSandro2000 SuperSandro2000 deleted the revert-122229-pika-21.05 branch June 13, 2021 17:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants