All of lore.kernel.org
 help / color / mirror / Atom feed
From: Martin Jansa <martin.jansa@gmail.com>
To: openembedded-devel@lists.openembedded.org
Subject: Re: [RFC][meta-oe][PATCH 3/3] blacklist.bbclass: Move to meta-angstrom
Date: Mon, 16 Jul 2012 19:58:19 +0200	[thread overview]
Message-ID: <20120716175819.GM3729@jama.jama.net> (raw)
In-Reply-To: <CAMKF1so4v-Sjm6sgHY4U_d7CwwwGp4DfQc0CSOq75n2kFwYvvA@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 2826 bytes --]

On Mon, Jul 16, 2012 at 10:50:58AM -0700, Khem Raj wrote:
> On Mon, Jul 16, 2012 at 10:46 AM, Martin Jansa <martin.jansa@gmail.com> wrote:
> > On Mon, Jul 16, 2012 at 10:38:36AM -0700, Khem Raj wrote:
> >> This class is angstrom specific so lets move it to more
> >> appropriate layer
> >
> > This is also used by SHR, that's why I've asked Koen to move blacklist
> > from meta-angstrom to meta-oe before
> > http://git.openembedded.org/meta-openembedded/commit/?id=68c15845d087df0a6f66c58a9fd80548a9bb3a59
> >
> 
> OK, OE-Core also has blacklist implementation. Have you given a
> thought to use it ?

Yes, but that would require distro config to be updated from
ANGSTROM_BLACKLIST_PN to PNBLACKLIST[pn] at the same time this class is
moved or duplicate all blacklist entries now and then remove
ANGSTROM_BLACKLIST_PN after it's moved..

Cheers,

> 
> > Cheers,
> >
> >>
> >> Signed-off-by: Khem Raj <raj.khem@gmail.com>
> >> ---
> >>  meta-oe/classes/blacklist.bbclass |   20 --------------------
> >>  1 file changed, 20 deletions(-)
> >>  delete mode 100644 meta-oe/classes/blacklist.bbclass
> >>
> >> diff --git a/meta-oe/classes/blacklist.bbclass b/meta-oe/classes/blacklist.bbclass
> >> deleted file mode 100644
> >> index 7bf4a73..0000000
> >> --- a/meta-oe/classes/blacklist.bbclass
> >> +++ /dev/null
> >> @@ -1,20 +0,0 @@
> >> -# anonymous support class from angstrom
> >> -#
> >> -# Features:
> >> -#
> >> -# * blacklist handling, set ANGSTROM_BLACKLIST_pn-blah = "message"
> >> -#
> >> -
> >> -python () {
> >> -    import bb
> >> -
> >> -    blacklist = bb.data.getVar("ANGSTROM_BLACKLIST", d, 1)
> >> -    pkgnm = bb.data.getVar("PN", d, 1)
> >> -    distro = bb.data.getVar("DISTRO", d, 1)
> >> -
> >> -    if blacklist:
> >> -     bb.note("%s DOES NOT support %s because %s" % (distro,pkgnm, blacklist))
> >> -        raise bb.parse.SkipPackage("%s DOES NOT support %s because %s" % (distro,pkgnm, blacklist))
> >> -
> >> -}
> >> -
> >> --
> >> 1.7.9.5
> >>
> >>
> >> _______________________________________________
> >> Openembedded-devel mailing list
> >> Openembedded-devel@lists.openembedded.org
> >> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
> >
> > --
> > Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com
> >
> > _______________________________________________
> > Openembedded-devel mailing list
> > Openembedded-devel@lists.openembedded.org
> > http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
> >
> 
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 205 bytes --]

  reply	other threads:[~2012-07-16 18:09 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-07-16 17:38 [RFC][meta-oe][PATCH 1/3] kernel.bbclass: Rename to machine_kernel_pr.bbclass which provides added functionality Khem Raj
2012-07-16 17:38 ` [RFC][meta-oe][PATCH 2/3] testlab.bbclass: Delete Khem Raj
2012-07-16 17:38 ` [RFC][meta-oe][PATCH 3/3] blacklist.bbclass: Move to meta-angstrom Khem Raj
2012-07-16 17:46   ` Martin Jansa
2012-07-16 17:50     ` Khem Raj
2012-07-16 17:58       ` Martin Jansa [this message]
2012-07-16 18:00         ` Khem Raj
2012-07-16 18:04           ` Martin Jansa
2012-07-18 11:25             ` Martin Jansa
2012-07-16 17:50 ` [RFC][meta-oe][PATCH 1/3] kernel.bbclass: Rename to machine_kernel_pr.bbclass which provides added functionality Martin Jansa
2012-07-16 17:55   ` Khem Raj
2012-07-17 12:39     ` Martin Jansa
2012-07-17 12:37   ` [meta-oe][PATCH 1/7] kernel: save $kerndir/tools and $kerndir/lib from pruning Martin Jansa
2012-07-17 12:37     ` [meta-oe][PATCH 2/7] recipes-kernel: make perf a standalone package Martin Jansa
2012-07-17 12:37     ` [meta-oe][PATCH 3/7] kernel: Add kernel headers to kernel-dev package Martin Jansa
2012-07-17 12:37     ` [meta-oe][PATCH 4/7] kernel.bbclass: add non-santized kernel provides Martin Jansa
2012-07-17 12:37     ` [meta-oe][PATCH 5/7] kernel.bbclass: Dont package kxgettext.o Martin Jansa
2012-07-17 12:37     ` [meta-oe][PATCH 6/7] kernel.bbclass: add deploy link to KERNEL_IMAGETYPE Martin Jansa
2012-07-17 12:38     ` [meta-oe][PATCH 7/7] kernel.bbclass: replace os.system with subprocess.call Martin Jansa

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=20120716175819.GM3729@jama.jama.net \
    --to=martin.jansa@gmail.com \
    --cc=openembedded-devel@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.