all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: zimoun <zimon.toutoune@gmail.com>
To: Efraim Flashner <efraim@flashner.co.il>
Cc: 50490@debbugs.gnu.org
Subject: [bug#50490] [PATCH 0/5] Some julia packages
Date: Mon, 27 Sep 2021 19:11:18 +0200	[thread overview]
Message-ID: <86tui6ymdl.fsf_-_@gmail.com> (raw)
In-Reply-To: <eeb8e3dfcaf3040232e9d53f07ace058f72510bf.1631624546.git.efraim@flashner.co.il> (Efraim Flashner's message of "Tue, 14 Sep 2021 21:19:27 +0300")

Hi Efraim,

On mar., 14 sept. 2021 at 21:19, Efraim Flashner <efraim@flashner.co.il> wrote:

I would add here an explanation, along these lines: «These four
variables are added by the same commit to avoid a cyclic dependency.»

> * gnu/packages/julia-xyz.scm (julia-documenter,
> julia-documentermarkdown, julia-documentertools): New variables.
> (julia-documenter-0.22): New hidden variable.
> ---
>  gnu/packages/julia-xyz.scm | 146 ++++++++++++++++++++++++++++++++++++-
>  1 file changed, 145 insertions(+), 1 deletion(-)

The patch does not apply…

> diff --git a/gnu/packages/julia-xyz.scm b/gnu/packages/julia-xyz.scm
> index 309687019d..e7e7680cca 100644
> --- a/gnu/packages/julia-xyz.scm
> +++ b/gnu/packages/julia-xyz.scm
> @@ -26,7 +26,9 @@ (define-module (gnu packages julia-xyz)
>    #:use-module (guix git-download)
>    #:use-module (guix build-system julia)
>    #:use-module (gnu packages gcc)
> -  #:use-module (gnu packages julia-jll))
> +  #:use-module (gnu packages julia-jll)
> +  #:use-module (gnu packages python)
> +  #:use-module (gnu packages version-control))
>
>  (define-public julia-abstractffts
>    (package
> @@ -990,6 +992,148 @@ (define-public julia-docstringextensions
>  valuable enough at this time.")
>      (license license:expat)))
>
> +;; By removing all the javascript and css downloads any HTML documentation
> +;; produced by this package will not be very useful.
> +(define-public julia-documenter
> +  (package
> +    (name "julia-documenter")
> +    (version "0.27.6")
> +    (source
> +      (origin
> +        (method git-fetch)
> +        (uri (git-reference
> +               (url "https://github.com/JuliaDocs/Documenter.jl")
> +               (commit (string-append "v" version))))
> +        (file-name (git-file-name name version))
> +        (sha256
> +         (base32 "1y6rql7cxc7hfhc8rfq1mdmffp70sqzyh4vnnq93fziwrc8c8sbj"))))
> +    (build-system julia-build-system)
> +    (arguments
> +     `(#:phases
> +       (modify-phases %standard-phases
> +         (add-after 'unpack 'patch-source
> +           (lambda* (#:key inputs #:allow-other-keys)
> +             (substitute* "src/Deps.jl"
> +               (("pip install")
> +                (string-append (assoc-ref inputs "python")
> +                               "/bin/pip install")))
> +             #t))
> +         (add-after 'unpack 'remove-javascript-downloads
> +           (lambda _
> +             (substitute* "src/Writers/HTMLWriter.jl"
> +               (("cdnjs.cloudflare.com") "example.com"))
> +             ;; Removing the javascript downloads causes these tests fail.
> +             (substitute* "test/examples/tests.jl"
> +               ((".*Main\\.examples_html_doc.*") "")
> +               ((".*Main\\.examples_html_mathjax3_doc.*") ""))
> +             #t)))))
> +    (propagated-inputs
> +     `(("julia-ansicoloredprinters" ,julia-ansicoloredprinters)
> +       ("julia-docstringextensions" ,julia-docstringextensions)
> +       ("julia-iocapture" ,julia-iocapture)
> +       ("julia-json" ,julia-json)))
> +    (inputs
> +     `(("python" ,python-wrapper)))
> +    (native-inputs
> +     `(("git" ,git-minimal)
> +       ("julia-documentermarkdown" ,julia-documentermarkdown)
> +       ("julia-documentertools" ,julia-documentertools)))
> +    (home-page "https://juliadocs.github.io/Documenter.jl")
> +    (synopsis "Documentation generator for Julia")
> +    (description "This package provides a documentation generator for Julia.")
> +    (license license:expat)))
> +
> +(define-public julia-documenter-0.22

Do we need to make it public?  Is it not enough to keep it hidden?

> +  (package
> +    (inherit julia-documenter)
> +    (name "julia-documenter")
> +    (version "0.22.6")
> +    (source
> +      (origin
> +        (method git-fetch)
> +        (uri (git-reference
> +               (url "https://github.com/JuliaDocs/Documenter.jl")
> +               (commit (string-append "v" version))))
> +        (file-name (git-file-name name version))
> +        (sha256
> +         (base32
> +          "1z8b7267y7yn5nx8sjwkmc0ph97vmv42q52jg7s89ghqb9xx3wv5"))))
> +    (arguments
> +     `(#:tests? #f      ; Some tests require network.
> +       #:phases
> +       (modify-phases %standard-phases
> +         (add-after 'unpack 'remove-javascript-downloads
> +           (lambda _
> +             ;; This isn't problematic because we only use
> +             ;; this package for bootstrapping.
> +             (substitute* '("assets/html/documenter.js"
> +                            "assets/html/search.js"
> +                            "src/Writers/HTMLWriter.jl")
> +               (("cdnjs.cloudflare.com") "example.com"))
> +             #t)))))
> +    (propagated-inputs
> +     `(("julia-docstringextensions" ,julia-docstringextensions)
> +       ("julia-json" ,julia-json)))
> +    (inputs `())
> +    (native-inputs `())
> +    (properties '((hidden? . #t)))))
> +
> +(define-public julia-documentermarkdown
> +  (package
> +    (name "julia-documentermarkdown")
> +    (version "0.2.2")
> +    (source
> +      (origin
> +        (method git-fetch)
> +        (uri (git-reference
> +               (url "https://github.com/JuliaDocs/DocumenterMarkdown.jl")
> +               (commit (string-append "v" version))))
> +        (file-name (git-file-name name version))
> +        (sha256
> +         (base32 "0sx89hi5p2f8zi2rp5qrv06m270d90pxj5d2y5cxls1spax7wqx8"))))
> +    (build-system julia-build-system)
> +    (inputs
> +     ;; We don't want to propagate the bootstrap version.
> +     ;; Cycle with Documenter.jl in later versions.
> +     `(("julia-documenter" ,julia-documenter-0.22)))
> +    (home-page "https://github.com/JuliaDocs/DocumenterMarkdown.jl")
> +    (synopsis "Documenter's Markdown")
> +    (description "This package enables the Markdown / MkDocs backend of
> +@code{Documenter.jl}.")
> +    (license license:expat)))
> +
> +(define-public julia-documentertools
> +  (package
> +    (name "julia-documentertools")
> +    (version "0.1.13")
> +    (source
> +      (origin
> +        (method git-fetch)
> +        (uri (git-reference
> +               (url "https://github.com/JuliaDocs/DocumenterTools.jl")
> +               (commit (string-append "v" version))))
> +        (file-name (git-file-name name version))
> +        (sha256
> +         (base32 "05p57p8xlkn42m1lv9gq4hl96vp7hpj19d51p828ai1rbpcpi3a6"))))
> +    (build-system julia-build-system)
> +    (arguments
> +     `(#:tests? #f))    ; Tests require network.
> +    (inputs
> +     ;; We don't want to propagate the bootstrap version.
> +     ;; Cycle with Documenter.jl in later versions.
> +     `(("julia-documenter" ,julia-documenter-0.22)))
> +    (propagated-inputs
> +     `(("julia-docstringextensions" ,julia-docstringextensions)
> +       ("julia-gumbo" ,julia-gumbo)
> +       ("julia-sass" ,julia-sass)))
> +    (native-inputs
> +     `(("julia-example" ,julia-example)))
> +    (home-page "https://github.com/JuliaDocs/DocumenterTools.jl")
> +    (synopsis "Extra tools for setting up Documenter")

I would write «setting up Documenter.jl»

> +    (description "This package contains utilities for setting up documentation
> +generation with @code{Documenter.jl}.")
> +    (license license:expat)))
> +
>  (define-public julia-diffresults
>    (package
>      (name "julia-diffresults")

…so I have not tried it.  It LGTM though.

Cheers,
simon




  reply	other threads:[~2021-09-27 17:24 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-09-14 13:08 [bug#50582] [PATCH 0/4] Julia-documenter Efraim Flashner
2021-09-14 13:11 ` [bug#50582] [PATCH 1/4] gnu: Add julia-libsass-jll Efraim Flashner
2021-09-27 17:01   ` [bug#50490] [PATCH 0/5] Some julia packages zimoun
2021-09-14 13:12 ` [bug#50582] [PATCH 2/4] gnu: Add julia-sass Efraim Flashner
2021-09-27 17:03   ` [bug#50490] [PATCH 0/5] Some julia packages zimoun
2021-09-14 18:19 ` [bug#50490] [PATCH 3/4] gnu: Add julia-ansicoloredprinters Efraim Flashner
2021-09-27 17:04   ` [bug#50490] [PATCH 0/5] Some julia packages zimoun
2021-09-14 18:19 ` [bug#50490] [PATCH 4/4] gnu: Add julia-documenter Efraim Flashner
2021-09-27 17:11   ` zimoun [this message]
2021-09-30 11:18     ` [bug#50490] [PATCH 0/5] Some julia packages Efraim Flashner
     [not found] ` <handler.50582.B.163162505210361.ack@debbugs.gnu.org>
2021-09-14 18:31   ` [bug#50582] Acknowledgement ([PATCH 0/4] Julia-documenter) Efraim Flashner
2021-09-27 17:15     ` [bug#50582] [PATCH 0/4] Julia-documenter zimoun
2021-09-29  7:21       ` Efraim Flashner
2021-09-30 11:26       ` bug#50582: " Efraim Flashner
  -- strict thread matches above, loose matches on Subject: below --
2021-09-09 18:16 [bug#50490] [PATCH 0/5] Some julia packages Efraim Flashner
2021-09-14 17:38 ` zimoun
2021-09-22 12:13 ` [bug#50490] [PATCH 1/5] gnu: Add julia-quadgk Efraim Flashner
2021-09-27 16:51   ` [bug#50490] [PATCH 0/5] Some julia packages zimoun
2021-09-22 12:13 ` [bug#50490] [PATCH 2/5] gnu: Add julia-measurements Efraim Flashner
2021-09-27 16:53   ` [bug#50490] [PATCH 0/5] Some julia packages zimoun
2021-09-22 12:13 ` [bug#50490] [PATCH 3/5] gnu: Add julia-mlstyle Efraim Flashner
2021-09-27 16:54   ` [bug#50490] [PATCH 0/5] Some julia packages zimoun
2021-09-22 12:13 ` [bug#50490] [PATCH 4/5] gnu: Add julia-pycall Efraim Flashner
2021-09-27 16:58   ` [bug#50490] [PATCH 0/5] Some julia packages zimoun
2021-09-22 12:13 ` [bug#50490] [PATCH 5/5] gnu: Add julia-ansicoloredprinters Efraim Flashner
2021-09-27 16:59   ` [bug#50490] [PATCH 0/5] Some julia packages zimoun

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=86tui6ymdl.fsf_-_@gmail.com \
    --to=zimon.toutoune@gmail.com \
    --cc=50490@debbugs.gnu.org \
    --cc=efraim@flashner.co.il \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.