From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:52084) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1fXcSl-0004LL-8O for guix-patches@gnu.org; Mon, 25 Jun 2018 21:08:08 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1fXcSg-000418-6r for guix-patches@gnu.org; Mon, 25 Jun 2018 21:08:07 -0400 Received: from debbugs.gnu.org ([208.118.235.43]:55822) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1fXcSg-00040N-1L for guix-patches@gnu.org; Mon, 25 Jun 2018 21:08:02 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1fXcSf-0002PD-IC for guix-patches@gnu.org; Mon, 25 Jun 2018 21:08:01 -0400 Subject: [bug#31940] [PATCH] gnu: Add Stalin. Resent-Message-ID: From: Adam Massmann References: <878t76655l.fsf@gmail.com> <87efgur4cv.fsf@gnu.org> Date: Mon, 25 Jun 2018 21:07:29 -0400 In-Reply-To: <87efgur4cv.fsf@gnu.org> ("Ludovic =?UTF-8?Q?Court=C3=A8s?="'s message of "Mon, 25 Jun 2018 22:36:00 +0200") Message-ID: <87y3f2we26.fsf@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable 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: Ludovic =?UTF-8?Q?Court=C3=A8s?= Cc: 31940-done@debbugs.gnu.org Hi Ludo, ludo@gnu.org (Ludovic Court=C3=A8s) writes: > I applied the patch, but I added a FIXME about the fact that it=E2=80=99s > =E2=80=9Cyogurt software=E2=80=9D as Ricardo says: its =E2=80=9Csource=E2= =80=9D includes C files > generated by itself, which are obviously not source code. We try to > avoid that, but OTOH we already have the same problem for several > compilers, for instance MIT/GNU=C2=A0Scheme. > > From the README, previous versions were apparently bootstrapped using > a separate Scheme->C compiler. We could perhaps fix the problem by > building one of these older versions of Stalin, and using it to build > the latest one. Would you like to give it a try? (On this topic, > don=E2=80=99t miss .) Thanks a lot for the code review and feedback - it's very helpful for me as I learn. At first glance I think in this case, as you point out, we should be able to use Scheme->C to generate the C files explicitly in the build process. Also, some of the documentation suggests that upstream *might* maintain a Scheme->C version for v0.11 and be willing to share it, which would make it even easier. I put it on my TODO list to dig in and try to fix it, but realistically might not get to it until August due to work. Bootstrapping the build should also extend the package to other architectures, which was something I also wanted to do at some point. > Anyway, thanks for the patch! Thank you!