Change:
I had i3pystatus fail to build because of a pytest error with praw on unstable. I tried to bump praw from 6.0.0 to 6.3.1 but it requires a new dep websocket-client that doesn't look packaged thus I ended up ditching it from i3pystatus dependencies. Maybe we should do the same upstream and provide a leaner i3pystatus as it's not a core plugin anyway (and fails ). nix build -f channel:nixos-unstable system: host os: multi-user?: sandbox: version: channels(teto): channels(root): nixpkgs:
CC ? what do you think of removing it from i3pystatus default ?
Agree. A user can add manually by overriding of parameter.
is this still relevant now? or can it be closed?
I just removed praw it from i3pystatus default dependencies but itself still won't build. So it makes sense to keep the issue open even if I have no personal interest in praw.
it was irritating me, so i updated it.

Discussion:
{ stdenv, buildPythonPackage, fetchFromGitHub <del> , requests, decorator, flake8, mock, six, update_checker, pytestrunner, prawcore , pytest_3, betamax, betamax-serializers, betamax-matchers, requests_toolbelt </del> <ins> , betamax , betamax-serializers , betamax-matchers , mock , six , pytestrunner , prawcore , pytest , requests-toolbelt , update_checker , websocket_client </ins> }: buildPythonPackage rec { pname = "praw"; <del> version = "6.0.0"; </del> <ins> version = "6.3.1"; </ins> src = fetchFromGitHub { owner = "praw-dev"; repo = "praw"; rev = "v${version}"; <del> sha256 = "0y6nyz8vf98gl1qfmnznv3dbvlbzdl6mz99vk673nyfn3hbs451i"; </del> <ins> sha256 = "0by89aw7m803dvjcc33m9390msjm6v5v8g3k8ink9gfm421lw8ky"; </ins> }; <del> postPatch = '' # drop upper bound of prawcore requirement sed -ri "s/'(prawcore >=.+), <.+'/'1'/" setup.py ''; </del> <ins> nativeBuildInputs = [ pytestrunner ]; </ins> propagatedBuildInputs = [ <del> requests decorator flake8 </del> mock <del> six update_checker pytestrunner </del> prawcore <ins> update_checker websocket_client </ins> ]; checkInputs = [ <del> pytest_3 </del> betamax betamax-serializers betamax-matchers <del> requests_toolbelt </del> <ins> mock pytest requests-toolbelt six </ins> ]; meta = with stdenv.lib; { description = "Python Reddit API wrapper"; <del> homepage = https://praw.readthedocs.org/; license = licenses.gpl3; </del> <ins> homepage = "https://praw.readthedocs.org/"; license = licenses.bsd2; </ins> platforms = platforms.all; maintainers = with maintainers; [ ]; };

<ins> { lib, buildPythonPackage, fetchPypi, isPy27 , six , backports_ssl_match_hostname }: buildPythonPackage rec { version = "0.56.0"; pname = "websocket_client"; src = fetchPypi { inherit pname version; sha256 = "0fpxjyr74klnyis3yf6m54askl0h5dchxcwbfjsq92xng0455m8z"; }; propagatedBuildInputs = [ six ] ++ lib.optional isPy27 backports_ssl_match_hostname; meta = with lib; { description = "Websocket client for python"; homepage = "https://github.com/websocket-client/websocket-client"; license = licenses.bsd3; maintainers = with maintainers; [ ]; }; } </ins>

<del> { stdenv, buildPythonPackage, fetchPypi, six }: buildPythonPackage rec { pname = "websocket_client"; version = "0.54.0"; src = fetchPypi { inherit version pname; sha256 = "e51562c91ddb8148e791f0155fdb01325d99bb52c4cdbb291aee7a3563fd0849"; }; prePatch = '' # ssl.match_hostname exists in python2.7 version maintained in nixpkgs, # the dependency is not necessary. sed -e "s/['backports.ssl_match_hostname']/[]/" -i setup.py ''; propagatedBuildInputs = [ six ]; meta = with stdenv.lib; { homepage = https://github.com/liris/websocket-client; description = "Websocket client for python"; license = licenses.lgpl2; }; } </del>

larch = callPackage ../development/python-modules/larch { }; <del> websocket_client = callPackage ../development/python-modules/websockets_client { }; </del> <ins> websocket_client = callPackage ../development/python-modules/websocket_client { }; </ins> webhelpers = callPackage ../development/python-modules/webhelpers { };