On Wed, 2020-06-24 at 09:47 +0800, Xu, Yanfei wrote:
> 
> On 6/24/20 5:51 AM, Richard Purdie wrote:
> > On Tue, 2020-06-23 at 10:31 -0400, Bruce Ashfield wrote:
> > > On Fri, Jun 19, 2020 at 7:47 AM <[email protected]> wrote:
> > > > From: Yanfei Xu <[email protected]>
> > > > 
> > > > Some filesystems don't support symlink, then you will get
> > > > failure
> > > > when
> > > > you install or update the kernel rpm package. Now we use a copy
> > > > of
> > > > image for these filesystems instead of symlink.
> > > > 
> > > > Suggested-by: Bruce Ashfield <[email protected]>
> > > > Suggested-by: Richard Purdie <
> > > > [email protected]>
> > > > Signed-off-by: Yanfei Xu <[email protected]>
> > > > ---
> > > >   meta/classes/kernel.bbclass | 19 ++++++++++++++++---
> > > >   1 file changed, 16 insertions(+), 3 deletions(-)
> > > > 
> > > > diff --git a/meta/classes/kernel.bbclass
> > > > b/meta/classes/kernel.bbclass
> > > > index 20a0135fc9..1269b54343 100644
> > > > --- a/meta/classes/kernel.bbclass
> > > > +++ b/meta/classes/kernel.bbclass
> > > > @@ -94,6 +94,22 @@ python __anonymous () {
> > > >           d.appendVar('RDEPENDS_%s-image' % kname, ' %s-image-
> > > > %s' %
> > > > (kname, typelower))
> > > >           d.setVar('PKG_%s-image-%s' % (kname,typelower), '%s-
> > > > image-
> > > > %s-${KERNEL_VERSION_PKG_NAME}' % (kname, typelower))
> > > >           d.setVar('ALLOW_EMPTY_%s-image-%s' % (kname,
> > > > typelower),
> > > > '1')
> > > > +        d.setVar('pkg_postinst_ontarget_%s-image-%s' %
> > > > (kname,typelower), """
> > > 
> > > You were previously concerned about boot issues if the
> > > unversioned
> > > image link/copy was deferred until boot. I don't see a summary of
> > > why
> > > that's no longer a concern in the v3 of the patch. Can you
> > > confirm
> > > that it isn't an issue ? Is it simply because the booted image
> > > isn't
> > > installed via this package, and hence we are fine ?
> > 
> > I'm suspecting this patch is why we're seeing selftest failures:
> > 
> > https://autobuilder.yoctoproject.org/typhoon/#/builders/86/builds/1055
> > 
> Do you mean this patch has been applied? and introduce new issues?

Yes, I reverted it and retested and confirmed this patch causes the
selftest failure in the link to the build above.

Cheers,

Richard

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#139888): 
https://lists.openembedded.org/g/openembedded-core/message/139888
Mute This Topic: https://lists.openembedded.org/mt/74977721/21656
Group Owner: [email protected]
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to