Re: [PATCH 1/4] linux-dtb.inc: use absolute upd-alt paths

Denys Dmytriyenko
2016-12-21T19:10:33Z

On Thu, Nov 12, 2015 at 04:41:28PM -0700, Christopher Larson wrote:
> From: Christopher Larson <chris_larson@mentor.com>
>
> This works around a limitation of the chkconfig update-alternatives, so it
> works with all our update-alternatives providers.

I know this change is old and has been in for over a year...

But the side-effect of it is that it breaks NFS-boot due to absolute DTB
symlinks. We had to implement a local workaround to switch them back to
relative links.

Is there another way to fix the original issue w/o absolute links?

--
Denys

> Signed-off-by: Christopher Larson <chris_larson@mentor.com>
> ---
> meta/recipes-kernel/linux/linux-dtb.inc | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/meta/recipes-kernel/linux/linux-dtb.inc b/meta/recipes-kernel/linux/linux-dtb.inc
> index ee3a5e1..5810b10 100644
> --- a/meta/recipes-kernel/linux/linux-dtb.inc
> +++ b/meta/recipes-kernel/linux/linux-dtb.inc
> @@ -64,7 +64,7 @@ pkg_postinst_kernel-devicetree () {
> do
> DTB_BASE_NAME=`basename ${DTB_FILE} | awk -F "." '{print $1}'`
> DTB_SYMLINK_NAME=`echo ${KERNEL_IMAGE_SYMLINK_NAME} | sed "s/${MACHINE}/${DTB_BASE_NAME}/g"`
> - update-alternatives --install /${KERNEL_IMAGEDEST}/${DTB_BASE_NAME}.dtb ${DTB_BASE_NAME}.dtb devicetree-${DTB_SYMLINK_NAME}.dtb ${KERNEL_PRIORITY} || true
> + update-alternatives --install /${KERNEL_IMAGEDEST}/${DTB_BASE_NAME}.dtb ${DTB_BASE_NAME}.dtb /boot/devicetree-${DTB_SYMLINK_NAME}.dtb ${KERNEL_PRIORITY} || true
> done
> }
>
> @@ -74,6 +74,6 @@ pkg_postrm_kernel-devicetree () {
> do
> DTB_BASE_NAME=`basename ${DTB_FILE} | awk -F "." '{print $1}'`
> DTB_SYMLINK_NAME=`echo ${KERNEL_IMAGE_SYMLINK_NAME} | sed "s/${MACHINE}/${DTB_BASE_NAME}/g"`
> - update-alternatives --remove ${DTB_BASE_NAME}.dtb devicetree-${DTB_SYMLINK_NAME}.dtb ${KERNEL_PRIORITY} || true
> + update-alternatives --remove ${DTB_BASE_NAME}.dtb /boot/devicetree-${DTB_SYMLINK_NAME}.dtb ${KERNEL_PRIORITY} || true
> done
> }
> --
> 2.2.1
>
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core