From: Eli Zaretskii <eliz@gnu.org>
To: Paul Eggert <eggert@cs.ucla.edu>
Cc: stefankangas@gmail.com, emacs-devel@gnu.org
Subject: Re: master bdb13a0e5c5 1/2: Adjust to Gnulib’s recent module renaming
Date: Thu, 02 Jan 2025 22:30:53 +0200 [thread overview]
Message-ID: <86ldvthsaq.fsf@gnu.org> (raw)
In-Reply-To: <6414b78b-ff06-4d51-9bc4-b36799343fe1@cs.ucla.edu> (message from Paul Eggert on Thu, 2 Jan 2025 11:38:34 -0800)
> Date: Thu, 2 Jan 2025 11:38:34 -0800
> Cc: emacs-devel@gnu.org
> From: Paul Eggert <eggert@cs.ucla.edu>
>
> On 2025-01-02 09:57, Stefan Kangas wrote:
> > I'd find a "warning: new modules added, notify emacs-devel for
> > possible w32 adaption" message more useful than something that blindly
> > pipes things to mail(1).
>
> Thanks for the suggestion; I installed the attached.
Thanks, this is an improvement.
> I don't see a nontrivial way to update nt/gnulib-cfg.mk automatically,
> even just for module renames as Eli suggested, as there are too many
> possibilities for screwups.
Can you elaborate on that? I only meant to edit the
OMIT_GNULIB_MODULE_* settings when a module gets renamed, nothing
else. That is, if module FOO is renamed to FOO_BAR, the
OMIT_GNULIB_MODULE_FOO should be edited to OMIT_GNULIB_MODULE_FOO_BAR.
> The "right" way to fix this would be to adjust Emacs and Gnulib so that
> builds would work the same way on MS-Windows that they do everywhere
> else (i.e., nt/gnulib-cfg.mk would become unnecessary). However, as I
> understand it this would be nontrivial and anyway it would require
> MS-Windows expertise that I lack.
The problem here is that we omit modules because Emacs on Windows
implements or emulates many functionalities in a way that is specific
to Emacs and might not work well elsewhere. One notable example is
that Emacs supports UTF-8 encoded file names, something Gnulib on
Windows does not, so we cannot use any Gnulib modules that handle file
names. Gnulib is a general-purpose library, so it cannot use some of
the enhanced functionality we use in Emacs because we have intimate
knowledge of what Emacs needs and which functionalities it actually
uses. For example, the emulation of pselect is tightly coupled to
what Emacs uses it for.
> In the meantime, module renames are rare enough that doing things by
> hand should take less work overall. Besides, the main problem here is
> changes to modules and/or adding modules, which I don't think
> admin/merge-gnulib can handle by itself.
Yes, all the other changes cannot be automated. But I thought at
least renames can be.
next prev parent reply other threads:[~2025-01-02 20:30 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <mailman.2001.1735773573.32144.emacs-diffs@gnu.org>
2025-01-02 17:30 ` master bdb13a0e5c5 1/2: Adjust to Gnulib’s recent module renaming Eli Zaretskii
2025-01-02 17:57 ` Stefan Kangas
2025-01-02 19:38 ` Paul Eggert
2025-01-02 20:30 ` Eli Zaretskii [this message]
2025-01-02 21:15 ` Paul Eggert
2025-01-03 6:42 ` Eli Zaretskii
2025-01-03 19:06 ` Paul Eggert
2025-01-03 20:01 ` Eli Zaretskii
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
List information: https://www.gnu.org/software/emacs/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=86ldvthsaq.fsf@gnu.org \
--to=eliz@gnu.org \
--cc=eggert@cs.ucla.edu \
--cc=emacs-devel@gnu.org \
--cc=stefankangas@gmail.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.
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs.git
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).