Re: ☠ Buildbot (Sourceware): glibc-snapshots-trunk - failed compile (failure) (master)

[Mark Wielaard <mark@klomp.org>]

Hi Andreas,

On Fri, Apr 26, 2024 at 10:25:23AM +1100, Andreas K. Huettel wrote:
> > > diff --git a/INSTALL b/INSTALL
> > > index 2f2d12c335..1047585695 100644
> > > --- a/INSTALL
> > > +++ b/INSTALL
> > > @@ -21,11 +21,11 @@ allows removing the whole build directory in case
> > > an error occurs, which
> > >  is the safest way to get a fresh start and should always be done.
> > > 
> > >     From your object directory, run the shell script ‘configure’ located
> > > -at the top level of the source tree.  In the scenario above, you’d type
> > > 
> > >                             ^
> > > +at the top level of the source tree.  In the scenario above, you'd type
> > > 
> > >                             ^
> > > 
> > 
> > I had to look 6 times till I saw the difference between the you’d and
> > you'd :)
> > 
> > Does anybody know what (which package upgrade) exactly caused this? And
> > what is "correct" in this case?
> 
> In general that's texinfo, and the equivalent change was introduced elsewhere in 
> (from memory) 2.38. 
> 
> commit 84e93afc734a3c30e35ed2d21466a44259ac577e
> Author: Paul Eggert <eggert@cs.ucla.edu>
> Date:   Thu Jun 29 09:20:41 2023 -0700
>     Switch to UTF-8 for INSTALL
>     This makes it slightly easier to read, and these days
>     everybody can read UTF-8.
> 
> Now for what changed precisely in Fedora 40 you'll have to ask someone else...

Aha, thanks. So it depends on whether --disable-encoding is given to
makeinfo.

So this seems a slight difference between texinfo 7.0.3 (Fedora 39)
and texinfo 7.1 (Fedora 40). With 7.1 some single quotes ('), like in
"Don't" are no longer translated as open-quotes (’).

I don't fully understand what the change was. It seems 7.1 has
ASCII_DASHES_AND_QUOTES default to on. But I don't understand how to
turn that off.

> > Maybe we should just spell it out?
> > "In the scenario above, you would type..."

But it is for almost all contractions, so that is not a good suggestio
in general.

> > Currently the snapshots are build using fedora-latest, which indeed
> > just switched to Fedora 40. We could use a distro that updates less
> > frequently, like debian-stable, if we don't want to be bothered by
> > these faster moving distros.
> 
> Millenial youngsters and their fancy unicode... :o)

I don't mind the fancy unicode in general, but for quote characters it
certainly is a little confusion.

Maybe the simplest "solution" is turning --disable-encoding back on?
At least that seems to generate the same (boring ASCII only) INSTALL
file with both 7.0.3 and 7.1.

Cheers,

Mark