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

pythonPackages.py3status: 3.0 -> 3.5 #26965

Closed
wants to merge 1 commit into from

Conversation

dotlambda
Copy link
Member

@dotlambda dotlambda commented Jun 29, 2017

I also added some patches to fix binary paths.
pkgs.file is needed to detect file encoding.

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
    • Linux
  • 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/)
  • Fits CONTRIBUTING.md.

Just to ping the maintainer: @garbas

@garbas garbas self-requested a review June 29, 2017 21:14
};
propagatedBuildInputs = with self; [ requests ];
propagatedBuildInputs = with self; [ requests dbus-python pkgs.file ];
doCheck = false;
Copy link
Member

Choose a reason for hiding this comment

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

Why do you disable tests?

@FRidh
Copy link
Member

FRidh commented Jul 31, 2017

ping

@dotlambda
Copy link
Member Author

Sorry for not getting back to you, exams and stuff...

I actually disabled tests to make this work on Python 3, but I think I'll need to work on this again to also call the python3 version in all-packages.nix.
Then, I'll also try to run the tests in python3 (I think they are simply not run when using version 2), but because of all the modules used by some component of py3status, I don't think I will have much luck. NixOS and dynamic loading of dependencies just don't go together.

@dotlambda
Copy link
Member Author

Actually, I think I'll open a new pull request when I have the time.

@dotlambda dotlambda closed this Sep 5, 2017
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

4 participants