Skip to content

Commit

Permalink
python.pkgs.hypothesis: 3.11.0 -> 3.11.1
Browse files Browse the repository at this point in the history
  • Loading branch information
@FRidh
FRidh committed Jun 1, 2017
1 parent 8742c1e commit 2050213
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions pkgs/development/python-modules/hypothesis.nix
View file
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ buildPythonPackage rec {
# pytz fake_factory django numpy pytest
# If you need these, you can just add them to your environment.

version = "3.11.0";
version = "3.11.1";
pname = "hypothesis";
name = "${pname}-${version}";

Expand All @@ -18,7 +18,7 @@ buildPythonPackage rec {
owner = "HypothesisWorks";
repo = "hypothesis-python";
rev = "${version}";
sha256 = "1s911pd3y9hvk0hq2fr6i68dqv1ciagryhgp13wgyfqh8hz8j6zv";
sha256 = "5344cc3327bc7fa543fc3b42e85c55f40dda0eeaec38327f9bf373c3ece42b39";
};

checkInputs = stdenv.lib.optionals doCheck [ pytest pytest_xdist flake8 flaky ];
Expand Down

10 comments on commit 2050213

@pmahoney
Copy link
Contributor

Choose a reason for hiding this comment

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

@FRidh I get this error building something on Darwin using latest master.

building path(s) ‘/nix/store/a4qbn0fajlpqlypz6sg6g5a0f8y3cj85-hypothesis-python-3.11.1-src’

trying https://github.com/HypothesisWorks/hypothesis-python/archive/3.11.1.tar.gz
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   141    0   141    0     0    728      0 --:--:-- --:--:-- --:--:--   734
100  323k  100  323k    0     0   383k      0 --:--:-- --:--:-- --:--:-- 1412k
unpacking source archive /private/var/folders/jq/z8z2183547s2yp4vw2k5mh3h0000gp/T/nix-build-hypothesis-python-3.11.1-src.drv-0/3.11.1.tar.gz
output path ‘/nix/store/a4qbn0fajlpqlypz6sg6g5a0f8y3cj85-hypothesis-python-3.11.1-src’ has r:sha256 hash ‘0damf6zbm0db2a3gfwrbbj92yal576wpmhhchc0w0np8vdnax70n’ when ‘0f9bwknc6wzkkdzk4f7cx87dl3glamffhhivzi1sazxw4wrwqi2k’ was expected

@risicle
Copy link
Contributor

@risicle risicle commented on 2050213 Jun 1, 2017

Choose a reason for hiding this comment

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

Yeah I think you may have the wrong hash here @FRidh - I get 0damf6zbm0db2a3gfwrbbj92yal576wpmhhchc0w0np8vdnax70n.

@vcunat
Copy link
Member

@vcunat vcunat commented on 2050213 Jun 1, 2017 •
edited

Choose a reason for hiding this comment

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

@FRidh: this is a mass rebuild commit (and perhaps some other commits around as well). More than half of all jobs are now rebuilt by Hydra on master. Go with such commits through staging first, please :-(

If you find it difficult to see what's a mass rebuild, you can use a WIP of mine; discussion on #19045.

@LnL7
Copy link
Member

@LnL7 LnL7 commented on 2050213 Jun 1, 2017

Choose a reason for hiding this comment

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

I think this is partly my fault, I merged something that introduced a bunch of extra python dependencies into the darwin stdenv.

@risicle
Copy link
Contributor

@risicle risicle commented on 2050213 Jun 1, 2017

Choose a reason for hiding this comment

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

It's not darwin-specific.

@FRidh
Copy link
Member Author

@FRidh FRidh commented on 2050213 Jun 2, 2017 via email

Choose a reason for hiding this comment

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

@FRidh
Copy link
Member Author

@FRidh FRidh commented on 2050213 Jun 2, 2017

Choose a reason for hiding this comment

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

So in the script I have to check whether we are using a) fetchPypi or b) an url that contains pypi. Otherwise, it should abort.

@vcunat
Copy link
Member

@vcunat vcunat commented on 2050213 Jun 2, 2017

Choose a reason for hiding this comment

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

I don't understand what mess you mean.

@FRidh
Copy link
Member Author

@FRidh FRidh commented on 2050213 Jun 2, 2017

Choose a reason for hiding this comment

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

@vcunat it took a month for commits that went into staging to arrive in master because there were a) commits that weren't fixed and/or b) evaluations that did not finish.

In any case, I've fixed the Hypothesis issue. I think its best to directly merge staging back in master so that Hydra won't build two different sets.

@vcunat
Copy link
Member

@vcunat vcunat commented on 2050213 Jun 2, 2017

Choose a reason for hiding this comment

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

For at least a week I was repeatedly killing the staging evaluations exactly because there were more and more mass rebuilds coming to master, so staging would never catch up (because it's low-priority set on Hydra). It's true that the Eelco's change also took me some time to fix most of the failing packages due to newly detected bad RPATH items.

Hmm, merged then. It's certainly better not to build it twice, though I would have preferred to do the rebuild on staging instead, as that's what it's for.

Please sign in to comment.