Re: [PATCH 6/9] gnu: Add python-pytest-httpbin-0.0.7.

[ng0 <contact.ng0@cryptolab.net>]

On 17-02-05 15:22:20, Maxim Cournoyer wrote:
> contact.ng0@cryptolab.net writes:
> 
> > From: ng0 <ng0@libertad.pw>
> >
> > * gnu/packages/python.scm (python-pytest-httpbin-0.0.7): New variable.
> > ---
> >  gnu/packages/python.scm | 23 +++++++++++++++++++++++
> >  1 file changed, 23 insertions(+)
> >
> > diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm
> > index 9c3a031e1..7bdd371b5 100644
> > --- a/gnu/packages/python.scm
> > +++ b/gnu/packages/python.scm
> > @@ -1975,6 +1975,29 @@ thread running httpbin and provides your test with the URL in the fixture.")
> >  (define-public python2-pytest-httpbin
> >    (package-with-python2 python-pytest-httpbin))
> >  
> > +;; Some packages like "searx" need this old version.
> > +(define-public python-pytest-httpbin-0.0.7
> > +  (package
> > +    (inherit python-pytest-httpbin)
> > +    (version "0.0.7")
> > +    (source
> > +     (origin
> > +       (method url-fetch)
> > +       (uri (pypi-uri "pytest-httpbin" version))
> > +       (sha256
> > +        (base32
> > +         "08ghq923dn33rllip3vap2p9fb680g0i96jdn5lcpfy8amq8mbq3"))))))
> > +
> > +(define-public python2-pytest-httpbin-0.0.7
> > +  (package
> > +    (inherit (package-with-python2
> > +              (strip-python2-variant python-pytest-httpbin-0.0.7)))
> > +    (inputs
> > +     `(("python2-flask" ,python2-flask)
> > +       ("python2-decorator" ,python2-decorator)
> > +       ("python2-httpbin" ,python2-httpbin)
> > +       ("python2-six" ,python2-six)))))
> > +
> 
> Again, shouldn't those be propagated-inputs? Unless I missed something
> this was the recommendation going forward for Python runtime dependencies.
> 
> >  (define-public python-scripttest
> >    (package
> >      (name "python-scripttest")

Definitely. These patches predate the proposed changes and I didn't
check all of them.

I don't have the time to apply changes this work week. I'll see to apply
changes on sunday or monday.

Any reviews etc in the next days are considered with the next patch
submission.

Thanks!
-- 
ng0 -- https://www.inventati.org/patternsinthechaos/