Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: "Martin Hundebøll" <mnhu@prevas.dk>,
	"Paul Barker" <pbarker@toganlabs.com>
Cc: openembedded-core@lists.openembedded.org
Subject: Re: [PATCH v2] kernel-module-split: rrecommend kernel-image instead of rdepend
Date: Mon, 11 Sep 2017 10:26:53 +0100	[thread overview]
Message-ID: <1505122013.18640.1.camel@linuxfoundation.org> (raw)
In-Reply-To: <aa0e9f4b-bc55-8f25-ab72-efdeca0f7c17@prevas.dk>

On Mon, 2017-09-11 at 09:02 +0200, Martin Hundebøll wrote:
> 
> On 2017-09-08 23:29, Paul Barker wrote:
> > 
> > On Fri, Sep 8, 2017 at 10:07 PM, Martin Hundebøll
> > <martin.hundeboll@prevas.dk> wrote:
> > > 
> > > 
> > No, you'd need it to be set in a conf file so it takes effect when
> > the
> > kernel-base package is created.
> > 
> > I can see the issue here though. Can you set `RDEPENDS_kernel-base
> > =
> > ""` globally and explicitly install kernel-image into the
> > appropriate
> > image? If not then yes we'll need to find an alternative solution
> > to
> > simply reverting the change.
> > 
> Seems like the inverted approach of disabling install of kernel-
> image 
> globally, and enabling it per-image works just fine.
> 
> Reverting the patch is fine by me - sorry for the noise.

Are you sure? I can't imagine being able to influence this at all from
an image recipe, either the dependency is present or its not and it
would be set from the global configuration, not from an image recipe...

Cheers,

Richard


  reply	other threads:[~2017-09-11  9:27 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-08-21  8:39 [PATCH] kernel-module-split: rrecommend kernel-image instead of rdepend Martin Hundebøll
2017-08-21  9:04 ` ✗ patchtest: failure for " Patchwork
2017-08-21  9:21 ` [PATCH v2] " Martin Hundebøll
2017-09-08 20:06   ` Paul Barker
2017-09-08 20:20     ` Andre McCurdy
2017-09-08 20:37     ` Denys Dmytriyenko
     [not found]       ` <75F5325D-9BEC-49A6-AEF8-75448B46BCF7@prevas.dk>
2017-09-08 21:29         ` Paul Barker
2017-09-11  7:02           ` Martin Hundebøll
2017-09-11  9:26             ` Richard Purdie [this message]
2017-09-11 12:19               ` Martin Hundebøll
2017-09-11 12:50                 ` Richard Purdie

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=1505122013.18640.1.camel@linuxfoundation.org \
    --to=richard.purdie@linuxfoundation.org \
    --cc=mnhu@prevas.dk \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=pbarker@toganlabs.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox