all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Sarah Morgensen <iskarian@mgsn.dev>
To: Stephen Paul Weber <singpolyma@singpolyma.net>
Cc: 50793@debbugs.gnu.org
Subject: [bug#50793] [PATCH 2/2] gnu: Add cloudflare-cli
Date: Fri, 24 Sep 2021 19:58:54 -0700	[thread overview]
Message-ID: <86ilypwec1.fsf@mgsn.dev> (raw)
In-Reply-To: <20210925013934.2763-2-singpolyma@singpolyma.net> (Stephen Paul Weber's message of "Fri, 24 Sep 2021 20:39:34 -0500 (1 hour, 5 minutes, 51 seconds ago)")

Hi,

I've (naturally) got a few suggestions for this one, too.

Stephen Paul Weber <singpolyma@singpolyma.net> writes:

> * gnu/packages/dns.scm (cloudflare-cli): New variable.
> ---
>  gnu/packages/dns.scm | 52 ++++++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 52 insertions(+)
>
> diff --git a/gnu/packages/dns.scm b/gnu/packages/dns.scm
> index 45e250138e..9e68d76a93 100644
> --- a/gnu/packages/dns.scm
> +++ b/gnu/packages/dns.scm
> @@ -42,6 +42,7 @@
>    #:use-module (gnu packages bash)
>    #:use-module (gnu packages certs)
>    #:use-module (gnu packages check)
> +  #:use-module (gnu packages curl)
>    #:use-module (gnu packages databases)
>    #:use-module (gnu packages documentation)
>    #:use-module (gnu packages compression)
> @@ -69,6 +70,7 @@
>    #:use-module (gnu packages python)
>    #:use-module (gnu packages python-xyz)
>    #:use-module (gnu packages ragel)
> +  #:use-module (gnu packages serialization)
>    #:use-module (gnu packages shells)
>    #:use-module (gnu packages sphinx)
>    #:use-module (gnu packages swig)
> @@ -82,11 +84,61 @@
>    #:use-module (guix download)
>    #:use-module (guix git-download)
>    #:use-module (guix utils)
> +  #:use-module (guix build-system copy)
>    #:use-module (guix build-system glib-or-gtk)
>    #:use-module (guix build-system gnu)
>    #:use-module (guix build-system meson)
>    #:use-module (guix build-system trivial))
>  
> +(define-public cloudflare-cli
> +  (package
> +    (name "cloudflare-cli")
> +    (version "2d986d3")

Please use the 'git-version' pattern as I mentioned in the other patch.

> +    (source
> +     (origin
> +       (method git-fetch)
> +       (uri (git-reference
> +             (url "https://github.com/earlchew/cloudflare-cli")
> +             (commit version)))
> +       (file-name (git-file-name name version))
> +       (sha256
> +        (base32
> +         "0f86g6n86kwykl3jnhqjrdfy8ybkp03ghr3dlr70q2552qw4axw2"))))
> +    (build-system copy-build-system)
> +    (arguments
> +     `(#:install-plan '(("cloudflare-cli" "bin/") ("cloudflare-cli.sh" "bin/"))
> +       #:phases
> +       (modify-phases %standard-phases
> +         (add-after 'unpack 'find-jsonsh
> +           (lambda* (#:key inputs #:allow-other-keys)
> +             (substitute* "cloudflare-cli.sh"
> +               (("\\$\\{0%/\\*\\}/jsonsh")
> +                (string-append (assoc-ref inputs "JSON.sh") "/bin/JSON.sh")))
> +             #t))
> +         (add-after 'install 'wrap-program
> +           (lambda* (#:key inputs outputs #:allow-other-keys)
> +             (wrap-program (string-append (assoc-ref outputs "out") "/bin/cloudflare-cli")
> +               `("PATH" ":" prefix
> +                 (,(string-join
> +                    (map (lambda (in) (string-append (assoc-ref inputs in) "/bin"))
> +                         '("grep" "curl" "coreutils"))

I don't think this uses anything from coreutils either, but I just took
a quick look.

> +                    ":")))))))))
> +    (inputs
> +     `(("bash-minimal" ,bash-minimal)
> +       ("coreutils" ,coreutils)
> +       ("curl" ,curl)
> +       ("grep" ,grep)
> +       ("JSON.sh" ,JSON.sh)))
> +    (synopsis
> +      "CLI to edit Cloudflare DNS records")
> +    (description
> +      "This command line tool to update Cloudfare DNS records is useful in the
> +following scenarios:
> +* Keeping dynamic DNS records up to date
> +* Updating DNS records as part of the ACME DNS-01 protocol")

Could you update this description to use sentences rather than bullets
(and, preferably, use active rather than passive verbiage)?

(If you do ever need to write a list in the description, you can use
@itemize (info "(texinfo) Lists and Tables"); see other packages for
examples.)

> +    (home-page "https://github.com/earlchew/cloudflare-cli")
> +    (license license:expat)))
> +
>  (define-public ldns
>    (package
>      (name "ldns")

Thank you!

--
Sarah




  reply	other threads:[~2021-09-25  2:59 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-09-25  1:38 [bug#50793] [PATCH 0/2] cloudflare-cli and dependency JSON.sh Stephen Paul Weber
2021-09-25  1:39 ` [bug#50793] [PATCH 1/2] gnu: Add JSON.sh Stephen Paul Weber
2021-09-25  1:39   ` [bug#50793] [PATCH 2/2] gnu: Add cloudflare-cli Stephen Paul Weber
2021-09-25  2:58     ` Sarah Morgensen [this message]
2021-09-26  1:21       ` Stephen Paul Weber
2021-09-25  2:58   ` [bug#50793] [PATCH 1/2] gnu: Add JSON.sh Sarah Morgensen
2021-09-26  1:08     ` Stephen Paul Weber
2021-09-26  3:02       ` Sarah Morgensen
2021-09-26 23:32         ` Stephen Paul Weber
2021-09-26  1:29 ` [bug#50793] [PATCH v2 " Stephen Paul Weber
2021-09-26  1:29   ` [bug#50793] [PATCH v2 2/2] gnu: Add cloudflare-cli Stephen Paul Weber
2021-09-26 23:35 ` [bug#50793] [PATCH v3 1/2] gnu: Add JSON.sh Stephen Paul Weber
2021-09-26 23:35   ` [bug#50793] [PATCH v3 2/2] gnu: Add cloudflare-cli Stephen Paul Weber
2021-10-13  9:08     ` bug#50793: [PATCH 0/2] cloudflare-cli and dependency JSON.sh Ludovic Courtès
2021-10-13  9:05   ` [bug#50793] " Ludovic Courtès
2021-10-13 19:03     ` Maxim Cournoyer
2021-10-14  8:54       ` zimoun
2021-10-15  3:26         ` Maxim Cournoyer
2021-10-15 15:28       ` Ludovic Courtès

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=86ilypwec1.fsf@mgsn.dev \
    --to=iskarian@mgsn.dev \
    --cc=50793@debbugs.gnu.org \
    --cc=singpolyma@singpolyma.net \
    /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.