Hi,

does that really solve your issue? With this patch applied, I run into the 
following build error because with the `-d` option, all arguments are treated 
as directories. So we should either use `-D` or create the directory accoring 
to the other created dirs in do_install

> 
> install: cannot create directory
> ‘/workdir/build/tmp/work/core2-64-wlanvm-linux/libvirt/v10.5.0+git/sources-unpack/libvirt-qemu.conf’:
> File exists
> install: cannot create directory
> ‘/workdir/build/tmp/work/core2-64-wlanvm-linux/libvirt/v10.5.0+git/image/usr/lib/sysusers.d/libvirt-qemu.conf’:
> File exists
> 

So I propose the following patch. Please let me know what you think.
Side note: This is my first contribution, I hope the format is correct, 
otherwise please let me know. I cannot use `git send-email`, so this is written 
from the webinterface.
Best regards,
Matthis

Patch:

3902ac8844a9868ab14fa38b663d3832daa4b9de leads to the following build
errors:
install: cannot create directory ‘{UNPACKDIR}/libvirt-qemu.conf’: File exists
install: cannot create directory ‘{D}/usr/lib/sysusers.d/libvirt-qemu.conf’: 
File exists
because `-d` treats all arguments as directories. Create the required
directory the same way as the other directories.
Signed-off-by: Matthis Hauschild <[email protected]>
---
recipes-extended/libvirt/libvirt_git.bb | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/recipes-extended/libvirt/libvirt_git.bb 
b/recipes-extended/libvirt/libvirt_git.bb
index bef98b69..5c5a9b3f 100644
--- a/recipes-extended/libvirt/libvirt_git.bb
+++ b/recipes-extended/libvirt/libvirt_git.bb
@@ -205,10 +205,11 @@ do_install:append() {
install -d ${D}/etc/init.d
install -d ${D}/etc/libvirt
install -d ${D}/etc/dnsmasq.d
+    install -d ${D}${nonarch_base_libdir}/sysusers.d

install -m 0755 ${UNPACKDIR}/libvirtd.sh ${D}/etc/init.d/libvirtd
install -m 0644 ${UNPACKDIR}/libvirtd.conf ${D}/etc/libvirt/libvirtd.conf
-    install -d -m 0644 ${UNPACKDIR}/libvirt-qemu.conf 
${D}${nonarch_base_libdir}/sysusers.d/libvirt-qemu.conf
+    install -m 0644 ${UNPACKDIR}/libvirt-qemu.conf 
${D}${nonarch_base_libdir}/sysusers.d/libvirt-qemu.conf

if ${@bb.utils.contains('DISTRO_FEATURES','sysvinit','true','false',d)}; then
# This will wind up in the libvirtd package, but will NOT be invoked by default.
--
2.43.0
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#8908): 
https://lists.yoctoproject.org/g/meta-virtualization/message/8908
Mute This Topic: https://lists.yoctoproject.org/mt/108714337/21656
Group Owner: [email protected]
Unsubscribe: https://lists.yoctoproject.org/g/meta-virtualization/unsub 
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to