Ben Hutchings <ben@xxxxxxxxxxxxxxx> writes: > [Please submit patches inline.] I've no way to control the way quilt is sending mails. > > On Wed, 2015-01-14 at 13:32 +0100, Arnaud Patard wrote: >> When building a package with make deb-pkg (say, for arm), the dtb files are >> not added to the package. Given that things are still evolving on arm, it >> make sense to have them along with the kernel and modules. >> >> v2: make use of dtbs_install >> >> Signed-off-by: Arnaud Patard <arnaud.patard@xxxxxxxxxxx> >> --- >> >> Index: linux-next/scripts/package/builddeb >> =================================================================== >> --- linux-next.orig/scripts/package/builddeb 2015-01-14 13:04:45.845922441 +0100 >> +++ linux-next/scripts/package/builddeb 2015-01-14 13:19:26.121883720 +0100 >> @@ -143,6 +143,10 @@ else >> cp arch/$ARCH/boot/$KBUILD_IMAGE "$tmpdir/$installed_image_path" >> fi >> >> +if grep -q "^CONFIG_OF=y" .config ; then >> + make INSTALL_DTBS_PATH="$tmpdir/usr/lib/$packagename" dtbs_install >> +fi > > Only arm and arm64 support that target. You should maybe run something > like 'make -n dtbs_install >/dev/null 2>&1' first to check that the > target is defined. There's a 'set -e' on top of the script so using make -n will likely result in the script failing, which wouldn't be so nice imho. >From a quick untested guess, I see 2 way of solving this: make INSTALL_DTBS_PATH="$tmpdir/usr/lib/$packagename" dtbs_install || /bin/true or set +e make -n dtbs_install >/dev/null 2>&1 ret=$? set -e if [ $ret -eq 0 ]; then make INSTALL_DTBS_PATH="$tmpdir/usr/lib/$packagename" dtbs_install fi Any preference ? Arnaud -- To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
