Merge pull request #112381 from SuperSandro2000/httpie

{ lib, fetchFromGitHub, python3Packages, docutils, fetchpatch }:

{ lib, fetchFromGitHub, python3Packages, docutils }:

python3Packages.buildPythonApplication rec {

  pname = "httpie";

  version = "2.2.0";

  version = "2.4.0";

  src = fetchFromGitHub {

    owner = "jakubroztocil";

    owner = "httpie";

    repo = "httpie";

    rev = version;

    sha256 = "0caazv24jr0844c4mdx77vzwwi5m869n10wa42cydb08ppx1xxj6";

    sha256 = "00lafjqg9nfnak0nhcr2l2hzzkwn2y6qv0wdkm6r6f69snizy3hf";

  };

  outputs = [ "out" "doc" "man" ];

  propagatedBuildInputs = with python3Packages; [ pygments requests setuptools ];

  dontUseSetuptoolsCheck = true;

  patches = [

    ./strip-venv.patch

    # Fix `test_ciphers_none_can_be_selected`

    # TODO: remove on next release

    (fetchpatch {

      url = "https://github.com/jakubroztocil/httpie/commit/49e71d252f54871a6bc49cb1cba103d385a543b8.patch";

      sha256 = "13b2faf50gimj7f17dlx4gmd8ph8ipgihpzfqbvmfjlbf1v95fsj";

    })

  ];

  outputs = [ "out" "doc" "man" ];

  propagatedBuildInputs = with python3Packages; [ pygments requests requests-toolbelt setuptools ];

  checkInputs = with python3Packages; [

    mock

    pytest

    toHtml CHANGELOG.rst $docdir/html/CHANGELOG.html

    toHtml CONTRIBUTING.rst $docdir/html/CONTRIBUTING.html

    # change a few links to the local files

    substituteInPlace $docdir/html/index.html \

      --replace \

        'https://github.com/jakubroztocil/httpie/blob/master/CHANGELOG.rst' \

        "CHANGELOG.html" \

      --replace \

        'https://github.com/jakubroztocil/httpie/blob/master/CONTRIBUTING.rst' \

        "CONTRIBUTING.html"

    ${docutils}/bin/rst2man \

      --strip-elements-with-class=no-web \

      --title=http \

    export PATH=${docutils}/bin:$PATH

  '';

  meta = {

  checkPhase = ''

    py.test ./httpie ./tests --doctest-modules --verbose ./httpie ./tests -k 'not test_chunked and not test_verbose_chunked and not test_multipart_chunked and not test_request_body_from_file_by_path_chunked'

  '';

  meta = with lib; {

    description = "A command line HTTP client whose goal is to make CLI human-friendly";

    homepage = "https://httpie.org/";

    license = lib.licenses.bsd3;

    maintainers = with lib.maintainers; [ antono relrod schneefux ];

    license = licenses.bsd3;

    maintainers = with maintainers; [ antono relrod schneefux SuperSandro2000 ];

  };

}

diff --git a/tests/test_docs.py b/tests/test_docs.py

index 7a41822..720ecf6 100644

index 340e64d..a6b4dc9 100644

--- a/tests/test_docs.py

+++ b/tests/test_docs.py

@@ -41,12 +41,10 @@ assert filenames

@@ -42,15 +42,10 @@ assert filenames

 # HACK: hardcoded paths, venv should be irrelevant, etc.

 # TODO: replaces the process with Python code

 # TODO: simplify by using the Python API instead of a subprocess

 #       then we wont’t need the paths.

-VENV_BIN = Path(__file__).parent.parent / 'venv/bin'

-VENV_PYTHON = VENV_BIN / 'python'

-VENV_RST2PSEUDOXML = VENV_BIN / 'rst2pseudoxml.py'

+VENV_RST2PSEUDOXML = 'rst2pseudoxml.py'

-@pytest.mark.skipif(not os.path.exists(VENV_RST2PSEUDOXML), reason='docutils not installed')

-@pytest.mark.skipif(

-    not VENV_RST2PSEUDOXML.exists(),

-    reason='docutils not installed',

-)

 @pytest.mark.parametrize('filename', filenames)

 def test_rst_file_syntax(filename):

     p = subprocess.Popen(