From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:470:142:3::10]:56877) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1ioei1-00030x-5U for guix-patches@gnu.org; Mon, 06 Jan 2020 21:35:06 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ioehz-0004Sh-71 for guix-patches@gnu.org; Mon, 06 Jan 2020 21:35:05 -0500 Received: from debbugs.gnu.org ([209.51.188.43]:41121) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1ioehy-0004Ro-9c for guix-patches@gnu.org; Mon, 06 Jan 2020 21:35:03 -0500 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1ioehy-0002F5-2G for guix-patches@gnu.org; Mon, 06 Jan 2020 21:35:02 -0500 Subject: [bug#38965] [PATCH 00/12] gnu: coq: Update to 8.10.2. Resent-Message-ID: Date: Mon, 06 Jan 2020 21:34:34 -0500 In-Reply-To: <87o8vgm37i.fsf@gnu.org> References: <874kx9xa9u.fsf@gnu.org> <53E54994-18D2-4D82-8E96-3D6E8A3A9D14@lepiller.eu> <87o8vgm37i.fsf@gnu.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable From: Julien Lepiller Message-ID: <2E36EFCF-C7C0-442B-8F03-46CC3167DA92@lepiller.eu> List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-patches-bounces+kyle=kyleam.com@gnu.org Sender: "Guix-patches" To: Brett Gilio Cc: 38965@debbugs.gnu.org Le 6 janvier 2020 21:04:49 GMT-05:00, Brett Gilio a =C3= =A9crit : >Julien Lepiller writes: > >> Le 6 janvier 2020 03:24:29 GMT-05:00, Brett Gilio a >=C3=A9crit : >>> >> >> Looks like lablgtk2 -> ocaml-cairo2 -> lablgtk3 so we can't get rid >of lablgtk2=2E Are we sure we need it though? >> >> In general, make sure to run guix lint on these patches, I could spot >missing double spaces in descriptions of the first two for instance=2E > >The OPAM page for cairo2 does not mention needing lablgtk2=2E So, if that >is the case I imagine we could be fine deprecating it in favor of >lablgtk3=2E > >I forgot to lint those who packages, yes=2E I can change those before >pushing them=2E How does everything else look? The rest looks pretty good :)=2E The introduction of (gnu packages autotoo= ls) is too late though, it should be added on the first patch that needs it= (5 I think)=2E