All of lore.kernel.org
 help / color / mirror / Atom feed
From: Michael Wood <michael.g.wood@intel.com>
To: Dave Lerner <dave.lerner@windriver.com>,
	toaster@yoctoproject.org, belen.barros.pena@linux.intel.com
Subject: Re: [PATCH 1/1 v2] toaster: fixup dependency excludes for customimage
Date: Fri, 4 Mar 2016 16:48:47 +0000	[thread overview]
Message-ID: <56D9BC6F.9090606@intel.com> (raw)
In-Reply-To: <1457041393-4803-1-git-send-email-dave.lerner@windriver.com>

Thanks Dave

Sent to bitbake-devel and committed to toaster-next

Michael

On 03/03/16 21:43, Dave Lerner wrote:
> [YOCTO #9156]
>
> For a customized image when adding a dependent package X that depends on
> dependency package Y, in addition to adding X to appends_set and Y to
> includes_set, make sure that Y is no longer in the excludes_set. Y may
> have been added to the excludes_set by a prior package removal.
>
> Signed-off-by: Dave Lerner <dave.lerner@windriver.com>
> ---
>   bitbake/lib/toaster/toastergui/views.py | 7 +++++++
>   1 file changed, 7 insertions(+)
>
> diff --git a/bitbake/lib/toaster/toastergui/views.py b/bitbake/lib/toaster/toastergui/views.py
> index 28b03d3..f3b8d3c 100755
> --- a/bitbake/lib/toaster/toastergui/views.py
> +++ b/bitbake/lib/toaster/toastergui/views.py
> @@ -2653,6 +2653,13 @@ if True:
>                                              name=dep.depends_on.name)
>   
>                           recipe.includes_set.add(cust_package)
> +                        try:
> +                            # when adding the pre-requisite package make sure it's not in the
> +                            #   excluded list from a prior removal.
> +                            recipe.excludes_set.remove(cust_package)
> +                        except Package.DoesNotExist:
> +                            #   Don't care if the package had never been excluded
> +                            pass
>                       except:
>                           logger.warning("Could not add package's suggested"
>                                          "dependencies to the list")

---------------------------------------------------------------------
Intel Corporation (UK) Limited
Registered No. 1134945 (England)
Registered Office: Pipers Way, Swindon SN3 1RJ
VAT No: 860 2173 47

This e-mail and any attachments may contain confidential material for
the sole use of the intended recipient(s). Any review or distribution
by others is strictly prohibited. If you are not the intended
recipient, please contact the sender and delete all copies.



      reply	other threads:[~2016-03-04 16:48 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-03-03 21:43 [PATCH 1/1 v2] toaster: fixup dependency excludes for customimage Dave Lerner
2016-03-04 16:48 ` Michael Wood [this message]

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=56D9BC6F.9090606@intel.com \
    --to=michael.g.wood@intel.com \
    --cc=belen.barros.pena@linux.intel.com \
    --cc=dave.lerner@windriver.com \
    --cc=toaster@yoctoproject.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.