Re: [PATCH] Install new autoreconf files.

Rene Kita <[email protected]>
Newsgroups gmane.mail.mutt.devel
Message-ID <[email protected]>
On Tue, Apr 14, 2026 at 08:07:10AM +0800, Kevin J. McCarthy wrote:
[...]
> I think I'd like to try removing the files from the repos, because if I just
> update them again it will merely cause problems yet again in a few years.
> 
> Here's what I plan on doing:
> 
> 1. Fix up the trixie docker image to include autopoint.  Just for reference,
> I store these in the "docker-images" repository on gitlab:
> https://gitlab.com/muttmua/docker-images
> 
> 2. Add "AM_GNU_GETTEXT_VERSION(0.21)" to configure.ac
> 
> 3. Remove and gitignore files that are reported as "Copy file" by
> `gettextize -f --dry-run --no-changelog`
> 
> 4. Change prepare script to "autoreconf -isvf".  I like the idea of the
> symlinks, since it makes it more obvious they are external files.
> 
> Any comments?  Did I miss a step?

Sounds reasonable.

> Matthias, I saw you also posted some fixes to the .gitlab-ci.yml that fixed
> the "current directory" assumptions.  Would you please send a patch with
> that change to this mailing list so Rene can easily review it too?

I fetched the MR, no objections from my side. Two small nits:
(1) I would use more consistent line wrapping in the Makefiles, e.g.:
#v+
+	(find $(top_srcdir) -name \*.sh \
+		&& echo "$(srcdir)/gen_defs $(srcdir)/muttbug $(srcdir)/prepare") \
+		| xargs shellcheck
#v-

(2) Those file name prefixes (Makefile.am:, configure.ac:, CI:) in the
   commit messages seem to be unusual for this project.

(1) is really not important.
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.