From 302aed5ab4e0362c701e0d44d4cf35dfe7da4965 Mon Sep 17 00:00:00 2001 From: Ruud van Asseldonk Date: Tue, 28 Jul 2020 16:43:44 +0200 Subject: [PATCH] python3Packages.sentry-sdk: fix trytond dependency Tryton is an optional dependency, if it is present, Sentry can integrate it, but when it is not present, Sentry should not pull it in as a propagated build input, it is only required for the tests that test this specific integration. In particular, pulling in trytond creates a transitive dependency on simplejson, which makes "requests" behave differently in a way that can break error handling. --- pkgs/development/python-modules/sentry-sdk/default.nix | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/pkgs/development/python-modules/sentry-sdk/default.nix b/pkgs/development/python-modules/sentry-sdk/default.nix index 9d2a533ecb6..3018cabd025 100644 --- a/pkgs/development/python-modules/sentry-sdk/default.nix +++ b/pkgs/development/python-modules/sentry-sdk/default.nix @@ -30,10 +30,10 @@ buildPythonPackage rec { sha256 = "0e5e947d0f7a969314aa23669a94a9712be5a688ff069ff7b9fc36c66adc160c"; }; - checkInputs = [ django flask tornado bottle rq falcon sqlalchemy werkzeug ] + checkInputs = [ django flask tornado bottle rq falcon sqlalchemy werkzeug trytond ] ++ stdenv.lib.optionals isPy3k [ celery pyramid sanic aiohttp ]; - propagatedBuildInputs = [ urllib3 certifi trytond ]; + propagatedBuildInputs = [ urllib3 certifi ]; meta = with stdenv.lib; { homepage = "https://github.com/getsentry/sentry-python";