Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: NixOS/nixpkgs
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: 39789683d049
Choose a base ref
...
head repository: NixOS/nixpkgs
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: 098cb9f63aa7
Choose a head ref
  • 3 commits
  • 2 files changed
  • 2 contributors

Commits on Mar 8, 2019

  1. pythonPackages.pybind11: apply patch to properly get headers directories

    It seems as the `pybind11` build code returns the Python headers
    directory (where the `pybind11` headers are stored as well on traditional
    setups) rather than returning the dedicated prefix[1].
    
    An exemplary fallout is the broken build of `pyopencl`[2].
    
    [1] pybind/pybind11#1425
    [2] #56082
    Ma27 committed Mar 8, 2019

    Verified

    This commit was signed with the committer’s verified signature. The key has expired.
    Ma27 Maximilian Bosch
    Copy the full SHA
    94c3ac2 View commit details

Commits on Mar 11, 2019

  1. python3Packages.pyopencl: fix build

    Fix the recently broken build by adding `pybind11`
    to the build. Also set $HOME to a temporary directory during the build
    to avoid "Permission denied" errors in the build script.
    
    This also unbreaks `sasview` and `pybitmessage`.
    
    See also #56826
    See also https://hydra.nixos.org/build/89037506
    Ma27 committed Mar 11, 2019

    Verified

    This commit was signed with the committer’s verified signature. The key has expired.
    Ma27 Maximilian Bosch
    Copy the full SHA
    d237919 View commit details

Commits on Mar 13, 2019

  1. Merge pull request #56082 from Ma27/fix-pyopencl

    python3Packages.pyopencl: fix build
    dotlambda authored Mar 13, 2019

    Verified

    This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
    Copy the full SHA
    098cb9f View commit details
Showing with 14 additions and 2 deletions.
  1. +8 −1 pkgs/development/python-modules/pybind11/default.nix
  2. +6 −1 pkgs/development/python-modules/pyopencl/default.nix
9 changes: 8 additions & 1 deletion pkgs/development/python-modules/pybind11/default.nix
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
{ lib, buildPythonPackage, fetchPypi }:
{ lib, buildPythonPackage, fetchPypi, fetchpatch }:

buildPythonPackage rec {
pname = "pybind11";
@@ -9,6 +9,13 @@ buildPythonPackage rec {
sha256 = "1kz1z2cg3q901q9spkdhksmcfiskaghzmbb9ivr5mva856yvnak4";
};

patches = [
(fetchpatch {
url = https://github.com/pybind/pybind11/commit/44a40dd61e5178985cfb1150cf05e6bfcec73042.patch;
sha256 = "047nzyfsihswdva96hwchnp4gj2mlbiqvmkdnhxrfi9sji8x31ka";
})
];

# Current PyPi version does not include test suite
doCheck = false;

7 changes: 6 additions & 1 deletion pkgs/development/python-modules/pyopencl/default.nix
Original file line number Diff line number Diff line change
@@ -11,14 +11,15 @@
, six
, opencl-headers
, ocl-icd
, pybind11
}:

buildPythonPackage rec {
pname = "pyopencl";
version = "2018.2.3";

checkInputs = [ pytest ];
buildInputs = [ opencl-headers ocl-icd ];
buildInputs = [ opencl-headers ocl-icd pybind11 ];

propagatedBuildInputs = [ numpy cffi pytools decorator appdirs six Mako ];

@@ -32,6 +33,10 @@ buildPythonPackage rec {
substituteInPlace setup.py --replace "pytest>=2" ""
'';

preBuild = ''
export HOME=$(mktemp -d)
'';

# gcc: error: pygpu_language_opencl.cpp: No such file or directory
doCheck = false;