From: Ross Burton <ross.burton@arm.com>
To: openembedded-core@lists.openembedded.org
Subject: [PATCH 4/4] recipes: remove redundant autoreconf excludes
Date: Mon, 30 Mar 2026 14:28:25 +0100 [thread overview]
Message-ID: <20260330132825.1784802-4-ross.burton@arm.com> (raw)
In-Reply-To: <20260330132825.1784802-1-ross.burton@arm.com>
With autoconf 2.73 autoreconf is better at not forcibly running aclocal
or autoheader if the relevant files are not managed by those tools.
This means we can remove a number of explicit excludes as they are no
longer needed.
Signed-off-by: Ross Burton <ross.burton@arm.com>
---
meta/recipes-connectivity/libpcap/libpcap_1.10.6.bb | 1 -
meta/recipes-connectivity/openssh/openssh_10.2p1.bb | 2 --
meta/recipes-connectivity/socat/socat_1.8.1.1.bb | 2 --
meta/recipes-core/readline/readline.inc | 3 ---
meta/recipes-devtools/diffstat/diffstat_1.68.bb | 2 --
meta/recipes-devtools/expect/expect_5.45.4.bb | 2 --
meta/recipes-devtools/help2man/help2man_1.49.3.bb | 4 ----
meta/recipes-devtools/quilt/quilt.inc | 2 --
meta/recipes-devtools/ruby/ruby_4.0.1.bb | 2 --
meta/recipes-devtools/subversion/subversion_1.14.5.bb | 2 --
meta/recipes-devtools/tcltk/tcl_9.0.3.bb | 1 -
meta/recipes-devtools/tcltk8/tcl8_8.6.17.bb | 1 -
meta/recipes-extended/bash/bash.inc | 1 -
meta/recipes-extended/cups/cups.inc | 2 --
meta/recipes-sato/rxvt-unicode/rxvt-unicode.inc | 3 ---
meta/recipes-support/nettle/nettle_3.10.2.bb | 2 --
16 files changed, 32 deletions(-)
diff --git a/meta/recipes-connectivity/libpcap/libpcap_1.10.6.bb b/meta/recipes-connectivity/libpcap/libpcap_1.10.6.bb
index 1b10001035b..9b259396f06 100644
--- a/meta/recipes-connectivity/libpcap/libpcap_1.10.6.bb
+++ b/meta/recipes-connectivity/libpcap/libpcap_1.10.6.bb
@@ -25,7 +25,6 @@ EXTRA_OECONF = " \
--without-dag \
--without-dpdk \
"
-EXTRA_AUTORECONF += "--exclude=aclocal"
PACKAGECONFIG ??= "${@bb.utils.contains('DISTRO_FEATURES', 'bluetooth', 'bluez5', '', d)} \
${@bb.utils.filter('DISTRO_FEATURES', 'ipv6', d)} \
diff --git a/meta/recipes-connectivity/openssh/openssh_10.2p1.bb b/meta/recipes-connectivity/openssh/openssh_10.2p1.bb
index 929c9e4b9fb..96cb09acc0e 100644
--- a/meta/recipes-connectivity/openssh/openssh_10.2p1.bb
+++ b/meta/recipes-connectivity/openssh/openssh_10.2p1.bb
@@ -66,8 +66,6 @@ PACKAGECONFIG[hostkey-rsa] = ""
PACKAGECONFIG[hostkey-ecdsa] = ""
PACKAGECONFIG[hostkey-ed25519] = ""
-EXTRA_AUTORECONF += "--exclude=aclocal"
-
# login path is hardcoded in sshd
EXTRA_OECONF = "'LOGIN_PROGRAM=${base_bindir}/login' \
${@bb.utils.contains('DISTRO_FEATURES', 'pam', '--with-pam', '--without-pam', d)} \
diff --git a/meta/recipes-connectivity/socat/socat_1.8.1.1.bb b/meta/recipes-connectivity/socat/socat_1.8.1.1.bb
index f6f2a3304b9..e662c79a754 100644
--- a/meta/recipes-connectivity/socat/socat_1.8.1.1.bb
+++ b/meta/recipes-connectivity/socat/socat_1.8.1.1.bb
@@ -17,8 +17,6 @@ SRC_URI[sha256sum] = "5ebc636b7f427053f98806696521653a614c7e06464910353cbf54e232
inherit autotools
-EXTRA_AUTORECONF += "--exclude=autoheader"
-
EXTRA_OECONF += "ac_cv_have_z_modifier=yes \
ac_cv_header_bsd_libutil_h=no \
sc_cv_termios_ispeed=no \
diff --git a/meta/recipes-core/readline/readline.inc b/meta/recipes-core/readline/readline.inc
index 06630dc455c..8b07ecbcce7 100644
--- a/meta/recipes-core/readline/readline.inc
+++ b/meta/recipes-core/readline/readline.inc
@@ -17,9 +17,6 @@ SRC_URI += "file://inputrc"
inherit autotools texinfo
-# Ensure we don't delete the shipped aclocal.m4 or config.h.in
-EXTRA_AUTORECONF += "--exclude=autoheader,aclocal"
-
EXTRA_OECONF += "bash_cv_termcap_lib=ncurses --with-shared-termcap-library"
LEAD_SONAME = "libreadline.so"
diff --git a/meta/recipes-devtools/diffstat/diffstat_1.68.bb b/meta/recipes-devtools/diffstat/diffstat_1.68.bb
index 4bc18158378..ce1ba79c2d1 100644
--- a/meta/recipes-devtools/diffstat/diffstat_1.68.bb
+++ b/meta/recipes-devtools/diffstat/diffstat_1.68.bb
@@ -17,8 +17,6 @@ SRC_URI[sha256sum] = "89f9294a8ac74fcef6f1b9ac408f43ebedf8d208e3efe0b99b4acc16dc
inherit autotools gettext ptest
-EXTRA_AUTORECONF += "--exclude=aclocal"
-
LDFLAGS += "${TOOLCHAIN_OPTIONS}"
do_install_ptest() {
diff --git a/meta/recipes-devtools/expect/expect_5.45.4.bb b/meta/recipes-devtools/expect/expect_5.45.4.bb
index 48b5b0d934b..f24dd3b7c81 100644
--- a/meta/recipes-devtools/expect/expect_5.45.4.bb
+++ b/meta/recipes-devtools/expect/expect_5.45.4.bb
@@ -39,8 +39,6 @@ UPSTREAM_CHECK_REGEX = "/Expect/(?P<pver>(\d+[\.\-_]*)+)/"
S = "${UNPACKDIR}/${BPN}${PV}"
-EXTRA_AUTORECONF += "--exclude=aclocal"
-
CFLAGS += "-std=gnu17"
do_install:append() {
diff --git a/meta/recipes-devtools/help2man/help2man_1.49.3.bb b/meta/recipes-devtools/help2man/help2man_1.49.3.bb
index 75931a511f5..1ba8d6090a1 100644
--- a/meta/recipes-devtools/help2man/help2man_1.49.3.bb
+++ b/meta/recipes-devtools/help2man/help2man_1.49.3.bb
@@ -10,10 +10,6 @@ SRC_URI[sha256sum] = "4d7e4fdef2eca6afe07a2682151cea78781e0a4e8f9622142d9f70c083
inherit autotools
-# This is a hand-maintained aclocal.m4 but our autotools class currently deletes
-# aclocal.m4.
-EXTRA_AUTORECONF += "--exclude=aclocal"
-
EXTRA_OECONF = "--disable-nls"
do_install:append () {
diff --git a/meta/recipes-devtools/quilt/quilt.inc b/meta/recipes-devtools/quilt/quilt.inc
index 59d13237e2f..387e132b602 100644
--- a/meta/recipes-devtools/quilt/quilt.inc
+++ b/meta/recipes-devtools/quilt/quilt.inc
@@ -28,8 +28,6 @@ CLEANBROKEN = "1"
EXTRA_OECONF = "--with-perl='${USRBINPATH}/env perl' --with-patch=patch --without-sendmail"
EXTRA_OECONF:append:class-native = " --disable-nls"
-EXTRA_AUTORECONF += "--exclude=aclocal"
-
CACHED_CONFIGUREVARS += "ac_cv_path_BASH=/bin/bash ac_cv_path_COLUMN=column"
# Make sure we don't have "-w" in shebang lines: it breaks using
diff --git a/meta/recipes-devtools/ruby/ruby_4.0.1.bb b/meta/recipes-devtools/ruby/ruby_4.0.1.bb
index e51d0d3e9cf..93df86333ea 100644
--- a/meta/recipes-devtools/ruby/ruby_4.0.1.bb
+++ b/meta/recipes-devtools/ruby/ruby_4.0.1.bb
@@ -34,8 +34,6 @@ UPSTREAM_CHECK_URI = "https://www.ruby-lang.org/en/downloads/"
inherit autotools ptest pkgconfig
-EXTRA_AUTORECONF += "--exclude=aclocal"
-
# This snippet lets compiled extensions which rely on external libraries,
# such as zlib, compile properly. If we don't do this, then when extmk.rb
# runs, it uses the native libraries instead of the target libraries, and so
diff --git a/meta/recipes-devtools/subversion/subversion_1.14.5.bb b/meta/recipes-devtools/subversion/subversion_1.14.5.bb
index 47e2b6c6932..13bec4ad595 100644
--- a/meta/recipes-devtools/subversion/subversion_1.14.5.bb
+++ b/meta/recipes-devtools/subversion/subversion_1.14.5.bb
@@ -42,8 +42,6 @@ EXTRA_OEMAKE += "pkgconfig_dir=${libdir}/pkgconfig"
CPPFLAGS += "-P"
BUILD_CPPFLAGS += "-P"
-EXTRA_AUTORECONF += "--exclude=aclocal"
-
do_configure:prepend () {
# autogen.sh manually puts things in the right place, and libtoolize doesn't
# work as they don't use aclocal
diff --git a/meta/recipes-devtools/tcltk/tcl_9.0.3.bb b/meta/recipes-devtools/tcltk/tcl_9.0.3.bb
index 18de3a333a2..ce8598fee6d 100644
--- a/meta/recipes-devtools/tcltk/tcl_9.0.3.bb
+++ b/meta/recipes-devtools/tcltk/tcl_9.0.3.bb
@@ -34,7 +34,6 @@ VER = "${PV}"
inherit autotools ptest binconfig
AUTOTOOLS_SCRIPT_PATH = "${S}/unix"
-EXTRA_AUTORECONF = "--exclude=aclocal"
EXTRA_OECONF = "--disable-rpath --enable-man-suffix=tcl9 --disable-zipfs"
diff --git a/meta/recipes-devtools/tcltk8/tcl8_8.6.17.bb b/meta/recipes-devtools/tcltk8/tcl8_8.6.17.bb
index 968cd6778e8..1b08fda5fa6 100644
--- a/meta/recipes-devtools/tcltk8/tcl8_8.6.17.bb
+++ b/meta/recipes-devtools/tcltk8/tcl8_8.6.17.bb
@@ -38,7 +38,6 @@ VER = "${PV}"
inherit autotools ptest binconfig
AUTOTOOLS_SCRIPT_PATH = "${S}/unix"
-EXTRA_AUTORECONF = "--exclude=aclocal"
EXTRA_OECONF = "--enable-threads --disable-rpath --enable-man-suffix=tcl8"
diff --git a/meta/recipes-extended/bash/bash.inc b/meta/recipes-extended/bash/bash.inc
index 634209c9115..858088cc772 100644
--- a/meta/recipes-extended/bash/bash.inc
+++ b/meta/recipes-extended/bash/bash.inc
@@ -7,7 +7,6 @@ DEPENDS = "ncurses bison-native virtual/libiconv"
inherit autotools gettext texinfo update-alternatives ptest
-EXTRA_AUTORECONF += "--exclude=autoheader,aclocal"
EXTRA_OECONF = "--enable-job-control --without-bash-malloc bash_cv_wexitstatus_offset=8"
# If NON_INTERACTIVE_LOGIN_SHELLS is defined, all login shells read the
diff --git a/meta/recipes-extended/cups/cups.inc b/meta/recipes-extended/cups/cups.inc
index 24ebcc4aae5..2724ce72fba 100644
--- a/meta/recipes-extended/cups/cups.inc
+++ b/meta/recipes-extended/cups/cups.inc
@@ -63,8 +63,6 @@ EXTRA_OECONF = " \
DSOFLAGS='${LDFLAGS}' \
"
-EXTRA_AUTORECONF += "--exclude=autoheader"
-
do_install () {
oe_runmake "BUILDROOT=${D}" install
diff --git a/meta/recipes-sato/rxvt-unicode/rxvt-unicode.inc b/meta/recipes-sato/rxvt-unicode/rxvt-unicode.inc
index 5a1b44a71c3..f8c0d8ab838 100644
--- a/meta/recipes-sato/rxvt-unicode/rxvt-unicode.inc
+++ b/meta/recipes-sato/rxvt-unicode/rxvt-unicode.inc
@@ -36,9 +36,6 @@ EXTRA_OECONF = "--enable-xim \
PACKAGECONFIG ??= ""
PACKAGECONFIG[startup] = "--enable-startup-notification,--disable-startup-notification,startup-notification,"
-# Ensure we don't delete the shipped aclocal.m4
-EXTRA_AUTORECONF += "--exclude=aclocal"
-
do_compile:prepend () {
echo '#define UTMP_FILE "${localstatedir}/run/utmp"' >> config.h
echo '#define WTMP_FILE "${localstatedir}/log/wtmp"' >> config.h
diff --git a/meta/recipes-support/nettle/nettle_3.10.2.bb b/meta/recipes-support/nettle/nettle_3.10.2.bb
index e651f68cb94..7b7be5f5941 100644
--- a/meta/recipes-support/nettle/nettle_3.10.2.bb
+++ b/meta/recipes-support/nettle/nettle_3.10.2.bb
@@ -26,8 +26,6 @@ UPSTREAM_CHECK_REGEX = "nettle-(?P<pver>\d+(\.\d+)+)\.tar"
inherit autotools ptest multilib_header lib_package
-EXTRA_AUTORECONF += "--exclude=aclocal"
-
EXTRA_OECONF = "--disable-openssl"
EXTRA_OECONF:append:armv7a = "${@bb.utils.contains("TUNE_FEATURES","neon",""," --disable-arm-neon --disable-fat",d)}"
--
2.43.0
prev parent reply other threads:[~2026-03-30 13:28 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-03-30 13:28 [PATCH 1/4] gnutls: update patch status Ross Burton
2026-03-30 13:28 ` [PATCH 2/4] diffstat: change configure.in to use standard autoconf Ross Burton
2026-03-30 13:45 ` Patchtest results for " patchtest
2026-03-30 13:28 ` [PATCH 3/4] autoconf: upgrade to 2.73 Ross Burton
2026-03-30 13:31 ` [OE-core] " Ross Burton
2026-04-01 20:33 ` Khem Raj
2026-04-07 19:43 ` Ross Burton
2026-04-07 20:46 ` Khem Raj
2026-03-30 13:28 ` Ross Burton [this message]
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=20260330132825.1784802-4-ross.burton@arm.com \
--to=ross.burton@arm.com \
--cc=openembedded-core@lists.openembedded.org \
/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