@GrahamcOfBorg build pythonPackages.weboob python3Packages.weboob

Commit msg should be pr title with the : before the actual msg

Backported in 22267de

Thanks @Kiwi ❇️

This should instead have been solved by adding pyqt5 to nativeBuildInputs too.

Oh, so it can. TIL. I can redo it if you want? @dotlambda

I think that would help with cross-compiling.
Btw, we should also remove that google_api_python_client_python27 from the expression and instead conditionally override the package on Python 2 from within python-packages.nix.

I'm not sure how to do all of that. I'm new to this and not very good at it yet I'm afraid. I'm more available on Freenode IRC as DigitalKiwi if that'd help figure it out.

This should instead have been solved by adding pyqt5 to nativeBuildInputs too.

Darn, I thought it was there and still didn't work for some reason.
I can make the other requested change with the override.

Btw, we should also remove that google_api_python_client_python27 from the expression and instead conditionally override the package on Python 2 from within python-packages.nix.

What do you mean by conditionally overriding from python-packages.nix?
I know you don't mean adding an older version 😄

What do you mean by conditionally overriding from python-packages.nix?

We can either an extra old.nix and call this for Python 2 or, to follow DRY, use something like

google_api_python_client = let
  google_api_python_client = callPackage ...
in if isPy3k then google_api_python_client else
  google_api_python_client.overridePythonAttrs { ... };

to follow DRY, use something like

google_api_python_client = let
 google_api_python_client = callPackage ...
in if isPy3k then google_api_python_client else
 google_api_python_client.overridePythonAttrs { ... };

Yep, that's what I thought you had in mind 👍