Skip to content

Commit

Permalink
python.pkgs.rlp: mark as broken
Browse files Browse the repository at this point in the history
  • Loading branch information
FRidh committed Jun 22, 2018
1 parent 9e10e51 commit e00ddc6
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions pkgs/development/python-modules/rlp/default.nix
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
{ lib, fetchPypi, buildPythonPackage, pytest }:
{ lib, fetchPypi, buildPythonPackage, pytest, hypothesis }:

buildPythonPackage rec {
name = "${pname}-${version}";
pname = "rlp";
version = "1.0.1";

Expand All @@ -10,12 +9,14 @@ buildPythonPackage rec {
sha256 = "492c11b18e89af42f98e96bca7671ffee4ad4cf5e69ea23b4d2221157d81b512";
};

buildInputs = [ pytest ];
checkInputs = [ pytest hypothesis ];
propagatedBuildInputs = [ ];

meta = {
description = "A package for encoding and decoding data in and from Recursive Length Prefix notation";
homepage = "https://github.com/ethereum/pyrlp";
license = lib.licenses.mit;
maintainers = with lib.maintainers; [ gebner ];
broken = true; # Requires a chain of unpackaged dependencies.
};
}

4 comments on commit e00ddc6

@Profpatsch
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wait, you updated this ten days earlier just to mark it broken? This breaks the trezor revdep, which is needed by the electrum application. Did @GrahamcOfBorg evaluation pass sucessfully?

@FRidh
Copy link
Member Author

@FRidh FRidh commented on e00ddc6 Jun 25, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This commit was part of one of my batch upgrades. I don't check every package. Maybe reverting the update fixes it, I don't know.

@Profpatsch
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hm, you commit batch updates directly to master without checking for breakage?

@dotlambda
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, not to master but to python-unstable, for which there is a Hydra job.

Please sign in to comment.