Subject: Re: [bug#33534] [PATCH] gnu: Add arcanist.
From: [email-masked] (Ludovic Courtès)
To: Robin Templeton <[email-masked]>
Cc: [email-masked]
Date: Thu, 29 Nov 2018 10:00:35 +0100
List-Id: <debbugs-submit.debbugs.gnu.org>

Hello Robin,

Robin Templeton <[email-masked]> skribis:

> * gnu/packages/phabricator.scm: New file.
> * gnu/local.mk (GNU_SYSTEM_MODULES): Add it.

Applied with minor changes to the description (attached), mostly to give
some context to those who do not know what Phabricator is.

BTW we traditionally write one patch for each new package, but I think
it’s okay here.

> +      (home-page "https://github.com/phacility/libphutil")
> +      (synopsis "PHP utility library")
> +      (description
> +       "@code{libphutil} is a collection of utility classes and functions for
> +PHP.")
> +      ;; Bundled libraries are expat-licensed.
> +      (license (list license:asl2.0 license:expat)))))

Do you think those libraries could be unbundled?  Would it make sense?

Thank you!

Ludo’.

diff --git a/gnu/packages/phabricator.scm b/gnu/packages/phabricator.scm
index 6834373fbc..6c94a13114 100644
--- a/gnu/packages/phabricator.scm
+++ b/gnu/packages/phabricator.scm
@@ -117,7 +117,8 @@ PHP.")
       (home-page "https://github.com/phacility/arcanist")
       (synopsis "Command-line interface for Phabricator")
       (description
-       "Arcanist is the command-line tool for Phabricator.  It allows you to
-interact with Phabricator installs to send code for review, download patches,
-transfer files, view status, make API calls, and various other things.")
+       "Arcanist is the command-line tool for the Phabricator software
+development service.  It allows you to interact with Phabricator installs to
+send code for review, download patches, transfer files, view status, make API
+calls, and various other things.")
       (license license:asl2.0))))