* [PATCH] ne10: use bb.debug for debug messages
@ 2016-04-07 20:40 Denys Dmytriyenko
2016-04-07 22:47 ` Khem Raj
2016-04-20 18:25 ` Denys Dmytriyenko
0 siblings, 2 replies; 5+ messages in thread
From: Denys Dmytriyenko @ 2016-04-07 20:40 UTC (permalink / raw)
To: openembedded-devel; +Cc: Denys Dmytriyenko
From: Denys Dmytriyenko <denys@ti.com>
Moreover, the messages are misleading, saying "Building for arch blah",
even when this recipe is not being built. It merely checks if the recipe
is cmaptible with the currently used architecture.
Signed-off-by: Denys Dmytriyenko <denys@ti.com>
---
meta-oe/recipes-support/ne10/ne10_1.2.1.bb | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/meta-oe/recipes-support/ne10/ne10_1.2.1.bb b/meta-oe/recipes-support/ne10/ne10_1.2.1.bb
index 4d753a0..de97982 100644
--- a/meta-oe/recipes-support/ne10/ne10_1.2.1.bb
+++ b/meta-oe/recipes-support/ne10/ne10_1.2.1.bb
@@ -20,10 +20,10 @@ COMPATIBLE_MACHINE_armv7a = "(.*)"
python () {
if any(t.startswith('armv7') for t in d.getVar('TUNE_FEATURES', True).split()):
d.setVar('NE10_TARGET_ARCH', 'armv7')
- bb.note('Building Ne10 for armv7')
+ bb.debug(2, 'Building Ne10 for armv7')
elif any(t.startswith('aarch64') for t in d.getVar('TUNE_FEATURES', True).split()):
d.setVar('NE10_TARGET_ARCH', 'aarch64')
- bb.note('Building Ne10 for aarch64')
+ bb.debug(2, 'Building Ne10 for aarch64')
else:
raise bb.parse.SkipPackage("Incompatible with archs other than armv7 and aarch64")
}
--
2.2.0
^ permalink raw reply related [flat|nested] 5+ messages in thread* Re: [PATCH] ne10: use bb.debug for debug messages
2016-04-07 20:40 [PATCH] ne10: use bb.debug for debug messages Denys Dmytriyenko
@ 2016-04-07 22:47 ` Khem Raj
2016-04-20 18:25 ` Denys Dmytriyenko
1 sibling, 0 replies; 5+ messages in thread
From: Khem Raj @ 2016-04-07 22:47 UTC (permalink / raw)
To: openembeded-devel; +Cc: Denys Dmytriyenko
On Thu, Apr 7, 2016 at 1:40 PM, Denys Dmytriyenko <denis@denix.org> wrote:
> From: Denys Dmytriyenko <denys@ti.com>
>
> Moreover, the messages are misleading, saying "Building for arch blah",
> even when this recipe is not being built. It merely checks if the recipe
> is cmaptible with the currently used architecture.
>
LGTM, I have been seeing these notes too.
> Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> ---
> meta-oe/recipes-support/ne10/ne10_1.2.1.bb | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/meta-oe/recipes-support/ne10/ne10_1.2.1.bb b/meta-oe/recipes-support/ne10/ne10_1.2.1.bb
> index 4d753a0..de97982 100644
> --- a/meta-oe/recipes-support/ne10/ne10_1.2.1.bb
> +++ b/meta-oe/recipes-support/ne10/ne10_1.2.1.bb
> @@ -20,10 +20,10 @@ COMPATIBLE_MACHINE_armv7a = "(.*)"
> python () {
> if any(t.startswith('armv7') for t in d.getVar('TUNE_FEATURES', True).split()):
> d.setVar('NE10_TARGET_ARCH', 'armv7')
> - bb.note('Building Ne10 for armv7')
> + bb.debug(2, 'Building Ne10 for armv7')
> elif any(t.startswith('aarch64') for t in d.getVar('TUNE_FEATURES', True).split()):
> d.setVar('NE10_TARGET_ARCH', 'aarch64')
> - bb.note('Building Ne10 for aarch64')
> + bb.debug(2, 'Building Ne10 for aarch64')
> else:
> raise bb.parse.SkipPackage("Incompatible with archs other than armv7 and aarch64")
> }
> --
> 2.2.0
>
> --
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH] ne10: use bb.debug for debug messages
2016-04-07 20:40 [PATCH] ne10: use bb.debug for debug messages Denys Dmytriyenko
2016-04-07 22:47 ` Khem Raj
@ 2016-04-20 18:25 ` Denys Dmytriyenko
2016-04-20 20:44 ` Martin Jansa
1 sibling, 1 reply; 5+ messages in thread
From: Denys Dmytriyenko @ 2016-04-20 18:25 UTC (permalink / raw)
To: openembedded-devel; +Cc: Denys Dmytriyenko
Ping. Also in master-next for over a week - how often does it get merged to
master?
On Thu, Apr 07, 2016 at 04:40:00PM -0400, Denys Dmytriyenko wrote:
> From: Denys Dmytriyenko <denys@ti.com>
>
> Moreover, the messages are misleading, saying "Building for arch blah",
> even when this recipe is not being built. It merely checks if the recipe
> is cmaptible with the currently used architecture.
>
> Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> ---
> meta-oe/recipes-support/ne10/ne10_1.2.1.bb | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/meta-oe/recipes-support/ne10/ne10_1.2.1.bb b/meta-oe/recipes-support/ne10/ne10_1.2.1.bb
> index 4d753a0..de97982 100644
> --- a/meta-oe/recipes-support/ne10/ne10_1.2.1.bb
> +++ b/meta-oe/recipes-support/ne10/ne10_1.2.1.bb
> @@ -20,10 +20,10 @@ COMPATIBLE_MACHINE_armv7a = "(.*)"
> python () {
> if any(t.startswith('armv7') for t in d.getVar('TUNE_FEATURES', True).split()):
> d.setVar('NE10_TARGET_ARCH', 'armv7')
> - bb.note('Building Ne10 for armv7')
> + bb.debug(2, 'Building Ne10 for armv7')
> elif any(t.startswith('aarch64') for t in d.getVar('TUNE_FEATURES', True).split()):
> d.setVar('NE10_TARGET_ARCH', 'aarch64')
> - bb.note('Building Ne10 for aarch64')
> + bb.debug(2, 'Building Ne10 for aarch64')
> else:
> raise bb.parse.SkipPackage("Incompatible with archs other than armv7 and aarch64")
> }
> --
> 2.2.0
>
> --
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH] ne10: use bb.debug for debug messages
2016-04-20 18:25 ` Denys Dmytriyenko
@ 2016-04-20 20:44 ` Martin Jansa
2016-04-22 1:04 ` Denys Dmytriyenko
0 siblings, 1 reply; 5+ messages in thread
From: Martin Jansa @ 2016-04-20 20:44 UTC (permalink / raw)
To: openembedded-devel; +Cc: Denys Dmytriyenko
When everything is at least once tested in my jenkins builds and I've time
to go through the list of pending changes.
I was away from computers almost whole previous week, so there is longer
delay now.
On Wed, Apr 20, 2016 at 8:25 PM, Denys Dmytriyenko <denis@denix.org> wrote:
> Ping. Also in master-next for over a week - how often does it get merged to
> master?
>
>
> On Thu, Apr 07, 2016 at 04:40:00PM -0400, Denys Dmytriyenko wrote:
> > From: Denys Dmytriyenko <denys@ti.com>
> >
> > Moreover, the messages are misleading, saying "Building for arch blah",
> > even when this recipe is not being built. It merely checks if the recipe
> > is cmaptible with the currently used architecture.
> >
> > Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> > ---
> > meta-oe/recipes-support/ne10/ne10_1.2.1.bb | 4 ++--
> > 1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/meta-oe/recipes-support/ne10/ne10_1.2.1.bb
> b/meta-oe/recipes-support/ne10/ne10_1.2.1.bb
> > index 4d753a0..de97982 100644
> > --- a/meta-oe/recipes-support/ne10/ne10_1.2.1.bb
> > +++ b/meta-oe/recipes-support/ne10/ne10_1.2.1.bb
> > @@ -20,10 +20,10 @@ COMPATIBLE_MACHINE_armv7a = "(.*)"
> > python () {
> > if any(t.startswith('armv7') for t in d.getVar('TUNE_FEATURES',
> True).split()):
> > d.setVar('NE10_TARGET_ARCH', 'armv7')
> > - bb.note('Building Ne10 for armv7')
> > + bb.debug(2, 'Building Ne10 for armv7')
> > elif any(t.startswith('aarch64') for t in d.getVar('TUNE_FEATURES',
> True).split()):
> > d.setVar('NE10_TARGET_ARCH', 'aarch64')
> > - bb.note('Building Ne10 for aarch64')
> > + bb.debug(2, 'Building Ne10 for aarch64')
> > else:
> > raise bb.parse.SkipPackage("Incompatible with archs other than
> armv7 and aarch64")
> > }
> > --
> > 2.2.0
> >
> > --
> > _______________________________________________
> > Openembedded-devel mailing list
> > Openembedded-devel@lists.openembedded.org
> > http://lists.openembedded.org/mailman/listinfo/openembedded-devel
> --
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
>
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH] ne10: use bb.debug for debug messages
2016-04-20 20:44 ` Martin Jansa
@ 2016-04-22 1:04 ` Denys Dmytriyenko
0 siblings, 0 replies; 5+ messages in thread
From: Denys Dmytriyenko @ 2016-04-22 1:04 UTC (permalink / raw)
To: openembedded-devel
On Wed, Apr 20, 2016 at 10:44:04PM +0200, Martin Jansa wrote:
> When everything is at least once tested in my jenkins builds and I've time
> to go through the list of pending changes.
>
> I was away from computers almost whole previous week, so there is longer
> delay now.
Thanks, no problem and congrats!
> On Wed, Apr 20, 2016 at 8:25 PM, Denys Dmytriyenko <denis@denix.org> wrote:
>
> > Ping. Also in master-next for over a week - how often does it get merged to
> > master?
> >
> >
> > On Thu, Apr 07, 2016 at 04:40:00PM -0400, Denys Dmytriyenko wrote:
> > > From: Denys Dmytriyenko <denys@ti.com>
> > >
> > > Moreover, the messages are misleading, saying "Building for arch blah",
> > > even when this recipe is not being built. It merely checks if the recipe
> > > is cmaptible with the currently used architecture.
> > >
> > > Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> > > ---
> > > meta-oe/recipes-support/ne10/ne10_1.2.1.bb | 4 ++--
> > > 1 file changed, 2 insertions(+), 2 deletions(-)
> > >
> > > diff --git a/meta-oe/recipes-support/ne10/ne10_1.2.1.bb
> > b/meta-oe/recipes-support/ne10/ne10_1.2.1.bb
> > > index 4d753a0..de97982 100644
> > > --- a/meta-oe/recipes-support/ne10/ne10_1.2.1.bb
> > > +++ b/meta-oe/recipes-support/ne10/ne10_1.2.1.bb
> > > @@ -20,10 +20,10 @@ COMPATIBLE_MACHINE_armv7a = "(.*)"
> > > python () {
> > > if any(t.startswith('armv7') for t in d.getVar('TUNE_FEATURES',
> > True).split()):
> > > d.setVar('NE10_TARGET_ARCH', 'armv7')
> > > - bb.note('Building Ne10 for armv7')
> > > + bb.debug(2, 'Building Ne10 for armv7')
> > > elif any(t.startswith('aarch64') for t in d.getVar('TUNE_FEATURES',
> > True).split()):
> > > d.setVar('NE10_TARGET_ARCH', 'aarch64')
> > > - bb.note('Building Ne10 for aarch64')
> > > + bb.debug(2, 'Building Ne10 for aarch64')
> > > else:
> > > raise bb.parse.SkipPackage("Incompatible with archs other than
> > armv7 and aarch64")
> > > }
> > > --
> > > 2.2.0
> > >
> > > --
> > > _______________________________________________
> > > Openembedded-devel mailing list
> > > Openembedded-devel@lists.openembedded.org
> > > http://lists.openembedded.org/mailman/listinfo/openembedded-devel
> > --
> > _______________________________________________
> > Openembedded-devel mailing list
> > Openembedded-devel@lists.openembedded.org
> > http://lists.openembedded.org/mailman/listinfo/openembedded-devel
> >
> --
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2016-04-22 1:04 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-04-07 20:40 [PATCH] ne10: use bb.debug for debug messages Denys Dmytriyenko
2016-04-07 22:47 ` Khem Raj
2016-04-20 18:25 ` Denys Dmytriyenko
2016-04-20 20:44 ` Martin Jansa
2016-04-22 1:04 ` Denys Dmytriyenko
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.