Skip to content

Commit

Permalink
pythonPackages.setuptools: Remove windows files and make reproducible
Browse files Browse the repository at this point in the history
- setuptools includes *.exe files by default, but can be excluded with an ENV variable.
- setuptools was built as an egg, which had reproducibility problems. Instead use a wheel

These are various *.exe and *.xml files used only on windows. setuptools
includes them by default since it normally creates a single release for
all operating systems.

This reduces the size from 1020.0K to 801.6K according to `nix-path -sh`.

The egg is a zip file. setuptools leaves timestamps in the egg,
which makes the build unreproducible. Unfortunately the files aren't
compressed so the size of setuptools increases to 2.3M from 0.8M
according to `nix path-info -sh`.

With this change, setuptools is reproducible according to

    nix-build -A python37Packages.setuptools --check
  • Loading branch information
singron authored and FRidh committed Oct 14, 2019
1 parent dd6261c commit 90be4c2
Showing 1 changed file with 15 additions and 10 deletions.
25 changes: 15 additions & 10 deletions pkgs/development/python-modules/setuptools/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -6,11 +6,17 @@
, unzip
, callPackage
, bootstrapped-pip
, lib
, pipInstallHook
, setuptoolsBuildHook
}:

buildPythonPackage rec {
pname = "setuptools";
version = "41.2.0";
# Because of bootstrapping we don't use the setuptoolsBuildHook that comes with format="setuptools" directly.
# Instead, we override it to remove setuptools to avoid a circular dependency.
# The same is done for pip and the pipInstallHook.
format = "other";

src = fetchPypi {
Expand All @@ -19,19 +25,18 @@ buildPythonPackage rec {
sha256 = "66b86bbae7cc7ac2e867f52dc08a6bd064d938bac59dfec71b9b565dd36d6012";
};

# There is nothing to build
dontBuild = true;
nativeBuildInputs = [
bootstrapped-pip
(pipInstallHook.override{pip=null;})
(setuptoolsBuildHook.override{setuptools=null; wheel=null;})
];

nativeBuildInputs = [ bootstrapped-pip ];

installPhase = ''
dst=$out/${python.sitePackages}
mkdir -p $dst
export PYTHONPATH="$dst:$PYTHONPATH"
${python.pythonForBuild.interpreter} setup.py install --prefix=$out
wrapPythonPrograms
preBuild = lib.strings.optionalString (!stdenv.hostPlatform.isWindows) ''
export SETUPTOOLS_INSTALL_WINDOWS_SPECIFIC_FILES=0
'';

pipInstallFlags = [ "--ignore-installed" ];

# Adds setuptools to nativeBuildInputs causing infinite recursion.
catchConflicts = false;

Expand Down

0 comments on commit 90be4c2

Please sign in to comment.