I chose to directly package Kivy 2.0.0rc4 because it supports cython 0.29.21, which is the version in nixpkgs.

For Kivy itself I think you're likely to also need pkg-config:

WARNING: A problem occurred while running pkg-config --libs --cflags gstreamer-1.0 (code 127)

b'/nix/store/516z50fm1jbpcl32qnzy7kynrh0vl22w-bash-4.4-p23/bin/sh: pkg-config: command not found\n'

WARNING: A problem occurred while running pkg-config --libs --cflags sdl2 SDL2_ttf SDL2_image SDL2_mixer (code 127)

b'/nix/store/516z50fm1jbpcl32qnzy7kynrh0vl22w-bash-4.4-p23/bin/sh: pkg-config: command not found\n'

WARNING: A problem occurred while running pkg-config --libs --cflags pangoft2 (code 127)

b'/nix/store/516z50fm1jbpcl32qnzy7kynrh0vl22w-bash-4.4-p23/bin/sh: pkg-config: command not found\n'

ERROR: Dependency for context.pyx not resolved: config.pxi

(I think you want pkg-config, not pkgconfig in nativeBuildInputs?)

(I think you want pkg-config, not pkgconfig in nativeBuildInputs?)

Yes indeed, it's weird it was building on my machine with nix-build . -A python3Packages.kivy but not when used in a project :/

I think you want pkg-config, not pkgconfig in nativeBuildInputs?)

pkgconfig is an alias of pkg-config and the later is preferred.

I think you want pkg-config, not pkgconfig in nativeBuildInputs?)

pkgconfig is an alias of pkg-config and the later is preferred.

I think one of the reasons for that is there's a separate Python module called "pkgconfig" which, err, does something different (so in the python-modules directory you end up with that instead, which doesn't work)

Result of nixpkgs-review pr 104694 run on x86_64-darwin 1

It's now been tested with an application that actually uses it:

🎉

you may have to do some heavy patching for kivy to work well with nix (as in, patch all the locations, and potentially have to regenerate that patch every package update) as normal dll calls will not work.

As far as I tested with a friend's project using this, it seems to work. It probably doesn't use all of Kivy's features though, so some stuff might be broken, but we can always open fixup PRs later on as we discover features that do not work.

I think this is now ready.

Result of nixpkgs-review pr 104694 run on x86_64-linux 1

kivy-garden

Ran 0 tests in 0.000s

Please fix that no tests are run or disable them and add a pythonImportCheck.

Please fix that no tests are run or disable them and add a pythonImportCheck.

Done!

Result of nixpkgs-review pr 104694 run on x86_64-linux 1

What is remaining on this PR prior to merging?

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.

Result of nixpkgs-review pr 104694 run on x86_64-linux 1

The following issues got detected with the above build packages.
Please fix at least the ones listed with your changed packages:

python37Packages.kivy-garden:

I fixed the typo in pythonImportsCheck for kivy.

Result of nixpkgs-review pr 104694 run on x86_64-linux 1

The pythonImportsCheckPhase is indeed ran, however there are no tests in the archive from Pypi. I very much don't want to fetch this package from github as it is split in multiple repos and don't want to have the burden of checking them all to update this package. Plus, it uses the buildozer tool which I know nothing about.

The pythonImportsCheckPhase is indeed ran, however there are no tests in the archive from Pypi. I very much don't want to fetch this package from github as it is split in multiple repos and don't want to have the burden of checking them all to update this package. Plus, it uses the buildozer tool which I know nothing about.

fine for me. Then just add doCheck = false; and a comment that building from source was not feasible because of buildozer and multiple repos.

fine for me. Then just add doCheck = false; and a comment that building from source was not feasible because of buildozer and multiple repos.

Done!

Thank you for taking the time to package this! I'll be using it later.

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.

Result of nixpkgs-review pr 104694 run on x86_64-linux 1

The following issues got detected with the above build packages.
Please fix at least the ones listed with your changed packages:

  |
2 | , buildPythonPackage, fetchPypi, fetchpatch
  |                                  ^

  |
7 | , pytestCheckHook
  |   ^

  |
2 | , buildPythonPackage, fetchPypi, fetchpatch
  |                                  ^

  |
7 | , pytestCheckHook
  |   ^

  |
2 | , buildPythonPackage, fetchPypi, fetchpatch
  |                                  ^

  |
7 | , pytestCheckHook
  |   ^

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.

Result of nixpkgs-review pr 104694 run on x86_64-darwin 1

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.

Result of nixpkgs-review pr 104694 run on x86_64-linux 1