All of lore.kernel.org
 help / color / mirror / Atom feed
From: Denys Dmytriyenko <denis@denix.org>
To: Christopher Larson <kergoth@gmail.com>
Cc: Christopher Larson <chris_larson@mentor.com>,
	openembedded-core@lists.openembedded.org
Subject: Re: [PATCH 1/4] linux-dtb.inc: use absolute upd-alt paths
Date: Wed, 21 Dec 2016 14:10:30 -0500	[thread overview]
Message-ID: <20161221191030.GF30576@denix.org> (raw)
In-Reply-To: <130d9a6c1f4ecea86748a44b73f51d8ddb2e63da.1447371049.git.chris_larson@mentor.com>

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


  reply	other threads:[~2016-12-21 19:10 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-11-12 23:41 [PATCH 0/4] Misc improvements 2 from Mentor Christopher Larson
2015-11-12 23:41 ` [PATCH 1/4] linux-dtb.inc: use absolute upd-alt paths Christopher Larson
2016-12-21 19:10   ` Denys Dmytriyenko [this message]
2016-12-21 20:18     ` Christopher Larson
2016-12-22 18:34       ` Denys Dmytriyenko
2015-11-12 23:41 ` [PATCH 2/4] opkg-utils: add update-alternatives PACKAGECONFIG Christopher Larson
2016-01-15  9:23   ` Robert Yang
2016-01-15 20:07     ` Christopher Larson
2016-01-16  5:57       ` Robert Yang
2015-11-12 23:41 ` [PATCH 3/4] systemd: add myhostname to nsswitch.conf Christopher Larson
2015-11-17  7:25   ` Andreas Müller
2015-11-17 13:55     ` Christopher Larson
2015-11-18  3:49   ` Dan McGregor
2015-12-07 19:49     ` Christopher Larson
2015-12-10 18:33       ` Dan McGregor
2015-11-12 23:41 ` [PATCH 4/4] systemd: arrange for volatile /etc/resolv.conf Christopher Larson
2016-01-05 19:08 ` [PATCH 0/4] Misc improvements 2 from Mentor Christopher Larson
2016-01-05 20:26   ` Burton, Ross

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20161221191030.GF30576@denix.org \
    --to=denis@denix.org \
    --cc=chris_larson@mentor.com \
    --cc=kergoth@gmail.com \
    --cc=openembedded-core@lists.openembedded.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.