Skip to content

pythonPackages.salmon-mail: 3.1.1 -> 3.2.0 #82166

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

Merged
merged 1 commit into from
Apr 1, 2020

Conversation

jluttine
Copy link
Member

@jluttine jluttine commented Mar 9, 2020

Motivation for this change

Upgrade Python package salmon-mail.

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 nixpkgs-review --run "nixpkgs-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.

Sorry, something went wrong.

@jluttine jluttine requested review from FRidh and jonringer as code owners March 9, 2020 16:34
@ofborg ofborg bot added 6.topic: python 11.by: package-maintainer This PR was created by the maintainer of the package it changes 10.rebuild-darwin: 1-10 10.rebuild-linux: 1-10 labels Mar 9, 2020
@marsam
Copy link
Contributor

marsam commented Mar 10, 2020

@GrahamcOfBorg build python37Packages.salmon-mail


# The tests use salmon executable installed by salmon itself so we need to add
# that to PATH
checkPhase = ''
PATH=$out/bin:$PATH nosetests .
PATH=$out/bin:$PATH python setup.py test
Copy link
Contributor

Choose a reason for hiding this comment

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

getting some failures:

warning: ignoring the user-specified setting 'sandbox', because it is a restricted setting and you are not a trusted user
builder for '/nix/store/446ahn0w3b5b7dqh1245hc55pcfsxbvj-python3.7-salmon-mail-3.2.0.drv' failed with exit code 1; last 10 log lines:
    File "/build/salmon-mail-3.2.0/salmon/routing.py", line 503, in routing_wrapper
      next_state = func(message, *args, **kw)
  TypeError: END() got an unexpected keyword argument 'anything'

  ----------------------------------------------------------------------
  Ran 147 tests in 2.561s

  FAILED (errors=1)
  Test failed: <unittest.runner.TextTestResult run=147 errors=1 failures=0>
  error: Test failed: <unittest.runner.TextTestResult run=147 errors=1 failures=0>
builder for '/nix/store/bl0xb0l6dmvkw46cq75dw5axzan75cxn-python3.8-salmon-mail-3.2.0.drv' failed with exit code 1; last 10 log lines:
    File "/build/salmon-mail-3.2.0/salmon/routing.py", line 503, in routing_wrapper
      next_state = func(message, *args, **kw)
  TypeError: END() got an unexpected keyword argument 'anything'

  ----------------------------------------------------------------------
  Ran 147 tests in 2.294s

  FAILED (errors=1)
  Test failed: <unittest.runner.TextTestResult run=147 errors=1 failures=0>
  error: Test failed: <unittest.runner.TextTestResult run=147 errors=1 failures=0>
cannot build derivation '/nix/store/j798yif7088svypcji7mzqqywq71dyw4-env.drv': 2 dependencies couldn't be built

Copy link
Member Author

Choose a reason for hiding this comment

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

Well, that's weird.. This builds fine on my machine:

$ nix-build '<nixpkgs>' -A python37Packages.salmon-mail
/nix/store/p701xnxwrwvhg8pc76m02z29923lgxv1-python3.7-salmon-mail-3.2.0

What could cause the difference? Difficult to fix if I'm not able to reproduce..

Copy link
Member Author

Choose a reason for hiding this comment

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

I now rebased on nixos-unstable. Still building fine on my machine...

Copy link
Contributor

Choose a reason for hiding this comment

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

I'll come in as a neutral third party and say this builds just fine on my machine as well:

[4 built, 7 copied (2.1 MiB), 0.2 MiB DL]
https://github.com/NixOS/nixpkgs/pull/82166
2 package built:
python37Packages.salmon-mail python38Packages.salmon-mail

That said, the OfBorg build is failing on Darwin:

 start explode localhost
EXPLODE!
Relay: <Relay to (localhost:0)>
error: Test failed: <unittest.runner.TextTestResult run=147 errors=0 failures=1>
builder for '/nix/store/dwdzz3r1j5abi09g48x6mhf5r406gy4r-python3.7-salmon-mail-3.2.0.drv' failed with exit code 1

Copy link
Member Author

Choose a reason for hiding this comment

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

Hmm.. Should I try if __darwinAllowLocalNetworking = true; fixes build on Darwin?

Copy link
Member Author

Choose a reason for hiding this comment

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

I now added that flag. Let's see if it helps. How can I use OfBorg to try building again for Darwin to see if it helped?

@jluttine jluttine force-pushed the salmon-mail-3.2.0 branch from 19135fe to 6546af6 Compare March 11, 2020 19:55
@FRidh
Copy link
Member

FRidh commented Mar 27, 2020

@GrahamcOfBorg build python37Packages.salmon-mail

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@jluttine jluttine force-pushed the salmon-mail-3.2.0 branch from 6546af6 to ef86430 Compare April 1, 2020 16:48
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.

still getting failures

warning: ignoring the user-specified setting 'sandbox', because it is a restricted setting and you are not a trusted user
builder for '/nix/store/mhfi91xvxccgwmv792a12xh9nkfk3dd7-python3.8-salmon-mail-3.2.0.drv' failed with exit code 1; last 10 log lines:
    File "/build/salmon-mail-3.2.0/salmon/routing.py", line 503, in routing_wrapper
      next_state = func(message, *args, **kw)
  TypeError: END() got an unexpected keyword argument 'anything'

  ----------------------------------------------------------------------
  Ran 147 tests in 1.510s

  FAILED (errors=1)
  Test failed: <unittest.runner.TextTestResult run=147 errors=1 failures=0>
  error: Test failed: <unittest.runner.TextTestResult run=147 errors=1 failures=0>
builder for '/nix/store/kd08nimwdzb8kk7nm7h160d8f35cs4bi-python3.7-salmon-mail-3.2.0.drv' failed with exit code 1; last 10 log lines:
    File "/build/salmon-mail-3.2.0/salmon/routing.py", line 503, in routing_wrapper
      next_state = func(message, *args, **kw)
  TypeError: END() got an unexpected keyword argument 'anything'

  ----------------------------------------------------------------------
  Ran 147 tests in 1.729s

  FAILED (errors=1)
  Test failed: <unittest.runner.TextTestResult run=147 errors=1 failures=0>
  error: Test failed: <unittest.runner.TextTestResult run=147 errors=1 failures=0>
cannot build derivation '/nix/store/aacgrwnl5b8pw7fsm271w6iwq6b3aw4m-env.drv': 2 dependencies couldn't be built

can you check to see if you have anything in your overlay

@jonringer
Copy link
Contributor

@GrahamcOfBorg build python3Packages.salmon-mail

@jluttine
Copy link
Member Author

jluttine commented Apr 1, 2020

@jonringer If I understand correctly, OfBorg builds fine for Linux, also it builds fine on my machine and also @bhipple machine. I'd need to be able to reproduce the failure in order to fix it. Could you share the command you used to build which resulted in the failure? But if it builds fine on OfBorg, isn't that then ok?

Apparently, Darwin is still failing, so I guess I should either disable tests for Darwin or disable the whole package for Darwin. Any suggestions?

@jonringer
Copy link
Contributor

if any one of these: chardet dnspython lmtpd python-daemon six click get changed, then I will have to build this when reviewing a package. And for me, it will show as a regression

@jonringer
Copy link
Contributor

Looking at the test suite, mine looks like a legitimate error. The dummy END() method does not in fact take an anything argument.

Also doesn't help that his test suite is hand done, and not using a normal testing framework

@jluttine
Copy link
Member Author

jluttine commented Apr 1, 2020

@jonringer I still don't understand why the build works for me if it's not supposed to. I ran nixpkgs review and it shows all good:

$ nixpkgs-review pr 82166
...
2 package built:
python37Packages.salmon-mail python38Packages.salmon-mail

Could you share the explicit command that you use to reproduce the build failure? I want to understand what I'm doing wrong here if this is not supposed to build fine on Linux.

@jonringer
Copy link
Contributor

I have no idea either. I'm just going to mentally blacklist the package for my machine. Maybe it's because it's a ryzen3, who knows? not I

@jonringer jonringer merged commit 094f7dd into NixOS:master Apr 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants