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

sequoia: 0.10.0 -> 0.11.0 #73104

Merged
merged 1 commit into from Nov 21, 2019
Merged

Conversation

doronbehar
Copy link
Contributor

Motivation for this change

Closes #70646.

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Ensured that relevant documentation is up to date
  • Fits CONTRIBUTING.md.
Notify maintainers

cc @minijackson

@minijackson
Copy link
Member

Hey, thank you for the update!

Can you tell me more on how this fixes #70646?

@minijackson
Copy link
Member

Unfortunately, nix-review fails with me, with a No space left on device error, so I can't test it :-/

@doronbehar
Copy link
Contributor Author

Unfortunately, nix-review fails with me, with a No space left on device error, so I can't test it :-/

Do you happen to be using boot.tmpOnTmpfs? If so, then you need something along the lines of:

    # Use an a tmpdir with unlimited space (as opposed to /tmp)
    systemd.services."nix-daemon".environment.TMPDIR = "/nix/tmp";

BTW, my cachix has a build of this derivation.

Can you tell me more on how this fixes #70646?

TBH, and @worldofpeace should be cc'd, I don't know why it failed on hydra in the first place, as it always builds locally. it seems to be only a matter of resources, hence the removed broken flag. If it'll keep fail on hydra, I think the matter should be treated by someone with access to hydra's settings. See my comment: #70646 (comment) .

@risicle
Copy link
Contributor

risicle commented Nov 9, 2019

@doronbehar
Copy link
Contributor Author

Hmm, interesting, did you try to build python3Packages.sequoia or just sequoia? A Google search implies this is a common python issue with older versions of SSL and / or the missing security optional dependency for Python...

@risicle
Copy link
Contributor

risicle commented Nov 9, 2019

Both gave a similar error.

@doronbehar
Copy link
Contributor Author

Could you please disable the tests doCheck = false and see if the executables are functional?

@risicle
Copy link
Contributor

risicle commented Nov 9, 2019

@worldofpeace
Copy link
Contributor

I guess failing on macos is a different issue.

@doronbehar
Copy link
Contributor Author

So @worldofpeace shall we disable tests for MacOS only in the meantime?

@risicle
Copy link
Contributor

risicle commented Nov 10, 2019

The tests aren't the problem on macos.

@doronbehar
Copy link
Contributor Author

Well I have no way to debug this for MacOS. what perplexes me though is that the 2 gists you've published indicate a different issue every time. What was the different between the 2 attempts?

@risicle
Copy link
Contributor

risicle commented Nov 10, 2019

The first time doCheck was true, the second doCheck was false, as you asked.

@doronbehar
Copy link
Contributor Author

The first time doCheck was true, the second doCheck was false, as you asked.

I see. If no MacOS user would like to fix this issue, then I'm marking the package as broken only for MacOS. See 3a7bbb6 .

@worldofpeace
Copy link
Contributor

@doronbehar Yeah, it's looking like a bit of a hassle. We can just do as I suggested as an alternative in a09e3e2, which is to drop that commit and just disable checkPhase. We can communicate with upstream.

@doronbehar
Copy link
Contributor Author

@worldofpeace what do you think about @curiousleo's suggestion:

Build time > 3600 seconds (with Python bindings) seems excessive - I've reported this upstream: https://gitlab.com/sequoia-pgp/sequoia/issues/357

?

@doronbehar
Copy link
Contributor Author

BTW @worldofpeace setting the ulimit fails the build locally for me, with:

/nix/store/kirg08m7yzlp226q7ki61q9j5zjf1x9p-stdenv-linux/setup: line 97: ulimit: open files: cannot modify limit: Operation not permitted

Pushed a revert.

@minijackson
Copy link
Member

(I've just noticed, but your commit message and PR title indicates the upgrade from 0.10.0 to 0.11.1, but the new version in the commit is 0.11.0 ) 😛

Disable check on Darwin.
@doronbehar doronbehar changed the title sequoia: 0.10.0 -> 0.11.1 sequoia: 0.10.0 -> 0.11.0 Nov 19, 2019
@doronbehar
Copy link
Contributor Author

Thanks @minijackson :)

@worldofpeace worldofpeace merged commit ac82d7a into NixOS:master Nov 21, 2019
@doronbehar doronbehar mentioned this pull request Dec 26, 2019
10 tasks
@puzzlewolf puzzlewolf mentioned this pull request Jun 19, 2020
10 tasks
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.

sequoia fails at checkPhase
4 participants