[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 15/16] gnu: python-ipython: Update to 4.0.0.
From: |
Ricardo Wurmus |
Subject: |
Re: [PATCH 15/16] gnu: python-ipython: Update to 4.0.0. |
Date: |
Fri, 4 Nov 2016 18:10:23 +0100 |
User-agent: |
mu4e 0.9.16; emacs 26.0.50.1 |
Roel Janssen <address@hidden> writes:
> Ricardo Wurmus writes:
>
>> * gnu/packages/python.scm (python-ipython): Update to 4.0.0.
>> [inputs]: Keep only "readline" and "which"; move the remaining inputs
>> to propagated-inputs, except for "python-requests" and "python-nose"
>> which are moved to native-inputs.
>> [propagated-inputs]: Add python-pexpect, python-pickleshare,
>> python-simplegeneric, python-traitlets, python-ipykernel.
>> [native-inputs]: Add "python-testpath".
>> [arguments]: Enable building of HTML documentation.
I noticed that I also needed to delete the patch and report it here.
Also fixed the indentation.
>> + (inputs
>> + `(("readline" ,readline)
>> + ("which" ,which)))
>
> I think it really is necessary to propagate these, isn't it?
Neither of these need propagation. “readline” is checked for by
setup.py, and for “which” we have a build phase to embed the reference
to the “which” executable.
>> (native-inputs
>> `(("pkg-config" ,pkg-config)
>> + ("python-requests" ,python-requests) ;; for tests
>> + ("python-testpath" ,python-testpath)
>> + ("python-nose" ,python-nose)
>> ("python-sphinx" ,python-sphinx)
>> ("texlive" ,texlive)
>> ("texinfo" ,texinfo)
>> @@ -4674,13 +4678,13 @@ tools for mocking system commands and recording
>> calls to those.")
>> (examples (string-append doc "/examples")))
>> (setenv "LANG" "en_US.utf8")
>> (with-directory-excursion "docs"
>> - ;; FIXME: html and pdf fail to build
>> - ;; (system* "make" "html")
>> - ;; (system* "make" "pdf" "PAPER=a4")
>> + ;; FIXME: pdf fails to build
>> + ;;(system* "make" "pdf" "PAPER=a4")
>> + (system* "make" "html")
>
> The last line isn't absolutely necessary to, but I it doesn't really
> matter and I see why you'd want to not have a space there.
> Nevertheless, you could shorten the patch by two lines if you feel like
> doing so.. :)
Are you saying that “make html” isn’t needed to build the html docs? I
think it is.
~~ Ricardo
[PATCH 16/16] gnu: Add jupyter., Ricardo Wurmus, 2016/11/01
Re: [PATCH 00/16] Add Jupyter., Roel Janssen, 2016/11/01