* [PATCH 0/2] Replace intl/ with out-of-tree GNU gettext
@ 2023-09-25 15:13 Arsen Arsenović via Gdb-patches
2023-09-25 15:13 ` [PATCH 2/2] *: suppress xgettext 0.22 charset name error Arsen Arsenović via Gdb-patches
2023-09-25 19:11 ` [PATCH 0/2] Replace intl/ with out-of-tree GNU gettext Kevin Buettner via Gdb-patches
0 siblings, 2 replies; 6+ messages in thread
From: Arsen Arsenović via Gdb-patches @ 2023-09-25 15:13 UTC (permalink / raw)
To: gdb-patches, binutils; +Cc: Iain Sandoe, Bruno Haible
Afternoon,
This patch series replaces the old (early 2000s era, AFAICT) libintl
implementation in-tree, which relies on C constructs some compilers
(newer clang, hopefully GCC 14) refuse to compile by default with
out-of-tree gettext, in a manner similar to GMP et al.
During that work I also noticed that the .pot files fail to regenerate
with new xgettext, leading to the discovery of a regression, but also a
slight simplification that can be made in the .pot file generation:
https://savannah.gnu.org/bugs/?64490#comment4
Brunos recommendation is included. Thanks!
I've omitted a patch that simply drops the intl/ directory from this
patchset, as it was held for moderation on the GCC ML, and I suspect the
same would happen here. It can be seen at:
https://git.sr.ht/~arsen/binutils-gdb/commit/300747ed91e6652e4e20b876c19b3c86c75ac57f
Note that this patch series results in some new symbols in
libinproctrace.so:
~/binutils-bld/gdbserver$ nm -g libinproctrace.so | grep libintl
0000000000015090 T libintl_dcgettext
0000000000015080 T libintl_gettext
0000000000019a90 T libintl_set_relocation_prefix
This happens because the library gets linked against libintl.a (because
it is required by ax.cc, gdbserver/ax.cc:925:(.text+0x2f8): undefined
reference to `libintl_gettext'), and so, the default visibility symbols
from libintl.a 'leak' through. I am decently sure this was also the
case with the in-tree intl implementation, though.
Perhaps it'd be good to add a version script to this library, to be more
selective about symbols.
Tested on x86_64-pc-linux-gnu --with-included-gettext,
x86_64-portbld-freebsd13.1, x86_64-darwin21, i686-darwin9 and
aarch64-darwin21 (thanks, Iain).
Example from FreeBSD:
[arsen@fbsd132 ~/binutils-bld/_pfx/bin]$ LANG=sr_RS.UTF-8 ./ld --version
ГНУ лд (GNU Binutils) 2.41.50.20230924
Copyright (C) 2023 Free Software Foundation, Inc.
Овај програм је слободан софтвер; можете да га расподељујете под одредбама
Гнуове опште јавне лиценце издања 3 или (према вашем мишљењу) било ког
новијег издања. Овај програм нема никакву гаранцију.
[arsen@fbsd132 ~/binutils-bld/_pfx/bin]$ LANG=fr_FR.UTF-8 ./as --version
Assembleur GNU (GNU Binutils) 2.41.50.20230924
Copyright © 2023 Free Software Foundation, Inc.
Ce logiciel est libre; vous pouvez le redistribuer selon les termes de la
version 3 de la licence GNU General Public License ou suivante.
Ce programme n'est couvert par AUCUNE garantie.
Cet assembleur a été configuré pour la cible « x86_64-unknown-freebsd13.2 ».
[arsen@fbsd132 ~/binutils-bld/_pfx/bin]$ LANG=fr_FR.UTF-8 ./gprof --version
GNU gprof (GNU Binutils) 2.41.50.20230924
Basé sur BSD gprof, copyright 1983 Regents de l'Université de Californie.
Ce logiciel est libre. AUCUNE garantie ne lui est apportée.
A similar patch was also sent to the GCC developers for evaluation. We
should coordinate to merge them around the same time, so that the same
code is merged.
Thanks in advance, have a lovely day.
Arsen Arsenović (2):
*: add modern gettext support
*: suppress xgettext 0.22 charset name error
.gitignore | 1 +
Makefile.def | 72 +-
Makefile.in | 1612 +++++++++++-----------
bfd/aclocal.m4 | 10 +-
bfd/configure | 1980 ++++++++++++++++++++++++++-
bfd/configure.ac | 2 +-
bfd/po/Make-in | 8 +-
binutils/aclocal.m4 | 2 +
binutils/configure | 2546 ++++++++++++++++++++++++++---------
binutils/po/Make-in | 5 +-
config/gettext-sister.m4 | 35 +-
config/gettext.m4 | 357 +++--
config/iconv.m4 | 313 +++--
config/intlmacosx.m4 | 65 +
configure | 44 +-
configure.ac | 44 +-
gas/aclocal.m4 | 6 +
gas/configure | 1980 ++++++++++++++++++++++++++-
gas/po/Make-in | 5 +-
gdb/acinclude.m4 | 3 +-
gdb/aclocal.m4 | 5 +
gdb/configure | 2739 ++++++++++++++++++++++++++++----------
gdb/configure.ac | 3 +-
gdbserver/Makefile.in | 3 +-
gdbserver/aclocal.m4 | 6 +
gdbserver/configure | 1817 ++++++++++++++++++++++++-
gdbsupport/Makefile.am | 1 +
gdbsupport/aclocal.m4 | 6 +
gdbsupport/configure | 2109 ++++++++++++++++++++++++++---
gold/aclocal.m4 | 8 +-
gold/configure | 1977 ++++++++++++++++++++++++++-
gold/po/Make-in | 5 +-
gprof/aclocal.m4 | 6 +
gprof/configure | 1980 ++++++++++++++++++++++++++-
gprof/po/Make-in | 5 +-
ld/aclocal.m4 | 6 +
ld/configure | 1980 ++++++++++++++++++++++++++-
ld/po/Make-in | 8 +-
libctf/Makefile.am | 2 +
libctf/aclocal.m4 | 9 +
libctf/configure | 1952 ++++++++++++++++++++++++++-
libctf/configure.ac | 2 +-
opcodes/aclocal.m4 | 6 +
opcodes/configure | 1980 ++++++++++++++++++++++++++-
opcodes/configure.ac | 2 +-
opcodes/po/Make-in | 5 +-
46 files changed, 22750 insertions(+), 2962 deletions(-)
create mode 100644 config/intlmacosx.m4
--
2.42.0
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 2/2] *: suppress xgettext 0.22 charset name error
2023-09-25 15:13 [PATCH 0/2] Replace intl/ with out-of-tree GNU gettext Arsen Arsenović via Gdb-patches
@ 2023-09-25 15:13 ` Arsen Arsenović via Gdb-patches
2023-09-25 15:46 ` Andreas Schwab via Gdb-patches
2023-09-25 19:11 ` [PATCH 0/2] Replace intl/ with out-of-tree GNU gettext Kevin Buettner via Gdb-patches
1 sibling, 1 reply; 6+ messages in thread
From: Arsen Arsenović via Gdb-patches @ 2023-09-25 15:13 UTC (permalink / raw)
To: gdb-patches, binutils; +Cc: Iain Sandoe, Bruno Haible
bfd.po: warning: Charset "CHARSET" is not a portable encoding name.
Message conversion to user's charset might not work.
/usr/bin/xgettext: present charset "CHARSET" is not a portable encoding name
As Bruno Haible <bruno@clisp.org> suggested:
> > Perhaps it'd be good to use -o to suppress this warning anyway
>
> Yes, regardless of gettext version, you profit of the .pot suffix handling by
> replacing
>
> --default-domain=$(PACKAGE)
>
> with
>
> -o $(PACKAGE)-tmp.pot
>
> (twice) and
>
> < $(PACKAGE).po > $@-t
>
> with
>
> < $(PACKAGE)-tmp.pot > $@-t
bfd/ChangeLog:
* po/Make-in ($(srcdir)/$(PACKAGE).pot): Output to a .pot
temporary file, to suppress xgettext checking charset names.
binutils/ChangeLog:
* po/Make-in ($(srcdir)/$(PACKAGE).pot): Output to a .pot
temporary file to suppress xgettext checking charset names.
gas/ChangeLog:
* po/Make-in ($(srcdir)/$(PACKAGE).pot): Output to a .pot
temporary file to suppress xgettext checking charset names.
gold/ChangeLog:
* po/Make-in ($(srcdir)/$(PACKAGE).pot): Output to a .pot
temporary file to suppress xgettext checking charset names.
gprof/ChangeLog:
* po/Make-in ($(srcdir)/$(PACKAGE).pot): Output to a .pot
temporary file to suppress xgettext checking charset names.
ld/ChangeLog:
* po/Make-in ($(srcdir)/$(PACKAGE).pot): Output to a .pot
temporary file to suppress xgettext checking charset names.
libctf/ChangeLog:
* po/Make-in ($(srcdir)/$(PACKAGE).pot): Output to a .pot
temporary file to suppress xgettext checking charset names.
---
bfd/po/Make-in | 8 +++++---
binutils/po/Make-in | 5 +++--
gas/po/Make-in | 5 +++--
gdbsupport/Makefile.am | 1 +
gold/po/Make-in | 5 +++--
gprof/po/Make-in | 5 +++--
ld/po/Make-in | 8 +++++---
libctf/Makefile.am | 2 ++
opcodes/po/Make-in | 5 +++--
9 files changed, 28 insertions(+), 16 deletions(-)
diff --git a/bfd/po/Make-in b/bfd/po/Make-in
index 30bcd5576e9..af8f4fb5757 100644
--- a/bfd/po/Make-in
+++ b/bfd/po/Make-in
@@ -93,19 +93,21 @@ all-yes: $(CATALOGS) @MAINT@ $(PACKAGE).pot
all-no:
$(srcdir)/$(PACKAGE).pot: $(SRC-POTFILES) $(BLD-POTFILES)
- $(XGETTEXT) --default-domain=$(PACKAGE) \
+ $(XGETTEXT) -o $(PACKAGE)-tmp.pot \
+ --default-domain=$(PACKAGE) \
--directory=$(top_srcdir) \
--add-comments --keyword=_ --keyword=N_ \
--msgid-bugs-address=https://sourceware.org/bugzilla/ \
--files-from=$(srcdir)/SRC-POTFILES.in
- $(XGETTEXT) --default-domain=$(PACKAGE) \
+ $(XGETTEXT) -o $(PACKAGE)-tmp.pot \
+ --default-domain=$(PACKAGE) \
--directory=.. \
--directory=. \
--add-comments --keyword=_ --keyword=N_ \
--join-existing \
--msgid-bugs-address=https://sourceware.org/bugzilla/ \
--files-from=$(srcdir)/BLD-POTFILES.in
- sed -e '/^#:/s,$(top_srcdir)/,,g' < $(PACKAGE).po > $@-t
+ sed -e '/^#:/s,$(top_srcdir)/,,g' < $(PACKAGE)-tmp.pot > $@-t
mv -f $@-t $@
$(srcdir)/cat-id-tbl.c: stamp-cat-id; @:
diff --git a/binutils/po/Make-in b/binutils/po/Make-in
index 167b04b7560..6ae60ac2bcd 100644
--- a/binutils/po/Make-in
+++ b/binutils/po/Make-in
@@ -87,12 +87,13 @@ all-yes: $(CATALOGS) @MAINT@ $(PACKAGE).pot
all-no:
$(srcdir)/$(PACKAGE).pot: $(POTFILES)
- $(XGETTEXT) --default-domain=$(PACKAGE) --directory=$(top_srcdir) \
+ $(XGETTEXT) -o $(PACKAGE)-tmp.pot --directory=$(top_srcdir) \
+ --default-domain=$(PACKAGE) \
--add-comments --keyword=_ --keyword=N_ \
--msgid-bugs-address=https://sourceware.org/bugzilla/ \
--files-from=$(srcdir)/POTFILES.in
rm -f $(srcdir)/$(PACKAGE).pot
- mv $(PACKAGE).po $(srcdir)/$(PACKAGE).pot
+ mv $(PACKAGE)-tmp.pot $(srcdir)/$(PACKAGE).pot
$(srcdir)/cat-id-tbl.c: stamp-cat-id; @:
$(srcdir)/stamp-cat-id: $(PACKAGE).pot
diff --git a/gas/po/Make-in b/gas/po/Make-in
index 167b04b7560..6ae60ac2bcd 100644
--- a/gas/po/Make-in
+++ b/gas/po/Make-in
@@ -87,12 +87,13 @@ all-yes: $(CATALOGS) @MAINT@ $(PACKAGE).pot
all-no:
$(srcdir)/$(PACKAGE).pot: $(POTFILES)
- $(XGETTEXT) --default-domain=$(PACKAGE) --directory=$(top_srcdir) \
+ $(XGETTEXT) -o $(PACKAGE)-tmp.pot --directory=$(top_srcdir) \
+ --default-domain=$(PACKAGE) \
--add-comments --keyword=_ --keyword=N_ \
--msgid-bugs-address=https://sourceware.org/bugzilla/ \
--files-from=$(srcdir)/POTFILES.in
rm -f $(srcdir)/$(PACKAGE).pot
- mv $(PACKAGE).po $(srcdir)/$(PACKAGE).pot
+ mv $(PACKAGE)-tmp.pot $(srcdir)/$(PACKAGE).pot
$(srcdir)/cat-id-tbl.c: stamp-cat-id; @:
$(srcdir)/stamp-cat-id: $(PACKAGE).pot
diff --git a/gdbsupport/Makefile.am b/gdbsupport/Makefile.am
index 00524e9a566..43aa670faf4 100644
--- a/gdbsupport/Makefile.am
+++ b/gdbsupport/Makefile.am
@@ -19,6 +19,7 @@
AUTOMAKE_OPTIONS = no-dist foreign
ACLOCAL_AMFLAGS = -I . -I ../config
+SUBDIRS =
AM_CPPFLAGS = -I$(srcdir)/../include -I$(srcdir)/../gdb \
-I../gnulib/import -I$(srcdir)/../gnulib/import \
diff --git a/gold/po/Make-in b/gold/po/Make-in
index e8e881ba3e3..9fea006bf50 100644
--- a/gold/po/Make-in
+++ b/gold/po/Make-in
@@ -87,12 +87,13 @@ all-yes: $(CATALOGS) @MAINT@ $(PACKAGE).pot
all-no:
$(srcdir)/$(PACKAGE).pot: $(POTFILES)
- $(XGETTEXT) --default-domain=$(PACKAGE) --directory=$(top_srcdir) \
+ $(XGETTEXT) -o $(PACKAGE)-tmp.pot --directory=$(top_srcdir) \
+ --default-domain=$(PACKAGE) \
--add-comments -C --keyword=_ --keyword=N_ \
--msgid-bugs-address=https://sourceware.org/bugzilla/ \
--files-from=$(srcdir)/POTFILES.in
rm -f $(srcdir)/$(PACKAGE).pot
- mv $(PACKAGE).po $(srcdir)/$(PACKAGE).pot
+ mv $(PACKAGE)-tmp.pot $(srcdir)/$(PACKAGE).pot
$(srcdir)/cat-id-tbl.c: stamp-cat-id; @:
$(srcdir)/stamp-cat-id: $(PACKAGE).pot
diff --git a/gprof/po/Make-in b/gprof/po/Make-in
index ba6f3923308..4b5c174f1e0 100644
--- a/gprof/po/Make-in
+++ b/gprof/po/Make-in
@@ -87,12 +87,13 @@ all-yes: $(CATALOGS) @MAINT@ $(PACKAGE).pot
all-no:
$(srcdir)/$(PACKAGE).pot: $(POTFILES)
- $(XGETTEXT) --default-domain=$(PACKAGE) --directory=$(top_srcdir) \
+ $(XGETTEXT) -o $(PACKAGE)-tmp.pot --directory=$(top_srcdir) \
+ --default-domain=$(PACKAGE) \
--add-comments --keyword=_ --keyword=N_ \
--msgid-bugs-address=https://sourceware.org/bugzilla/ \
--files-from=$(srcdir)/POTFILES.in
rm -f $(srcdir)/$(PACKAGE).pot
- mv $(PACKAGE).po $(srcdir)/$(PACKAGE).pot
+ mv $(PACKAGE)-tmp.pot $(srcdir)/$(PACKAGE).pot
$(srcdir)/cat-id-tbl.c: stamp-cat-id; @:
$(srcdir)/stamp-cat-id: $(PACKAGE).pot
diff --git a/ld/po/Make-in b/ld/po/Make-in
index 26e318570fb..c6a301def68 100644
--- a/ld/po/Make-in
+++ b/ld/po/Make-in
@@ -89,17 +89,19 @@ all-yes: $(CATALOGS) @MAINT@ $(PACKAGE).pot
all-no:
$(srcdir)/$(PACKAGE).pot: $(SRC-POTFILES) $(BLD-POTFILES)
- $(XGETTEXT) --default-domain=$(PACKAGE) --directory=$(top_srcdir) \
+ $(XGETTEXT) -o $(PACKAGE)-tmp.pot --directory=$(top_srcdir) \
+ --default-domain=$(PACKAGE) \
--add-comments --keyword=_ --keyword=N_ \
--msgid-bugs-address=https://sourceware.org/bugzilla/ \
--files-from=$(srcdir)/SRC-POTFILES.in
- $(XGETTEXT) --default-domain=$(PACKAGE) --directory=.. \
+ $(XGETTEXT) -o $(PACKAGE)-tmp.pot --directory=.. \
+ --default-domain=$(PACKAGE) \
--add-comments --keyword=_ --keyword=N_ \
--join-existing \
--msgid-bugs-address=https://sourceware.org/bugzilla/ \
--files-from=$(srcdir)/BLD-POTFILES.in
rm -f $(srcdir)/$(PACKAGE).pot
- mv $(PACKAGE).po $(srcdir)/$(PACKAGE).pot
+ mv $(PACKAGE)-tmp.pot $(srcdir)/$(PACKAGE).pot
$(srcdir)/cat-id-tbl.c: stamp-cat-id; @:
$(srcdir)/stamp-cat-id: $(PACKAGE).pot
diff --git a/libctf/Makefile.am b/libctf/Makefile.am
index b1dbc2f6ba4..8c9f623f58c 100644
--- a/libctf/Makefile.am
+++ b/libctf/Makefile.am
@@ -17,6 +17,8 @@
# <http://www.gnu.org/licenses/>.
#
+SUBDIRS =
+
ACLOCAL_AMFLAGS = -I .. -I ../config -I ../bfd
AUTOMAKE_OPTIONS = dejagnu foreign info-in-builddir no-texinfo.tex
diff --git a/opcodes/po/Make-in b/opcodes/po/Make-in
index 167b04b7560..6ae60ac2bcd 100644
--- a/opcodes/po/Make-in
+++ b/opcodes/po/Make-in
@@ -87,12 +87,13 @@ all-yes: $(CATALOGS) @MAINT@ $(PACKAGE).pot
all-no:
$(srcdir)/$(PACKAGE).pot: $(POTFILES)
- $(XGETTEXT) --default-domain=$(PACKAGE) --directory=$(top_srcdir) \
+ $(XGETTEXT) -o $(PACKAGE)-tmp.pot --directory=$(top_srcdir) \
+ --default-domain=$(PACKAGE) \
--add-comments --keyword=_ --keyword=N_ \
--msgid-bugs-address=https://sourceware.org/bugzilla/ \
--files-from=$(srcdir)/POTFILES.in
rm -f $(srcdir)/$(PACKAGE).pot
- mv $(PACKAGE).po $(srcdir)/$(PACKAGE).pot
+ mv $(PACKAGE)-tmp.pot $(srcdir)/$(PACKAGE).pot
$(srcdir)/cat-id-tbl.c: stamp-cat-id; @:
$(srcdir)/stamp-cat-id: $(PACKAGE).pot
--
2.42.0
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH 2/2] *: suppress xgettext 0.22 charset name error
2023-09-25 15:13 ` [PATCH 2/2] *: suppress xgettext 0.22 charset name error Arsen Arsenović via Gdb-patches
@ 2023-09-25 15:46 ` Andreas Schwab via Gdb-patches
2023-09-25 15:55 ` Arsen Arsenović via Gdb-patches
0 siblings, 1 reply; 6+ messages in thread
From: Andreas Schwab via Gdb-patches @ 2023-09-25 15:46 UTC (permalink / raw)
To: Arsen Arsenović via Binutils; +Cc: Iain Sandoe, Bruno Haible, gdb-patches
On Sep 25 2023, Arsen Arsenović via Binutils wrote:
> libctf/ChangeLog:
>
> * po/Make-in ($(srcdir)/$(PACKAGE).pot): Output to a .pot
> temporary file to suppress xgettext checking charset names.
There is no po/Make-in in libctf.
> diff --git a/gdbsupport/Makefile.am b/gdbsupport/Makefile.am
> index 00524e9a566..43aa670faf4 100644
> --- a/gdbsupport/Makefile.am
> +++ b/gdbsupport/Makefile.am
> @@ -19,6 +19,7 @@
>
> AUTOMAKE_OPTIONS = no-dist foreign
> ACLOCAL_AMFLAGS = -I . -I ../config
> +SUBDIRS =
>
> AM_CPPFLAGS = -I$(srcdir)/../include -I$(srcdir)/../gdb \
> -I../gnulib/import -I$(srcdir)/../gnulib/import \
Spurious change.
> diff --git a/libctf/Makefile.am b/libctf/Makefile.am
> index b1dbc2f6ba4..8c9f623f58c 100644
> --- a/libctf/Makefile.am
> +++ b/libctf/Makefile.am
> @@ -17,6 +17,8 @@
> # <http://www.gnu.org/licenses/>.
> #
>
> +SUBDIRS =
> +
> ACLOCAL_AMFLAGS = -I .. -I ../config -I ../bfd
>
> AUTOMAKE_OPTIONS = dejagnu foreign info-in-builddir no-texinfo.tex
Likewise.
--
Andreas Schwab, SUSE Labs, schwab@suse.de
GPG Key fingerprint = 0196 BAD8 1CE9 1970 F4BE 1748 E4D4 88E3 0EEA B9D7
"And now for something completely different."
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 2/2] *: suppress xgettext 0.22 charset name error
2023-09-25 15:46 ` Andreas Schwab via Gdb-patches
@ 2023-09-25 15:55 ` Arsen Arsenović via Gdb-patches
0 siblings, 0 replies; 6+ messages in thread
From: Arsen Arsenović via Gdb-patches @ 2023-09-25 15:55 UTC (permalink / raw)
To: Andreas Schwab
Cc: Arsen Arsenović via Binutils, gdb-patches, Bruno Haible,
Iain Sandoe
[-- Attachment #1: Type: text/plain, Size: 2010 bytes --]
Andreas Schwab <schwab@suse.de> writes:
> On Sep 25 2023, Arsen Arsenović via Binutils wrote:
>
>> libctf/ChangeLog:
>>
>> * po/Make-in ($(srcdir)/$(PACKAGE).pot): Output to a .pot
>> temporary file to suppress xgettext checking charset names.
>
> There is no po/Make-in in libctf.
Seems to have been an error when I was filling the changelog entries.
The libctf changelog should be:
* Makefile.am (SUBDIRS): Define to empty to suppress Automake
stating 'configure.ac: error: AM_GNU_GETTEXT used but SUBDIRS not
defined'
... which also covers your latter comment.
>
>> diff --git a/gdbsupport/Makefile.am b/gdbsupport/Makefile.am
>> index 00524e9a566..43aa670faf4 100644
>> --- a/gdbsupport/Makefile.am
>> +++ b/gdbsupport/Makefile.am
>> @@ -19,6 +19,7 @@
>>
>> AUTOMAKE_OPTIONS = no-dist foreign
>> ACLOCAL_AMFLAGS = -I . -I ../config
>> +SUBDIRS =
>>
>> AM_CPPFLAGS = -I$(srcdir)/../include -I$(srcdir)/../gdb \
>> -I../gnulib/import -I$(srcdir)/../gnulib/import \
>
> Spurious change.
This has the same rationale as the libctf change above.
>> diff --git a/libctf/Makefile.am b/libctf/Makefile.am
>> index b1dbc2f6ba4..8c9f623f58c 100644
>> --- a/libctf/Makefile.am
>> +++ b/libctf/Makefile.am
>> @@ -17,6 +17,8 @@
>> # <http://www.gnu.org/licenses/>.
>> #
>>
>> +SUBDIRS =
>> +
>> ACLOCAL_AMFLAGS = -I .. -I ../config -I ../bfd
>>
>> AUTOMAKE_OPTIONS = dejagnu foreign info-in-builddir no-texinfo.tex
>
> Likewise.
These directories utilize ZW_GNU_GETTEXT_SISTER_DIR to get information
about where gettext is despite never having any translations of their
own, leading to the Automake warning above.
Apologies for the ChangeLog confusion, it must've been late when I was
splitting and merging commits, and so I mislabeled and mismerged some
changes. I can split this patch into two if you agree, and fix the
libctf ChangeLog entry.
Thanks, have a lovely day.
--
Arsen Arsenović
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 381 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 0/2] Replace intl/ with out-of-tree GNU gettext
2023-09-25 15:13 [PATCH 0/2] Replace intl/ with out-of-tree GNU gettext Arsen Arsenović via Gdb-patches
2023-09-25 15:13 ` [PATCH 2/2] *: suppress xgettext 0.22 charset name error Arsen Arsenović via Gdb-patches
@ 2023-09-25 19:11 ` Kevin Buettner via Gdb-patches
2023-09-25 19:43 ` Arsen Arsenović via Gdb-patches
1 sibling, 1 reply; 6+ messages in thread
From: Kevin Buettner via Gdb-patches @ 2023-09-25 19:11 UTC (permalink / raw)
To: Arsen Arsenović via Gdb-patches; +Cc: Iain Sandoe, Bruno Haible, binutils
On Mon, 25 Sep 2023 17:13:39 +0200
Arsen Arsenović via Gdb-patches <gdb-patches@sourceware.org> wrote:
> Arsen Arsenović (2):
> *: add modern gettext support
> *: suppress xgettext 0.22 charset name error
I haven't seen part 1, "add modern gettext support", in either email that
I receive from the list(s) or in the public archives.
Kevin
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 0/2] Replace intl/ with out-of-tree GNU gettext
2023-09-25 19:11 ` [PATCH 0/2] Replace intl/ with out-of-tree GNU gettext Kevin Buettner via Gdb-patches
@ 2023-09-25 19:43 ` Arsen Arsenović via Gdb-patches
0 siblings, 0 replies; 6+ messages in thread
From: Arsen Arsenović via Gdb-patches @ 2023-09-25 19:43 UTC (permalink / raw)
To: Kevin Buettner
Cc: Arsen Arsenović via Gdb-patches, binutils, Iain Sandoe,
Bruno Haible
[-- Attachment #1: Type: text/plain, Size: 813 bytes --]
Kevin Buettner <kevinb@redhat.com> writes:
> On Mon, 25 Sep 2023 17:13:39 +0200
> Arsen Arsenović via Gdb-patches <gdb-patches@sourceware.org> wrote:
>
>> Arsen Arsenović (2):
>> *: add modern gettext support
>> *: suppress xgettext 0.22 charset name error
>
> I haven't seen part 1, "add modern gettext support", in either email that
> I receive from the list(s) or in the public archives.
Yes, it was held for moderation due to size. It should be visible here:
https://inbox.sourceware.org/gdb-patches/20230925153247.908901-2-arsen@aarsen.me/
I'll send a v2 when the conversation under [PATCH 2/2] settles, and trim
that patch a bit (remove regenerated files) so that it fits under the
size limit.
Apologies for the inconvenience.
Have a lovely night!
--
Arsen Arsenović
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 381 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2023-09-25 19:45 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-09-25 15:13 [PATCH 0/2] Replace intl/ with out-of-tree GNU gettext Arsen Arsenović via Gdb-patches
2023-09-25 15:13 ` [PATCH 2/2] *: suppress xgettext 0.22 charset name error Arsen Arsenović via Gdb-patches
2023-09-25 15:46 ` Andreas Schwab via Gdb-patches
2023-09-25 15:55 ` Arsen Arsenović via Gdb-patches
2023-09-25 19:11 ` [PATCH 0/2] Replace intl/ with out-of-tree GNU gettext Kevin Buettner via Gdb-patches
2023-09-25 19:43 ` Arsen Arsenović via Gdb-patches
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox