From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:60009)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <Debian-debbugs@debbugs.gnu.org>) id 1dF2Cc-0002EL-El
	for guix-patches@gnu.org; Sun, 28 May 2017 13:42:07 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <Debian-debbugs@debbugs.gnu.org>) id 1dF2CY-0008FO-Hr
	for guix-patches@gnu.org; Sun, 28 May 2017 13:42:06 -0400
Received: from debbugs.gnu.org ([208.118.235.43]:39263)
	by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16)
	(Exim 4.71) (envelope-from <Debian-debbugs@debbugs.gnu.org>)
	id 1dF2CY-0008FK-Eg
	for guix-patches@gnu.org; Sun, 28 May 2017 13:42:02 -0400
Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2)
	(envelope-from <Debian-debbugs@debbugs.gnu.org>) id 1dF2CY-0003mm-5O
	for guix-patches@gnu.org; Sun, 28 May 2017 13:42:02 -0400
Subject: bug#27049: [PATCH] gnu: Add mathjax.
Resent-Message-ID: <handler.27049.B27049.149599330214524@debbugs.gnu.org>
From: ludo@gnu.org (Ludovic =?UTF-8?Q?Court=C3=A8s?=)
References: <15cc163d.ADsAAWz_hBsAAAAAAAAAAAOtUOAAAAACwQwAAAAAAAW9WABZJK0O@mailjet.com>
	<3dec65bd-ffb9-ad71-8c0c-c374fd342ab8@openmailbox.org>
Date: Sun, 28 May 2017 19:41:28 +0200
In-Reply-To: <3dec65bd-ffb9-ad71-8c0c-c374fd342ab8@openmailbox.org> (Brendan
	Tildesley's message of "Thu, 25 May 2017 00:47:09 +1000")
Message-ID: <87tw44ri9j.fsf@gnu.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable
List-Id: <guix-patches.gnu.org>
List-Unsubscribe: <https://lists.gnu.org/mailman/options/guix-patches>,
	<mailto:guix-patches-request@gnu.org?subject=unsubscribe>
List-Archive: <http://lists.gnu.org/archive/html/guix-patches/>
List-Post: <mailto:guix-patches@gnu.org>
List-Help: <mailto:guix-patches-request@gnu.org?subject=help>
List-Subscribe: <https://lists.gnu.org/mailman/listinfo/guix-patches>,
	<mailto:guix-patches-request@gnu.org?subject=subscribe>
Errors-To: guix-patches-bounces+kyle=kyleam.com@gnu.org
Sender: "Guix-patches" <guix-patches-bounces+kyle=kyleam.com@gnu.org>
To: Brendan Tildesley <brendan.tildesley@openmailbox.org>
Cc: 27049@debbugs.gnu.org

Hello,

Brendan Tildesley <brendan.tildesley@openmailbox.org> skribis:

> Arun Isaac =E6=96=BC 2017-05-24 07:43 =E5=AF=AB=E9=81=93:
>> * gnu/packages/web.scm (mathjax): New variable.
>> [...]
>> +
>> +         (let ((install-dir (string-append %output "/share/webapps/math=
jax"))
>> +               (tar (string-append (assoc-ref %build-inputs "tar") "/bi=
n/tar"))
>> +               (path (string-append (assoc-ref %build-inputs "gzip") "/=
bin")))
>> +           (mkdir-p install-dir)
>> +           (chdir install-dir)
>> +           (setenv "PATH" path)
>> +           (system* tar "xvf" (assoc-ref %build-inputs "source") "--str=
ip" "1")))))
> This is installing many extraneous files like .gitignore, .npmignore,
> README.md, CONTRIBUTING, LICENSE, docs/, test/, latest.js. It may be
> best start with an 'unpack phase, and then install the needed files to
> the output directory.
> This package is around 178MiB like it is, and most of that due to the
> fonts. Would it be better to break this into two packages, mathjax, and
> font-mathjax?

Also, Arun: is this package really source code?  Or does it contain
=E2=80=9Cminified=E2=80=9D code and also bundles all its dependencies (whic=
h would not
be okay)?

Thanks,
Ludo=E2=80=99.