diff options
author | Richard Purdie <richard.purdie@linuxfoundation.org> | 2024-05-30 17:15:14 +0100 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2024-05-31 16:58:36 +0100 |
commit | 6b9eb4e9f04387b68e1b55e06b0d91edfa572abb (patch) | |
tree | 109ab229eedec3d928449df270885dd54c097ca7 | |
parent | 8fc598580e0222c022fedec79233b6de2320b0c4 (diff) | |
download | poky-6b9eb4e9f04387b68e1b55e06b0d91edfa572abb.tar.gz |
siteconfig: Drop siteconfig class/code/support
The siteconfig code was only used for 5 cache values. The complexity added to sstate
to support this code was considerable and the runtime much more significant than
any benefit the cache files would have added. Drop the support for this which
was only used minimally for ncurses and zlib.
(From OE-Core rev: f3766dc038f7ba9780ddaf5eb8d27385ea31d7d0)
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
-rw-r--r-- | meta/classes-recipe/autotools.bbclass | 2 | ||||
-rw-r--r-- | meta/classes-recipe/siteinfo.bbclass | 16 | ||||
-rw-r--r-- | meta/classes-recipe/toolchain-scripts.bbclass | 7 | ||||
-rw-r--r-- | meta/classes/siteconfig.bbclass | 39 | ||||
-rw-r--r-- | meta/recipes-core/ncurses/ncurses.inc | 2 | ||||
-rw-r--r-- | meta/recipes-core/ncurses/site_config/headers | 5 | ||||
-rw-r--r-- | meta/recipes-core/zlib/site_config/headers | 1 | ||||
-rwxr-xr-x | scripts/gen-site-config | 43 |
8 files changed, 1 insertions, 114 deletions
diff --git a/meta/classes-recipe/autotools.bbclass b/meta/classes-recipe/autotools.bbclass index 9359c9b4e1..7ee1b0d9c7 100644 --- a/meta/classes-recipe/autotools.bbclass +++ b/meta/classes-recipe/autotools.bbclass | |||
@@ -254,8 +254,6 @@ autotools_do_install() { | |||
254 | fi | 254 | fi |
255 | } | 255 | } |
256 | 256 | ||
257 | inherit siteconfig | ||
258 | |||
259 | EXPORT_FUNCTIONS do_configure do_compile do_install | 257 | EXPORT_FUNCTIONS do_configure do_compile do_install |
260 | 258 | ||
261 | B = "${WORKDIR}/build" | 259 | B = "${WORKDIR}/build" |
diff --git a/meta/classes-recipe/siteinfo.bbclass b/meta/classes-recipe/siteinfo.bbclass index 68aefb8eda..25b53d929a 100644 --- a/meta/classes-recipe/siteinfo.bbclass +++ b/meta/classes-recipe/siteinfo.bbclass | |||
@@ -221,20 +221,6 @@ def siteinfo_get_files(d, sysrootcache=False): | |||
221 | # This would show up as breaking sstatetests.SStateTests.test_sstate_32_64_same_hash for example | 221 | # This would show up as breaking sstatetests.SStateTests.test_sstate_32_64_same_hash for example |
222 | searched = [] | 222 | searched = [] |
223 | 223 | ||
224 | if not sysrootcache: | ||
225 | return sitefiles, searched | ||
226 | |||
227 | # Now check for siteconfig cache files in sysroots | ||
228 | path_siteconfig = d.getVar('SITECONFIG_SYSROOTCACHE') | ||
229 | if path_siteconfig and os.path.isdir(path_siteconfig): | ||
230 | for i in os.listdir(path_siteconfig): | ||
231 | if not i.endswith("_config"): | ||
232 | continue | ||
233 | filename = os.path.join(path_siteconfig, i) | ||
234 | sitefiles.append(filename) | ||
235 | return sitefiles, searched | 224 | return sitefiles, searched |
236 | 225 | ||
237 | # | 226 | |
238 | # Make some information available via variables | ||
239 | # | ||
240 | SITECONFIG_SYSROOTCACHE = "${STAGING_DATADIR}/${TARGET_SYS}_config_site.d" | ||
diff --git a/meta/classes-recipe/toolchain-scripts.bbclass b/meta/classes-recipe/toolchain-scripts.bbclass index cbef412e79..b59a295abc 100644 --- a/meta/classes-recipe/toolchain-scripts.bbclass +++ b/meta/classes-recipe/toolchain-scripts.bbclass | |||
@@ -201,7 +201,6 @@ EOF | |||
201 | 201 | ||
202 | #we get the cached site config in the runtime | 202 | #we get the cached site config in the runtime |
203 | TOOLCHAIN_CONFIGSITE_NOCACHE = "${@' '.join(siteinfo_get_files(d)[0])}" | 203 | TOOLCHAIN_CONFIGSITE_NOCACHE = "${@' '.join(siteinfo_get_files(d)[0])}" |
204 | TOOLCHAIN_CONFIGSITE_SYSROOTCACHE = "${STAGING_DIR}/${MLPREFIX}${MACHINE}/${target_datadir}/${TARGET_SYS}_config_site.d" | ||
205 | TOOLCHAIN_NEED_CONFIGSITE_CACHE ??= "virtual/${MLPREFIX}libc ncurses" | 204 | TOOLCHAIN_NEED_CONFIGSITE_CACHE ??= "virtual/${MLPREFIX}libc ncurses" |
206 | DEPENDS += "${TOOLCHAIN_NEED_CONFIGSITE_CACHE}" | 205 | DEPENDS += "${TOOLCHAIN_NEED_CONFIGSITE_CACHE}" |
207 | 206 | ||
@@ -223,14 +222,8 @@ toolchain_create_sdk_siteconfig () { | |||
223 | sitefile=`echo $sitefile | tr / _` | 222 | sitefile=`echo $sitefile | tr / _` |
224 | sitefile=`cat ${STAGING_DIR_TARGET}/sysroot-providers/$sitefile` | 223 | sitefile=`cat ${STAGING_DIR_TARGET}/sysroot-providers/$sitefile` |
225 | esac | 224 | esac |
226 | |||
227 | if [ -r ${TOOLCHAIN_CONFIGSITE_SYSROOTCACHE}/${sitefile}_config ]; then | ||
228 | cat ${TOOLCHAIN_CONFIGSITE_SYSROOTCACHE}/${sitefile}_config >> $siteconfig | ||
229 | fi | ||
230 | done | 225 | done |
231 | } | 226 | } |
232 | # The immediate expansion above can result in unwanted path dependencies here | ||
233 | toolchain_create_sdk_siteconfig[vardepsexclude] = "TOOLCHAIN_CONFIGSITE_SYSROOTCACHE" | ||
234 | 227 | ||
235 | python __anonymous () { | 228 | python __anonymous () { |
236 | import oe.classextend | 229 | import oe.classextend |
diff --git a/meta/classes/siteconfig.bbclass b/meta/classes/siteconfig.bbclass deleted file mode 100644 index 953cafd285..0000000000 --- a/meta/classes/siteconfig.bbclass +++ /dev/null | |||
@@ -1,39 +0,0 @@ | |||
1 | # | ||
2 | # Copyright OpenEmbedded Contributors | ||
3 | # | ||
4 | # SPDX-License-Identifier: MIT | ||
5 | # | ||
6 | |||
7 | python siteconfig_do_siteconfig () { | ||
8 | shared_state = sstate_state_fromvars(d) | ||
9 | if shared_state['task'] != 'populate_sysroot': | ||
10 | return | ||
11 | if not os.path.isdir(os.path.join(d.getVar('FILE_DIRNAME'), 'site_config')): | ||
12 | bb.debug(1, "No site_config directory, skipping do_siteconfig") | ||
13 | return | ||
14 | sstate_install(shared_state, d) | ||
15 | bb.build.exec_func('do_siteconfig_gencache', d) | ||
16 | sstate_clean(shared_state, d) | ||
17 | } | ||
18 | |||
19 | EXTRASITECONFIG ?= "" | ||
20 | |||
21 | siteconfig_do_siteconfig_gencache () { | ||
22 | mkdir -p ${WORKDIR}/site_config_${MACHINE} | ||
23 | gen-site-config ${FILE_DIRNAME}/site_config \ | ||
24 | >${WORKDIR}/site_config_${MACHINE}/configure.ac | ||
25 | cd ${WORKDIR}/site_config_${MACHINE} | ||
26 | autoconf | ||
27 | rm -f ${BPN}_cache | ||
28 | CONFIG_SITE="" ${EXTRASITECONFIG} ./configure ${CONFIGUREOPTS} --cache-file ${BPN}_cache | ||
29 | sed -n -e "/ac_cv_c_bigendian/p" -e "/ac_cv_sizeof_/p" \ | ||
30 | -e "/ac_cv_type_/p" -e "/ac_cv_header_/p" -e "/ac_cv_func_/p" \ | ||
31 | < ${BPN}_cache > ${BPN}_config | ||
32 | mkdir -p ${SYSROOT_DESTDIR}${datadir}/${TARGET_SYS}_config_site.d | ||
33 | cp ${BPN}_config ${SYSROOT_DESTDIR}${datadir}/${TARGET_SYS}_config_site.d | ||
34 | |||
35 | } | ||
36 | |||
37 | do_populate_sysroot[sstate-interceptfuncs] += "do_siteconfig " | ||
38 | |||
39 | EXPORT_FUNCTIONS do_siteconfig do_siteconfig_gencache | ||
diff --git a/meta/recipes-core/ncurses/ncurses.inc b/meta/recipes-core/ncurses/ncurses.inc index 2a3fd4f2c5..f5e37b94da 100644 --- a/meta/recipes-core/ncurses/ncurses.inc +++ b/meta/recipes-core/ncurses/ncurses.inc | |||
@@ -20,8 +20,6 @@ EXTRA_AUTORECONF = "-I m4" | |||
20 | CACHED_CONFIGUREVARS = "cf_cv_func_nanosleep=yes" | 20 | CACHED_CONFIGUREVARS = "cf_cv_func_nanosleep=yes" |
21 | CACHED_CONFIGUREVARS:append:linux = " cf_cv_working_poll=yes" | 21 | CACHED_CONFIGUREVARS:append:linux = " cf_cv_working_poll=yes" |
22 | 22 | ||
23 | EXTRASITECONFIG = "CFLAGS='${CFLAGS} -I${SYSROOT_DESTDIR}${includedir}'" | ||
24 | |||
25 | # Whether to enable separate widec libraries; must be 'true' or 'false' | 23 | # Whether to enable separate widec libraries; must be 'true' or 'false' |
26 | # | 24 | # |
27 | # TODO: remove this variable when widec is supported in every setup? | 25 | # TODO: remove this variable when widec is supported in every setup? |
diff --git a/meta/recipes-core/ncurses/site_config/headers b/meta/recipes-core/ncurses/site_config/headers deleted file mode 100644 index 087b7bfd5e..0000000000 --- a/meta/recipes-core/ncurses/site_config/headers +++ /dev/null | |||
@@ -1,5 +0,0 @@ | |||
1 | curses.h | ||
2 | ncurses/curses.h | ||
3 | ncurses.h | ||
4 | ncurses/termcap.h | ||
5 | |||
diff --git a/meta/recipes-core/zlib/site_config/headers b/meta/recipes-core/zlib/site_config/headers deleted file mode 100644 index 50268918aa..0000000000 --- a/meta/recipes-core/zlib/site_config/headers +++ /dev/null | |||
@@ -1 +0,0 @@ | |||
1 | zlib.h | ||
diff --git a/scripts/gen-site-config b/scripts/gen-site-config deleted file mode 100755 index 727b809c0f..0000000000 --- a/scripts/gen-site-config +++ /dev/null | |||
@@ -1,43 +0,0 @@ | |||
1 | #! /bin/sh | ||
2 | # Copyright (c) 2005-2008 Wind River Systems, Inc. | ||
3 | # | ||
4 | # SPDX-License-Identifier: GPL-2.0-only | ||
5 | # | ||
6 | |||
7 | cat << EOF | ||
8 | AC_PREREQ(2.57) | ||
9 | AC_INIT([site_wide],[1.0.0]) | ||
10 | |||
11 | EOF | ||
12 | |||
13 | # Disable as endian is set in the default config | ||
14 | #echo AC_C_BIGENDIAN | ||
15 | #echo | ||
16 | |||
17 | if [ -e $1/types ] ; then | ||
18 | while read type ; do | ||
19 | echo "AC_CHECK_SIZEOF([$type])" | ||
20 | done < $1/types | ||
21 | |||
22 | echo | ||
23 | fi | ||
24 | |||
25 | if [ -e $1/funcs ]; then | ||
26 | while read func ; do | ||
27 | echo "AC_CHECK_FUNCS([$func])" | ||
28 | done < $1/funcs | ||
29 | |||
30 | echo | ||
31 | fi | ||
32 | |||
33 | if [ -e $1/headers ]; then | ||
34 | while read header ; do | ||
35 | echo "AC_CHECK_HEADERS([$header])" | ||
36 | done < $1/headers | ||
37 | |||
38 | echo | ||
39 | fi | ||
40 | |||
41 | cat << EOF | ||
42 | AC_OUTPUT | ||
43 | EOF | ||