Re: [PATCH] libvirt: do not install /var/log for target

Bruce Ashfield

On Thu, Nov 11, 2021 at 11:49 PM Kai <kai.kang@...> wrote:

On 11/12/21 12:03 PM, Bruce Ashfield wrote:
On Thu, Nov 11, 2021 at 10:23 PM kai <kai.kang@...> wrote:
From: Kai Kang <kai.kang@...>

/var/log is normally a link to /var/volatile/log and /var/volatile is a tmpfs
mount. So anything created in /var/log will not be available when the tmpfs is

Not install /var/log to avoid 'empty-dirs' qa issue:

| ERROR: libvirt-7.2.0-r0 do_package_qa: QA Issue: libvirt installs files in
/var/volatile, but it is expected to be empty [empty-dirs]

Signed-off-by: Kai Kang <kai.kang@...>
recipes-extended/libvirt/ | 4 ++++
1 file changed, 4 insertions(+)

diff --git a/recipes-extended/libvirt/ b/recipes-extended/libvirt/
index b7f8383a..b49bd11a 100644
--- a/recipes-extended/libvirt/
+++ b/recipes-extended/libvirt/
@@ -232,6 +232,10 @@ do_install:append() {
echo "d root root 0755 /run/libvirt/qemu none" \
>> ${D}${sysconfdir}/default/volatiles/99_libvirt

+ # /var/log is normally a link to /var/volatile/log
+ # and /var/volatile is a tmpfs mount which should be clean.
+ rm -rf ${D}${localstatedir}/log
We should be preventing the install from happening at all, not
deleting it after the fact.

If libvirt thinks something needs to be in the directory, then we
should be looking into why it does exactly that.
OK.I'll figure out.
I poked around at this myself, and then asked Ross to help me out .. I
assumed it was meson doing this, and he pointed out the right files to me.

I'm trying to figure out how to balance some non volatile use cases,
with this new QA check. If we universally delete those files, then
those cases are broken.

I'm leaning towards leaving the directories in place, and just
inhibiting the QA check.




# Manually set permissions and ownership to match polkit recipe
if ${@bb.utils.contains('PACKAGECONFIG', 'polkit', 'true', 'false', d)}; then
install -d -m 0700 ${D}/${datadir}/polkit-1/rules.d

Kai Kang
Wind River Linux

- Thou shalt not follow the NULL pointer, for chaos and madness await
thee at its end
- "Use the force Harry" - Gandalf, Star Trek II

Join to automatically receive all group messages.