Re: [PATCH v3 0/2] Replace intl/ with out-of-tree GNU gettext

2023-11-21 Thread Eric Gallager
On Tue, Nov 21, 2023 at 8:51 AM Arsen Arsenović wrote: > > > Arsen Arsenović writes: > > > Bruno Haible writes: > > > >> Arsen Arsenović wrote: > >>> Comparing stages 2 and 3 > >>> Bootstrap comparison failure! > >>> gettext/libasprintf/autosprintf.o differs > >>> make[2]: *** [Makefile:

Re: [PATCH v3 0/2] Replace intl/ with out-of-tree GNU gettext

2023-11-21 Thread David Edelsohn
On Tue, Nov 21, 2023 at 8:51 AM Arsen Arsenović wrote: > > Arsen Arsenović writes: > > > Bruno Haible writes: > > > >> Arsen Arsenović wrote: > >>> Comparing stages 2 and 3 > >>> Bootstrap comparison failure! > >>> gettext/libasprintf/autosprintf.o differs > >>> make[2]: *** [Makefile:2

Re: [PATCH v3 0/2] Replace intl/ with out-of-tree GNU gettext

2023-11-21 Thread Arsen Arsenović
Arsen Arsenović writes: > Bruno Haible writes: > >> Arsen Arsenović wrote: >>> Comparing stages 2 and 3 >>> Bootstrap comparison failure! >>> gettext/libasprintf/autosprintf.o differs >>> make[2]: *** [Makefile:23435: compare] Error 1 >> >> You should be able to work around this by pass

Re: [PATCH v3 0/2] Replace intl/ with out-of-tree GNU gettext

2023-11-20 Thread Arsen Arsenović
Bruno Haible writes: > Arsen Arsenović wrote: >> Comparing stages 2 and 3 >> Bootstrap comparison failure! >> gettext/libasprintf/autosprintf.o differs >> make[2]: *** [Makefile:23435: compare] Error 1 > > You should be able to work around this by passing the additional option > --disabl

Re: [PATCH v3 0/2] Replace intl/ with out-of-tree GNU gettext

2023-11-20 Thread Arsen Arsenović
David Edelsohn writes: > _GLOBAL__F_xxx is the EH frame data. > > It's using the filename with full path for the unique name, which is why it > includes .._.._.. . Apparently it is adding a random number as well for > uniqueness. I guess that this is the downside of building in tree, and > app

Re: [PATCH v3 0/2] Replace intl/ with out-of-tree GNU gettext

2023-11-20 Thread Bruno Haible
Arsen Arsenović wrote: > Comparing stages 2 and 3 > Bootstrap comparison failure! > gettext/libasprintf/autosprintf.o differs > make[2]: *** [Makefile:23435: compare] Error 1 You should be able to work around this by passing the additional option --disable-libasprintf to gettext-runtime/co

Re: [PATCH v3 0/2] Replace intl/ with out-of-tree GNU gettext

2023-11-20 Thread David Edelsohn
_GLOBAL__F_xxx is the EH frame data. It's using the filename with full path for the unique name, which is why it includes .._.._.. . Apparently it is adding a random number as well for uniqueness. I guess that this is the downside of building in tree, and apparently it is rebuilding gettext itse

Re: [PATCH v3 0/2] Replace intl/ with out-of-tree GNU gettext

2023-11-20 Thread Arsen Arsenović
David Edelsohn writes: > On Sun, Nov 19, 2023 at 5:15 PM Bruno Haible wrote: > >> David Edelsohn wrote: >> > --disable-threads currently does not completely disable threads. Bruno >> is >> > suggesting --enable-threads=isoc that relies on mtx mutex functions in >> libc. >> >> Unfortunately, as

Re: [PATCH v3 0/2] Replace intl/ with out-of-tree GNU gettext

2023-11-19 Thread David Edelsohn
On Sun, Nov 19, 2023 at 5:15 PM Bruno Haible wrote: > David Edelsohn wrote: > > --disable-threads currently does not completely disable threads. Bruno > is > > suggesting --enable-threads=isoc that relies on mtx mutex functions in > libc. > > Unfortunately, as said in the other mail today, relyi

Re: [PATCH v3 0/2] Replace intl/ with out-of-tree GNU gettext

2023-11-19 Thread Andrew Pinski
On Sun, Nov 19, 2023 at 3:01 PM Bruno Haible wrote: > > I wrote: > > you (Arsen) should state in the documentation > > (gcc/doc/install.texi) that for --disable-threads to have this effect, > > one needs gettext version 0.22.4 or newer. > > Not in gcc/doc/install.texi, but elsewhere. This topic is

Re: [PATCH v3 0/2] Replace intl/ with out-of-tree GNU gettext

2023-11-19 Thread Bruno Haible
I wrote: > you (Arsen) should state in the documentation > (gcc/doc/install.texi) that for --disable-threads to have this effect, > one needs gettext version 0.22.4 or newer. Not in gcc/doc/install.texi, but elsewhere. This topic is not relevant to the average user who installs GCC from a tarball

Re: [PATCH v3 0/2] Replace intl/ with out-of-tree GNU gettext

2023-11-19 Thread Bruno Haible
David Edelsohn wrote: > --disable-threads currently does not completely disable threads. Bruno is > suggesting --enable-threads=isoc that relies on mtx mutex functions in libc. Unfortunately, as said in the other mail today, relying only on mtx_* functions did not drop the dependency towards libp

Re: [PATCH v3 0/2] Replace intl/ with out-of-tree GNU gettext

2023-11-18 Thread Arsen Arsenović
David Edelsohn writes: > On Fri, Nov 17, 2023 at 10:17 AM Arsen Arsenović wrote: > >> >> David Edelsohn writes: >> >> > On Fri, Nov 17, 2023 at 3:46 AM Arsen Arsenović wrote: >> > >> >> >> >> David Edelsohn writes: >> >> >> >> > On Thu, Nov 16, 2023 at 5:52 PM Arsen Arsenović >> wrote: >> >

Re: [PATCH v3 0/2] Replace intl/ with out-of-tree GNU gettext

2023-11-17 Thread David Edelsohn
On Fri, Nov 17, 2023 at 10:17 AM Arsen Arsenović wrote: > > David Edelsohn writes: > > > On Fri, Nov 17, 2023 at 3:46 AM Arsen Arsenović wrote: > > > >> > >> David Edelsohn writes: > >> > >> > On Thu, Nov 16, 2023 at 5:52 PM Arsen Arsenović > wrote: > >> > > >> > [snip] > >> >> Sure, but my p

Re: [PATCH v3 0/2] Replace intl/ with out-of-tree GNU gettext

2023-11-17 Thread Arsen Arsenović
David Edelsohn writes: > On Fri, Nov 17, 2023 at 3:46 AM Arsen Arsenović wrote: > >> >> David Edelsohn writes: >> >> > On Thu, Nov 16, 2023 at 5:52 PM Arsen Arsenović wrote: >> > >> > [snip] >> >> Sure, but my patch does insert --disable-shared: >> >> >> >> --8<---cut here

Re: [PATCH v3 0/2] Replace intl/ with out-of-tree GNU gettext

2023-11-17 Thread David Edelsohn
On Fri, Nov 17, 2023 at 3:46 AM Arsen Arsenović wrote: > > David Edelsohn writes: > > > On Thu, Nov 16, 2023 at 5:52 PM Arsen Arsenović wrote: > > > > [snip] > >> Sure, but my patch does insert --disable-shared: > >> > >> --8<---cut here---start->8--- > >> ho

Re: [PATCH v3 0/2] Replace intl/ with out-of-tree GNU gettext

2023-11-17 Thread Arsen Arsenović
Richard Biener writes: > On Fri, Nov 17, 2023 at 9:46 AM Arsen Arsenović wrote: >> >> >> David Edelsohn writes: >> >> > On Thu, Nov 16, 2023 at 5:52 PM Arsen Arsenović wrote: >> > >> > [snip] >> >> Sure, but my patch does insert --disable-shared: >> >> >> >> --8<---cut here---

Re: [PATCH v3 0/2] Replace intl/ with out-of-tree GNU gettext

2023-11-17 Thread Richard Biener
On Fri, Nov 17, 2023 at 9:46 AM Arsen Arsenović wrote: > > > David Edelsohn writes: > > > On Thu, Nov 16, 2023 at 5:52 PM Arsen Arsenović wrote: > > > > [snip] > >> Sure, but my patch does insert --disable-shared: > >> > >> --8<---cut here---start->8--- > >> h

Re: [PATCH v3 0/2] Replace intl/ with out-of-tree GNU gettext

2023-11-17 Thread Arsen Arsenović
David Edelsohn writes: > On Thu, Nov 16, 2023 at 5:52 PM Arsen Arsenović wrote: > > [snip] >> Sure, but my patch does insert --disable-shared: >> >> --8<---cut here---start->8--- >> host_modules= { module= gettext; bootstrap=true; no_install=true; >>

Re: [PATCH v3 0/2] Replace intl/ with out-of-tree GNU gettext

2023-11-16 Thread David Edelsohn
On Thu, Nov 16, 2023 at 5:52 PM Arsen Arsenović wrote: > > David Edelsohn writes: > > > On Thu, Nov 16, 2023 at 5:22 PM Arsen Arsenović wrote: > > > >> > >> David Edelsohn writes: > >> > >> > Don't build with the dependent libraries in tree. Don't build the > >> > dependent libraries as share

Re: [PATCH v3 0/2] Replace intl/ with out-of-tree GNU gettext

2023-11-16 Thread Arsen Arsenović
David Edelsohn writes: > On Thu, Nov 16, 2023 at 5:22 PM Arsen Arsenović wrote: > >> >> David Edelsohn writes: >> >> > Don't build with the dependent libraries in tree. Don't build the >> > dependent libraries as shared libraries. The libraries are already built >> > and in /opt/cfarm, as men

Re: [PATCH v3 0/2] Replace intl/ with out-of-tree GNU gettext

2023-11-16 Thread David Edelsohn
On Thu, Nov 16, 2023 at 5:22 PM Arsen Arsenović wrote: > > David Edelsohn writes: > > > Don't build with the dependent libraries in tree. Don't build the > > dependent libraries as shared libraries. The libraries are already built > > and in /opt/cfarm, as mentioned in the Compile Farm wiki. >

Re: [PATCH v3 0/2] Replace intl/ with out-of-tree GNU gettext

2023-11-16 Thread Arsen Arsenović
David Edelsohn writes: > Don't build with the dependent libraries in tree. Don't build the > dependent libraries as shared libraries. The libraries are already built > and in /opt/cfarm, as mentioned in the Compile Farm wiki. > > AIX is not Solaris and not Linux. It doesn't use ELF. AIX share

Re: [PATCH v3 0/2] Replace intl/ with out-of-tree GNU gettext

2023-11-16 Thread David Edelsohn
Don't build with the dependent libraries in tree. Don't build the dependent libraries as shared libraries. The libraries are already built and in /opt/cfarm, as mentioned in the Compile Farm wiki. AIX is not Solaris and not Linux. It doesn't use ELF. AIX shared libraries *ARE* shared object fil

Re: [PATCH v3 0/2] Replace intl/ with out-of-tree GNU gettext

2023-11-16 Thread Arsen Arsenović
Arsen Arsenović writes: > [[PGP Signed Part:Good signature from 52C294301EA2C493 Arsen Arsenović > (Gentoo Developer UID) (trust ultimate) created at > 2023-11-16T19:47:16+0100 using EDDSA]] > > David Edelsohn writes: > >> On Wed, Nov 15, 2023 at 9:22 AM Arsen Arsenović wrote: >> >>> >>> Da

Re: [PATCH v3 0/2] Replace intl/ with out-of-tree GNU gettext

2023-11-16 Thread Arsen Arsenović
Xi Ruoyao writes: > On Wed, 2023-11-15 at 15:14 +0100, Arsen Arsenović wrote: >> That is interesting.  They should be using the same checks.  I've >> checked trunk and regenerated files on it, and saw no significant diff >> (some whitespace changes only).  Could you post the config.log of >> bot

Re: [PATCH v3 0/2] Replace intl/ with out-of-tree GNU gettext

2023-11-16 Thread Arsen Arsenović
David Edelsohn writes: > On Wed, Nov 15, 2023 at 9:22 AM Arsen Arsenović wrote: > >> >> David Edelsohn writes: >> >> > GCC had been working on AIX with NLS, using "--with-included-gettext". >> > --disable-nls gets past the breakage, but GCC does not build for me on >> AIX >> > with NLS enabled

Re: [PATCH v3 0/2] Replace intl/ with out-of-tree GNU gettext

2023-11-15 Thread David Edelsohn
On Wed, Nov 15, 2023 at 9:22 AM Arsen Arsenović wrote: > > David Edelsohn writes: > > > GCC had been working on AIX with NLS, using "--with-included-gettext". > > --disable-nls gets past the breakage, but GCC does not build for me on > AIX > > with NLS enabled. > > That should still work with ge

Re: [PATCH v3 0/2] Replace intl/ with out-of-tree GNU gettext

2023-11-15 Thread David Edelsohn
On Wed, Nov 15, 2023 at 9:22 AM Arsen Arsenović wrote: > > David Edelsohn writes: > > > GCC had been working on AIX with NLS, using "--with-included-gettext". > > --disable-nls gets past the breakage, but GCC does not build for me on > AIX > > with NLS enabled. > > That should still work with ge

Re: [PATCH v3 0/2] Replace intl/ with out-of-tree GNU gettext

2023-11-15 Thread Xi Ruoyao
On Wed, 2023-11-15 at 15:14 +0100, Arsen Arsenović wrote: > That is interesting.  They should be using the same checks.  I've > checked trunk and regenerated files on it, and saw no significant diff > (some whitespace changes only).  Could you post the config.log of > both? You did not regenerate

Re: [PATCH v3 0/2] Replace intl/ with out-of-tree GNU gettext

2023-11-15 Thread Arsen Arsenović
David Edelsohn writes: > GCC had been working on AIX with NLS, using "--with-included-gettext". > --disable-nls gets past the breakage, but GCC does not build for me on AIX > with NLS enabled. That should still work with gettext 0.22+ extracted in-tree (it should be fetched by download_prerequi

Re: [PATCH v3 0/2] Replace intl/ with out-of-tree GNU gettext

2023-11-14 Thread David Edelsohn
On Tue, Nov 14, 2023 at 6:09 PM Arsen Arsenović wrote: > Hi David, > > David Edelsohn writes: > > > Arsen, > > > > Unfortunately this broke bootstrap on AIX. > > > > I had not seen this series of patches. > > I've added Bruno to CC as the libintl maintainer, to keep him in the > loop. Could you

Re: [PATCH v3 0/2] Replace intl/ with out-of-tree GNU gettext

2023-11-14 Thread Arsen Arsenović
Hi David, David Edelsohn writes: > Arsen, > > Unfortunately this broke bootstrap on AIX. > > I had not seen this series of patches. I've added Bruno to CC as the libintl maintainer, to keep him in the loop. Could you provide some extra information w.r.t. the failure mode? I'll try to investig

Re: [PATCH v3 0/2] Replace intl/ with out-of-tree GNU gettext

2023-11-14 Thread David Edelsohn
Arsen, Unfortunately this broke bootstrap on AIX. I had not seen this series of patches. David

Re: [PATCH v3 0/2] Replace intl/ with out-of-tree GNU gettext

2023-11-10 Thread Richard Biener
On Thu, Nov 2, 2023 at 9:43 AM Arsen Arsenović wrote: > > Morning! > > This patch is a rebase and slight wording tweak of > https://inbox.sourceware.org/20231006140501.3370874-1-ar...@aarsen.me > > Changes since v2: > - Elaborate on the libintl requirement on non-glibc hosts, per Andrews > reque

[PATCH v3 0/2] Replace intl/ with out-of-tree GNU gettext

2023-11-02 Thread Arsen Arsenović
Morning! This patch is a rebase and slight wording tweak of https://inbox.sourceware.org/20231006140501.3370874-1-ar...@aarsen.me Changes since v2: - Elaborate on the libintl requirement on non-glibc hosts, per Andrews request Range diff since v2 (since it seems sufficiently readable here): @@