Danny Milosavljevic <dan...@scratchpost.org> writes: > * gnu/packages/python.scm (python-dulwich, python2-dulwich): New variables. > --- > gnu/packages/python.scm | 44 ++++++++++++++++++++++++++++++++++++++++++++ > 1 file changed, 44 insertions(+) > > diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm > index 6c7a6353d..b064c9f47 100644 > --- a/gnu/packages/python.scm > +++ b/gnu/packages/python.scm > @@ -12740,3 +12740,47 @@ Features: > @item Compiles templates into optimized, yet readable, Python code. > @end enumerate") > (license (license:x11-style "file://LICENSE")))) > + > +(define-public python-dulwich > + (package > + (name "python-dulwich") > + (version "0.16.3") > + (source > + (origin > + (method url-fetch) > + (uri (pypi-uri "dulwich" version))
Upstream provides the same releaases on their home page. I would add (uri (list (string-append "https://www.dulwich.io/releases/dulwich-" version ".tar.gz") (pypi-uri "cram" version))) > + (sha256 > + (base32 "0fl47vzfgc3w3rmhn8naii905cjqcp0vc68iyvymxp7567hh6als")))) > + (build-system python-build-system) > + (arguments > + `(#:phases > + (modify-phases %standard-phases > + (add-before 'check 'fix-tests > + (lambda* (#:key inputs #:allow-other-keys) > + (substitute* "dulwich/tests/compat/utils.py" > + (("'git'") (string-append "'" > + (assoc-ref inputs "git") > + "/bin/git'"))) Why is this necessary? Git is available in PATH, no? > + (substitute* '("dulwich/tests/test_repository.py" > + "dulwich/tests/test_hooks.py") > + (("#!/bin/sh") (string-append "#!" (which "sh")))) > + (setenv "TEST_RUNNER" "unittest") > + (setenv "PYTHONHASHSEED" "random") > + #t))))) > + (propagated-inputs > + `(("python-fastimport" ,python-fastimport))) > + (native-inputs > + `(("python-mock" ,python-mock) > + ("python-geventhttpclient" ,python-geventhttpclient) > + ("git" ,git))) > + (home-page "https://www.dulwich.io/") > + (synopsis "Git implementation in Python") > + (description "Dulwich is an implementation of the Git file formats and > +protocols written in pure Python.") > + ;; dual-licensed > + (license (list license:asl2.0 license:gpl2+)))) Please clarify whether it's both, or either. Otherwise LGTM, thanks!
signature.asc
Description: PGP signature