From: David Edelsohn <dje.gcc@gmail.com>
To: Iain Sandoe <idsandoe@googlemail.com>, Bruno Haible <bruno@clisp.org>
Cc: GCC Development <gcc@gcc.gnu.org>
Subject: Re: remove intl/ directory?
Date: Sat, 18 Jun 2022 14:14:10 -0400 [thread overview]
Message-ID: <CAGWvnymbMoA+Wp0oPfZKP6O3iDO5vChXMDxmGVCp+GboOO1uVg@mail.gmail.com> (raw)
In-Reply-To: <2F423461-3AF4-4BA9-95D8-EC53B49B43A2@googlemail.com>
On Sat, Jun 18, 2022 at 1:44 PM Iain Sandoe via Gcc <gcc@gcc.gnu.org> wrote:
>
> Hi Bruno,
>
> > On 18 Jun 2022, at 18:01, Bruno Haible <bruno@clisp.org> wrote:
>
> > As the long-term GNU gettext maintainer, I would suggest to remove the intl/
> > directory from the GCC distribution.
> >
> > The effect for the users would be:
> > * On systems without glibc, users who want an internationalized GCC
> > installation would have to install GNU gettext first. Then the GCC
> > binaries would be linked with the shared library libintl.so
> > (unless gettext was built with --disable-shared); they would no longer
> > contain the libintl code in 'cc1', 'cc1plus', etc.
>
> As a maintainer for GCC on a non-glibc system, I would:
>
> (a) welcome a more modern version of intl, with the bug-fixes etc.
> (b) not want to [force] add a shared lib dependency for my downstream.
>
> - so, please could we follow the pattern for GMP et. al. where the library
> can be provided with —with-intl= pointing to an installation, or be built in-
> tree by symlinking an approved version into the GCC tree.
>
> For such [non-glibs] systems where these libraries are not ’normally’
> installed, it is still very much preferable to be able to statically link them
> so that a compiler can be distributed with no deps other than those
> provided by the OS (and we can test what we ship and ship what we test).
As another maintainer for GCC of a non-GLIBC system, I echo Iain's request.
The broad list of systems supported by GCC requires effort, but is one
of its advantages. Making it more difficult to support GCC on
non-GLIBC systems will be detrimental to GCC.
Thanks, David
>
> thanks,
> Iain
>
>
> > * On systems with glibc, no change.
> >
> > The effect for the GCC maintainers would be:
> > * Easier to stay up-to-date with upstream libintl.
> > * Less maintenance work with *.m4 files such as
> > codeset.m4
> > glibc21.m4
> > intdiv0.m4
> > inttypes_h.m4
> > inttypes.m4
> > inttypes-pri.m4
> > lcmessage.m4
> > stdint_h.m4
> > uintmax_t.m4
> > ulonglong.m4
> > * Reduced risk of a CVE that would impact GCC binaries.
> >
> > Rationale:
> > * This intl/ code is from 2003; of course several bugs have been
> > fixed in it over the last 19 years.
> > * At that time GNU packages were still favouring static libraries.
> > GNU libtool became widely reliable only about in 2005.
> > * Since then, distros have been favouring shared libraries over
> > static libraries, in order to be able to fix CVEs without
> > rebuilding many dependent binaries.
> > * For this reason, GNU gettext removed the support for this way
> > of packaging libintl in version 0.20 (May 2019). The NEWS entry
> > said:
> > - The --intl option of the gettextize program (deprecated since 2010) is
> > no longer available. Instead of including the intl sources in your
> > package, we suggest making the libintl library an optional prerequisite
> > of your package. This will simplify the build system of your package.
> > - Accordingly, the Autoconf macro AM_GNU_GETTEXT_INTL_SUBDIR is gone
> > as well.
> >
> > This point came up while discussing with Eric Gallager, who is working on
> > the GCC configury.
> >
> > I don't volunteer to implement this suggestion, but I can give advice where
> > needed.
> >
> > Bruno
> >
> >
> >
>
next prev parent reply other threads:[~2022-06-18 18:14 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-06-18 17:01 Bruno Haible
2022-06-18 17:42 ` Iain Sandoe
2022-06-18 18:14 ` David Edelsohn [this message]
2022-06-19 0:53 ` Bruno Haible
2022-06-19 0:32 ` Bruno Haible
2022-06-19 8:40 ` Iain Sandoe
2022-06-19 20:15 ` Iain Sandoe
2022-06-21 2:05 ` Bruno Haible
2022-06-22 20:21 ` Iain Sandoe
2022-06-23 4:24 ` Bruno Haible
2022-06-23 6:51 ` Iain Sandoe
2022-06-23 15:40 ` Iain Sandoe
2022-06-23 15:50 ` Iain Sandoe
2022-06-23 15:13 ` Eric Gallager
2022-06-20 19:45 ` Eric Gallager
2022-06-21 2:09 ` Bruno Haible
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=CAGWvnymbMoA+Wp0oPfZKP6O3iDO5vChXMDxmGVCp+GboOO1uVg@mail.gmail.com \
--to=dje.gcc@gmail.com \
--cc=bruno@clisp.org \
--cc=gcc@gcc.gnu.org \
--cc=idsandoe@googlemail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).