diff mbox series

[4/4] vim.inc: remove xxd from the destination directory

Message ID 20260508214020.2488025-5-joaomarcos.costa@bootlin.com
State New
Headers show
Series Split vim and xxd in different recipes | expand

Commit Message

Joao Marcos Costa May 8, 2026, 9:40 p.m. UTC
Since the default target depends on xxd, building vim leads to building xxd
as well, and this raises (for instance) the error below:

error: check_data_file_clashes: Package vim wants to install file
/src/build/tmp/work/qemuriscv64-oe-linux/core-image-minimal/1.0/rootfs/usr/bin/xxd
But that file is already provided by package vim-xxd

This is not the ideal approach, because we are just removing something
after it is compiled instead of simply not compiling it in the first place.

That being said, merely compiling xxd does not add much (or any?) actual overhead.

Signed-off-by: João Marcos Costa <joaomarcos.costa@bootlin.com>
---
 meta/recipes-support/vim/vim.inc | 4 ++++
 1 file changed, 4 insertions(+)
diff mbox series

Patch

diff --git a/meta/recipes-support/vim/vim.inc b/meta/recipes-support/vim/vim.inc
index 74777864e7..4a7092e4a0 100644
--- a/meta/recipes-support/vim/vim.inc
+++ b/meta/recipes-support/vim/vim.inc
@@ -97,6 +97,10 @@  do_install() {
 	# delete the block.
 	sed -i '/the mouse works just fine/,+4d' ${D}/${datadir}/${BPN}/vimrc
     fi
+
+    # FIXME: ideally, Vim's makefiles should be tweaked to not build xxd,
+    # considering xxd (vim-xxd) has its own recipe now
+    rm ${D}${bindir}/xxd
 }
 
 PARALLEL_MAKEINST = ""