Re: [PATCH] meta: image: detect broken symlinks

[Henning Schild <henning.schild@siemens.com>]

Am Sun, 4 Mar 2018 22:28:36 +0300
schrieb Alexander Smirnov <asmirnov@ilbers.de>:

> On 03/02/2018 07:30 PM, Alexander Smirnov wrote:
> > On 02/27/2018 08:23 PM, Henning Schild wrote:  
> >> Switch get_image_name from os.path.lexists to os.path.exists
> >> because the latter detects broken symlinks correctly.
> >>
> >> Issue:
> >> If you build a kernel without CONFIG_BLK_DEV_INITRD the postinst
> >> hook will
> >> still call "linux-update-symlinks" and create a broken symlink.
> >> After that os.path.lexists will return True and "sudo cp" in
> >> do_copy_boot_files will fail.
> >>  
> > 
> > Applied, thanks.
> > 
> > P.S.: fixed to upper case after semicolon in patch topic.  
> 
> Dropped from the next branch. The patch breaks 'start_vm' script, 
> $INITRD_IMAGE variable is no more valid with it.

Ok, that is a bug in the script since it can not handle the valid case
that the variable is not set. Anyways, i updated the script along with
my patch and your jenkins is currently testing that.
Will send an updated version of that patch soon.

Henning

> Alex
>