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

Move python pluggy to separate module #32391

Closed
wants to merge 36 commits into from

Conversation

maiksensi
Copy link
Contributor

@maiksensi maiksensi commented Dec 6, 2017

Motivation for this change

Pytest 3.3.x onward introduces new dependencies. One of them is pluggy. To make possible further changes easier it makes sense to extract it to a separate module.

I hope this is even the correct target branch for this change. I am sorry if not, please let me know and I'll change it as adviced.

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
  • 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 nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/) (as pluggy is not an executable, I manually used a interactive python shell to import pluggy.
  • Fits CONTRIBUTING.md.

pbogdan and others added 30 commits December 5, 2017 15:42
Add testing reqs, but keep tests disabled

Tests are runnable but still produce errors.

To get tests working, add locale setting, replace killall
and add nose package. To run the tests enable 'doCheck'.

Hunspell needs to be explicitly installed to use.

Patch SpellCheck.py to put hunspell in system datadir location.
For example, '/var/run/current-system/sw/share/hunspell/'
or '${HOME}/.nix-profile/share/hunspell/'

To get rid of atspi errors set
'services.gnome3.at-spi2-core.enable = true'
i386-linux: broken
aarch64-linux: x86_64-linux version is installed
haskellPackages.tmp-postgres: Add pginit dependency
Also compile against latest boost.
Add pythonPackages.salmon and dependencies
dezgeg and others added 6 commits December 6, 2017 17:29
This is required on the ThunderX CPUs on the Packet.net Type-2A
machines that have a GICv3. For some reason the default is to create a
GICv2 independent of the host hardware...
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