From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:470:142:3::10]:35931) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1ieCgF-0005BK-VP for guix-patches@gnu.org; Mon, 09 Dec 2019 01:38:05 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ieCgE-0002QI-Is for guix-patches@gnu.org; Mon, 09 Dec 2019 01:38:03 -0500 Received: from debbugs.gnu.org ([209.51.188.43]:46547) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1ieCgE-0002Q5-Er for guix-patches@gnu.org; Mon, 09 Dec 2019 01:38:02 -0500 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1ieCgE-0002JZ-BU for guix-patches@gnu.org; Mon, 09 Dec 2019 01:38:02 -0500 Subject: [bug#38390] Gash 0.2.0 Resent-Message-ID: From: Jan Nieuwenhuizen References: <87muc8g68f.fsf@ngyro.com> <87eexk1mag.fsf@gnu.org> <878snped7a.fsf@ngyro.com> <87tv6bv9ds.fsf@ngyro.com> <87d0cyh96s.fsf@gnu.org> <877e36v56p.fsf@ngyro.com> Date: Mon, 09 Dec 2019 07:37:29 +0100 In-Reply-To: <877e36v56p.fsf@ngyro.com> (Timothy Sample's message of "Sun, 08 Dec 2019 17:11:10 -0500") Message-ID: <878snmgg2e.fsf@gnu.org> 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: Timothy Sample Cc: 38390@debbugs.gnu.org Timothy Sample writes: Hi Timothy, >> I just found two things I'm not too happy about. Both stem from using >> our bootstrap guile-2.0.9. Previously, I used guile-2.0 (=3D 2.0.14) to >> compile the .go files that were later used during bootstrap. > > I did all of my bootstrapping tests with the bootstrap Guile. I=E2=80=99= ve > attached my package definition for reference (it belongs in > =E2=80=9Ccommencement.scm=E2=80=9D). The reason I say this is to reassur= e you that > going from 2.0.14 to 2.0.9 shouldn=E2=80=99t be too disruptive. Ah, that's great. >> then I (of course) still get the Shift/Reduce warnings and everything >> "seems" to work. It seems that the empty locale is problematic for >> guile 2.0.9 or our bootstrap environment. I just tried a patch and >> pushed to my wip-0.2.0. > > I added (at the last minute) the call to =E2=80=9Csetlocale=E2=80=9D to n= ormalize some > differences between Guile 2 and Guile 2.2 (I was getting test failures > without it). From reading Guile=E2=80=99s NEWS, version 2.2 started > initializing the locale automatically, so I =E2=80=9Cbackported=E2=80=9D = that to Guile 2 > by calling =E2=80=9Csetlocale=E2=80=9D with an empty string (as discussed= in the > manual). Anyway, this is definitely my fault! Perhaps it should try > that and then fall back on =E2=80=9C(setlocale LC_ALL "C")=E2=80=9D if th= ere is a > problem. I=E2=80=99ll figure it out before releasing. OK, thanks. >> I am less cheerful about the Shift/Reduce warnings; I am afraid they >> might break something somewhere during the bootstrap build. Any ideas >> why these surface at this time? I'll play some more with >> lalr.upstream.scm versions... > > My understanding is that these are spurious. That's what I was hoping...although > There is upstream commit > 92b64f9b3c02b0086a4aea7cbe7dc38376768150 which is not included in Guile > 2.0.9 but included in later versions (maybe starting with 2.0.10 =E2=80= =93 I > don=E2=80=99t recall). The commit message is =E2=80=9CNo more unexpected= shift-reduce > conflicts with LR driver.=E2=80=9D See . > Since the newer versions are fine with the parser, and I have parsed > oodles of shell script using Guile 2.0.9, I think the these warnings are > just the result of a bug. :) ...if something in our bootstrap build is asserting that stderr is empty, they might break on these spurious warnings? Would it be a good idea to include this fixed lalr when the guile version is too old? Note that in the bootstrap build, I am using the `guile-build-system' (no sh or make yet when bootstrapping our first gash :-). > -- Tim > > (define gash-boot Ah, nice; thanks. Greetings, janneke --=20 Jan Nieuwenhuizen | GNU LilyPond http://lilypond.org Freelance IT http://JoyofSource.com | Avatar=C2=AE http://AvatarAcademy.com