From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:50884)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <Debian-debbugs@debbugs.gnu.org>) id 1dB55D-0003O4-60
	for guix-patches@gnu.org; Wed, 17 May 2017 15:58:08 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <Debian-debbugs@debbugs.gnu.org>) id 1dB558-0008AK-IV
	for guix-patches@gnu.org; Wed, 17 May 2017 15:58:07 -0400
Received: from debbugs.gnu.org ([208.118.235.43]:48675)
	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 1dB558-0008AE-FK
	for guix-patches@gnu.org; Wed, 17 May 2017 15:58:02 -0400
Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2)
	(envelope-from <Debian-debbugs@debbugs.gnu.org>) id 1dB558-00068I-9H
	for guix-patches@gnu.org; Wed, 17 May 2017 15:58:02 -0400
Subject: bug#25836: [PATCH] gnu: Add gspell
Resent-Message-ID: <handler.25836.B25836.149505103023517@debbugs.gnu.org>
From: ludo@gnu.org (Ludovic =?UTF-8?Q?Court=C3=A8s?=)
References: <87wpbx9z7b.fsf@gnu.org> <cuc1stfht92.fsf@thomasdanckaert.be>
	<87a86bwoh9.fsf@gnu.org>
	<20170517.171626.2148481571502189736.post@thomasdanckaert.be>
Date: Wed, 17 May 2017 21:56:55 +0200
In-Reply-To: <20170517.171626.2148481571502189736.post@thomasdanckaert.be>
	(Thomas Danckaert's message of "Wed, 17 May 2017 17:16:26 +0200
	(CEST)")
Message-ID: <87o9uruuig.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: Thomas Danckaert <post@thomasdanckaert.be>
Cc: 25836@debbugs.gnu.org

Thomas Danckaert <post@thomasdanckaert.be> skribis:

> From: ludo@gnu.org (Ludovic Court=C3=A8s)
> Subject: Re: bug#25836: [PATCH] gnu: Add gspell
> Date: Wed, 17 May 2017 16:24:18 +0200
>
>>> I'm not really familiar with the gobject system, but I suppose
>>> generating the bindings is useful? :)
>>
>> For now we don=E2=80=99t have any users of gspell at all, let alone user=
s of
>> its
>> Vala bindings, so perhaps we can leave it as is?
>>
>> Building the docs to a separate =E2=80=9Cdoc=E2=80=9D output could be ma=
rginally
>> useful,
>> I think.
>
> For me, it's me as it is.

Hi! Is it you?  :-)

> I have no use for those bindings or the documentation myself.  I just
> stumbled upon them while trying to debug the package, and thought I'd
> mention it just in case.
>
> When I submit a package, I usually try to enable as many options as I
> can, even though I don't use them myself, just in case someone might
> need them.  Maybe this is overkill?  Is there a general recommendation
> for Guix?

It depends!  The recommendation is to check what =E2=80=98guix size=E2=80=
=99 says and
weigh the usefulness/size ratio of the added dependency:

  https://www.gnu.org/software/guix/manual/html_node/Submitting-Patches.html

Ludo=E2=80=99.