From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mout-p-201.mailbox.org (mout-p-201.mailbox.org [IPv6:2001:67c:2050:0:465::201]) by sourceware.org (Postfix) with ESMTPS id 740A03858CDA; Mon, 25 Sep 2023 16:07:22 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 740A03858CDA Authentication-Results: sourceware.org; dmarc=pass (p=reject dis=none) header.from=aarsen.me Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=aarsen.me Received: from smtp102.mailbox.org (smtp102.mailbox.org [10.196.197.102]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange ECDHE (P-384) server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by mout-p-201.mailbox.org (Postfix) with ESMTPS id 4RvSSR3v44z9sQy; Mon, 25 Sep 2023 18:07:19 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=aarsen.me; s=MBO0001; t=1695658039; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=w7tgBVU4ZoyFonMusRpvcnj5ngb92v4bVia6epl6tzk=; b=FpCXs1XQHYSPNtuNs80p7asbu0KHFZgnhDiVfCktdD2VL+LuEWVCN53dTNCDRm0EtzUV/m h94UdB4m7yaT8BmpSGFbKYuHr37KY7uU9l/r2Baen8KsDSF8Dh5jWZFb3ri/JuM99erOQG R1vXNDANZf8RamaGlJAKXpJdgb4akukSFJEgvpZ+R6Xkw8ELR16gip8TGviT7HrJY9Q/bq a1tpnoXg/kuHZXlR1aSD9OL3MXZWwK2SFHtauGdZyayH5hfXH9Mju6RQZw6Rtm99UMAZGd AY42eDS9OKeUf9pH0gqxJ3WqkDJuA6jwoTZzzsSzEBu5piRPwpSopocc1f1EpA== References: <20230925153247.908901-1-arsen@aarsen.me> <20230925153247.908901-3-arsen@aarsen.me> From: Arsen =?utf-8?Q?Arsenovi=C4=87?= To: Andreas Schwab Cc: Arsen =?utf-8?Q?Arsenovi=C4=87?= via Binutils , gdb-patches@sourceware.org, Bruno Haible , Iain Sandoe Subject: Re: [PATCH 2/2] *: suppress xgettext 0.22 charset name error Date: Mon, 25 Sep 2023 17:55:20 +0200 In-reply-to: Message-ID: <86o7hqz16y.fsf@aarsen.me> MIME-Version: 1.0 Content-Type: multipart/signed; boundary="=-=-="; micalg=pgp-sha512; protocol="application/pgp-signature" X-Spam-Status: No, score=-11.2 required=5.0 tests=BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,GIT_PATCH_0,KAM_INFOUSMEBIZ,KAM_SHORT,RCVD_IN_DNSWL_LOW,SPF_HELO_NONE,SPF_PASS,TXREP autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org List-Id: --=-=-= Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Andreas Schwab writes: > On Sep 25 2023, Arsen Arsenovi=C4=87 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 @@ >>=20=20 >> AUTOMAKE_OPTIONS =3D no-dist foreign >> ACLOCAL_AMFLAGS =3D -I . -I ../config >> +SUBDIRS =3D >>=20=20 >> AM_CPPFLAGS =3D -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.=20 >> 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 @@ >> # . >> # >>=20=20 >> +SUBDIRS =3D=20 >> + >> ACLOCAL_AMFLAGS =3D -I .. -I ../config -I ../bfd >>=20=20 >> AUTOMAKE_OPTIONS =3D 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.=20 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. =2D-=20 Arsen Arsenovi=C4=87 --=-=-= Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iOYEARYKAI4WIQT+4rPRE/wAoxYtYGFSwpQwHqLEkwUCZRGwNV8UgAAAAAAuAChp c3N1ZXItZnByQG5vdGF0aW9ucy5vcGVucGdwLmZpZnRoaG9yc2VtYW4ubmV0RkVF MkIzRDExM0ZDMDBBMzE2MkQ2MDYxNTJDMjk0MzAxRUEyQzQ5MxAcYXJzZW5AYWFy c2VuLm1lAAoJEFLClDAeosSTwVgA/0pqYkb3mq84TS4ljlfFUdlKlMAOZc+YVSgQ Fr+InEzsAQDnvy/qMU4tnGFaKna5JrmqxgRADPfr/JAZp+dFgXZDDw== =S5bG -----END PGP SIGNATURE----- --=-=-=--