[bug#38390] Gash 0.2.0

[Jan Nieuwenhuizen <janneke@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 (= 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’ve
> attached my package definition for reference (it belongs in
> “commencement.scm”).  The reason I say this is to reassure you that
> going from 2.0.14 to 2.0.9 shouldn’t 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 “setlocale” to normalize some
> differences between Guile 2 and Guile 2.2 (I was getting test failures
> without it).  From reading Guile’s NEWS, version 2.2 started
> initializing the locale automatically, so I “backported” that to Guile 2
> by calling “setlocale” 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 “(setlocale LC_ALL "C")” if there is a
> problem.  I’ll 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 – I
> don’t recall).  The commit message is “No more unexpected shift-reduce
> conflicts with LR driver.”  See <https://github.com/schemeway/lalr-scm>.
> 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

-- 
Jan Nieuwenhuizen <janneke@gnu.org> | GNU LilyPond http://lilypond.org
Freelance IT http://JoyofSource.com | Avatar® http://AvatarAcademy.com