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

[20.03] python2Packages.faulthandler: 3.1 -> 3.2 and fix build #83508

Merged
merged 3 commits into from Mar 27, 2020

Conversation

bhipple
Copy link
Contributor

@bhipple bhipple commented Mar 27, 2020

This backports the commits in #83466, which
"fixes" the broken faulthandler build on python2 by disabling the 1 failing
test, and removing bad dependencies on pytest-faulthandler (which should no
longer be used as of pytest 5.0).

There are ~40 python packages that depend on it and can now attempt builds!

CC @NixOS/nixos-release-managers

ZHF: #80379

It is currently failing on master and 20.03. I spent some time reading the src
code but was not able to figure out why the env var activation is not working.
Since this is currently failing, and since it's dying alongside python 2 anyways
I propose we just disable the 1 failing test.

There's some more information inline in the comment if someone wants to fix this
or dig further.

(cherry picked from commit 643d102)
This has been upstreamed into pytest itself as of pytest 5.0, which we have:
https://github.com/pytest-dev/pytest-faulthandler

Since it should no longer be used, let's remove from nix.

(cherry picked from commit f5ffd14)
@bhipple bhipple changed the title [20.03] python2Packages.faulthandler: fix build [20.03] python2Packages.faulthandler: 3.1 -> 3.2 and fix build Mar 27, 2020
@ofborg ofborg bot requested a review from sauyon March 27, 2020 17:21
Copy link
Contributor

@drewrisinger drewrisinger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Diffs LGTM. didn't attempt build.

Copy link
Contributor

@jonringer jonringer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

diff LGTM
commits LGTM

https://github.com/NixOS/nixpkgs/pull/83508
1 package marked as broken and skipped:
pants

1 package built:
python27Packages.faulthandler

@jonringer jonringer merged commit a6f3555 into NixOS:release-20.03 Mar 27, 2020
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