diff options
author | Josef Kemetmueller <josef.kemetmueller@gmail.com> | 2017-05-30 02:14:46 +0200 |
---|---|---|
committer | Josef Kemetmueller <josef.kemetmueller@gmail.com> | 2017-05-30 02:18:40 +0200 |
commit | eb2994a0d5541d591cd4fc2435b978b00b19bc46 (patch) | |
tree | 4a00347b2ee246cae925cb88d3388cbef3754ca9 /pkgs/development/python-modules/pandas/default.nix | |
parent | 1d56c2fa434b2cf875aab4eabcfd69fa89fe25eb (diff) | |
download | nixpkgs-eb2994a0d5541d591cd4fc2435b978b00b19bc46.tar nixpkgs-eb2994a0d5541d591cd4fc2435b978b00b19bc46.tar.gz nixpkgs-eb2994a0d5541d591cd4fc2435b978b00b19bc46.tar.bz2 nixpkgs-eb2994a0d5541d591cd4fc2435b978b00b19bc46.tar.lz nixpkgs-eb2994a0d5541d591cd4fc2435b978b00b19bc46.tar.xz nixpkgs-eb2994a0d5541d591cd4fc2435b978b00b19bc46.tar.zst nixpkgs-eb2994a0d5541d591cd4fc2435b978b00b19bc46.zip |
pandas: Fix build on darwin
Since the commands "locale", "pbpaste" and "pbcopy" are not in nixpkgs, as they are impure darwin dependencies, we currently cannot get the clipboard and locale functionality to work properly. We disable the tests "test_locale" and "test_clipboard" on darwin, so we get a mostly working pandas. Additionally to disabling the test via py.test, we also need to provide fake pbpaste/pbcopy commands, so py.test won't fail during the collection phase. This closes #25417 and closes #11623.
Diffstat (limited to 'pkgs/development/python-modules/pandas/default.nix')
-rw-r--r-- | pkgs/development/python-modules/pandas/default.nix | 23 |
1 files changed, 14 insertions, 9 deletions
diff --git a/pkgs/development/python-modules/pandas/default.nix b/pkgs/development/python-modules/pandas/default.nix index 103907c26d1..0d3a2d2dd5c 100644 --- a/pkgs/development/python-modules/pandas/default.nix +++ b/pkgs/development/python-modules/pandas/default.nix @@ -19,7 +19,6 @@ , openpyxl , tables , xlwt -, darwin ? {} , libcxx ? null }: @@ -53,7 +52,7 @@ in buildPythonPackage rec { openpyxl tables xlwt - ] ++ optional isDarwin darwin.locale; # provides the locale command + ]; # For OSX, we need to add a dependency on libcxx, which provides # `complex.h` and other libraries that pandas depends on to build. @@ -63,16 +62,22 @@ in buildPythonPackage rec { substituteInPlace setup.py \ --replace "['pandas/src/klib', 'pandas/src']" \ "['pandas/src/klib', 'pandas/src', '$cpp_sdk']" - - # disable clipboard tests since pbcopy/pbpaste are not open source - substituteInPlace pandas/io/tests/test_clipboard.py \ - --replace pandas.util.clipboard no_such_module \ - --replace OSError ImportError ''; checkPhase = '' runHook preCheck - py.test $out/${python.sitePackages}/pandas --skip-slow --skip-network + '' + # TODO: Get locale and clipboard support working on darwin. + # Until then we disable the tests. + + optionalString isDarwin '' + # Fake the impure dependencies pbpaste and pbcopy + echo "#!/bin/sh" > pbcopy + echo "#!/bin/sh" > pbpaste + chmod a+x pbcopy pbpaste + export PATH=$(pwd):$PATH + '' + '' + py.test $out/${python.sitePackages}/pandas --skip-slow --skip-network \ + ${if isDarwin then "-k 'not test_locale and not test_clipboard'" else ""} runHook postCheck ''; @@ -83,7 +88,7 @@ in buildPythonPackage rec { homepage = "http://pandas.pydata.org/"; description = "Python Data Analysis Library"; license = stdenv.lib.licenses.bsd3; - maintainers = with stdenv.lib.maintainers; [ raskin fridh ]; + maintainers = with stdenv.lib.maintainers; [ raskin fridh knedlsepp ]; platforms = stdenv.lib.platforms.unix; }; } |