diff options
author | Hongxu Jia <hongxu.jia@windriver.com> | 2025-09-07 20:17:18 -0700 |
---|---|---|
committer | Steve Sakoman <steve@sakoman.com> | 2025-09-22 12:21:23 -0700 |
commit | 10b0d27225335513968f51f50d4c9e4b301bb633 (patch) | |
tree | 6f0ad458f21d10ad65391d081de7aa7d6e1e7f90 | |
parent | 0aefe8656329951d11db1acd18356d82212e654f (diff) | |
download | poky-10b0d27225335513968f51f50d4c9e4b301bb633.tar.gz |
rpm: correct tool path in macros for no usrmerge
While no usrmerge in sysvinit, some tools defined in rpm macro have wrong path
$ echo 'INIT_MANAGER="sysvinit"' >> conf/local.conf
$ echo 'IMAGE_INSTALL:append = " rpm busybox"' >> conf/local.conf
$ bitbake core-image-minimal
$ runqemu tmp/deploy/images/qemux86-64/core-image-minimal-qemux86-64.rootfs.qemuboot.conf
root@qemux86-64:~# which sed tar rm mkdir cp cat chown chmod gzip grep mv
/bin/sed
/bin/tar
/bin/rm
/bin/mkdir
/bin/cp
/bin/cat
/bin/chown
/bin/chmod
/bin/gzip
/bin/grep
/bin/mv
root@qemux86-64:~# rpm --eval "%{__sed} %{__tar} %{__rm} %{__mkdir} %{__cp} %{__cat} %{__chown} %{__chmod} %{__gzip} %{__grep} %{__mv}"
/usr/bin/sed /usr/bin/tar /usr/bin/rm /usr/bin/mkdir /usr/bin/cp /usr/bin/cat /usr/bin/chown /usr/bin/chmod /usr/bin/gzip /usr/bin/grep /usr/bin/mv
Here to explain how __rm was set in rpm during build. The build system
of rpm is cmake. Take rpm rpm-4.19.x for example:
The '__RM rm' is defected by findutil [1], and function findutil
calls find_program to search for tool, if not found on host, then
hardcode with "/usr/bin" prefix [2]
Yocto explicitly set OECMAKE_FIND_ROOT_PATH_MODE_PROGRAM = "ONLY" [3][4]
to search tools from CMAKE_FIND_ROOT_PATH [5] which locates in recipe sysroot,
if not found in recipe sysroot, hardcode with "/usr/bin" prefix
If "${base_bindir}" != "${bindir}, explicitly correct tools in rpm
macros, use ${base_bindir} to instead original ${bindir}. Only do the
operation for target, it is not necessary for native and nativesdk,
because most host distribution supports usrmerge
After applying this commit, on target:
root@qemux86-64:~# rpm --eval "%{__sed} %{__tar} %{__rm} %{__mkdir} %{__cp} %{__cat} %{__chown} %{__chmod} %{__gzip} %{__grep} %{__mv}"
/bin/sed /bin/tar /bin/rm /bin/mkdir /bin/cp /bin/cat /bin/chown /bin/chmod /bin/gzip /bin/grep /bin/mv
root@qemux86-64:~# ls /bin/sed /bin/tar /bin/rm /bin/mkdir /bin/cp /bin/cat /bin/chown /bin/chmod /bin/gzip /bin/grep /bin/mv
/bin/cat /bin/chmod /bin/chown /bin/cp /bin/grep /bin/gzip /bin/mkdir /bin/mv /bin/rm /bin/sed /bin/tar
In order to save size, this commit does not add these tools to
runtime depends, user should explicitly add them if necessary
(such as use rpm to build packages)
[1] https://github.com/rpm-software-management/rpm/blob/rpm-4.19.x/CMakeLists.txt#L121
[2] https://github.com/rpm-software-management/rpm/blob/rpm-4.19.x/CMakeLists.txt#L59
[3] https://git.openembedded.org/openembedded-core/commit/?id=f4ea12f6635125ee793f4dd801c538c0186f9dc3
[4] https://cmake.org/cmake/help/latest/variable/CMAKE_FIND_ROOT_PATH_MODE_PROGRAM.html
[5] https://git.openembedded.org/openembedded-core/tree/meta/classes-recipe/cmake.bbclass?id=f4ea12f6635125ee793f4dd801c538c0186f9dc3#n123
(From OE-Core rev: 3958989d976fb429c84b12427bab3db0bc0d5d16)
Signed-off-by: Hongxu Jia <hongxu.jia@windriver.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
(master rev: c89c7177be2df5d2be44478a6ac43b35ad46db9e)
Signed-off-by: Hongxu Jia <hongxu.jia@windriver.com>
Signed-off-by: Steve Sakoman <steve@sakoman.com>
-rw-r--r-- | meta/recipes-devtools/rpm/rpm_4.20.0.bb | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/meta/recipes-devtools/rpm/rpm_4.20.0.bb b/meta/recipes-devtools/rpm/rpm_4.20.0.bb index 281fde1c82..4ded77e8ce 100644 --- a/meta/recipes-devtools/rpm/rpm_4.20.0.bb +++ b/meta/recipes-devtools/rpm/rpm_4.20.0.bb | |||
@@ -97,6 +97,8 @@ WRAPPER_TOOLS = " \ | |||
97 | ${libdir}/rpm/rpmdeps \ | 97 | ${libdir}/rpm/rpmdeps \ |
98 | " | 98 | " |
99 | 99 | ||
100 | base_bindir_progs = "sed tar rm mv mkdir cp cat chown chmod gzip grep" | ||
101 | |||
100 | do_install:append:class-native() { | 102 | do_install:append:class-native() { |
101 | for tool in ${WRAPPER_TOOLS}; do | 103 | for tool in ${WRAPPER_TOOLS}; do |
102 | test -x ${D}$tool && create_wrapper ${D}$tool \ | 104 | test -x ${D}$tool && create_wrapper ${D}$tool \ |
@@ -119,9 +121,15 @@ do_install:append:class-nativesdk() { | |||
119 | EOF | 121 | EOF |
120 | } | 122 | } |
121 | 123 | ||
122 | # Rpm's make install creates var/tmp which clashes with base-files packaging | ||
123 | do_install:append:class-target() { | 124 | do_install:append:class-target() { |
125 | # Rpm's make install creates var/tmp which clashes with base-files packaging | ||
124 | rm -rf ${D}/var | 126 | rm -rf ${D}/var |
127 | |||
128 | if [ "${base_bindir}" != "${bindir}" ]; then | ||
129 | for prog in ${base_bindir_progs}; do | ||
130 | sed -i "s|^%__${prog}.*|%__${prog} ${base_bindir}/${prog}|g" ${D}${libdir}/rpm/macros | ||
131 | done | ||
132 | fi | ||
125 | } | 133 | } |
126 | do_install:append:class-nativesdk() { | 134 | do_install:append:class-nativesdk() { |
127 | rm -rf ${D}${SDKPATHNATIVE}/var | 135 | rm -rf ${D}${SDKPATHNATIVE}/var |