public inbox for isar-users@googlegroups.com
 help / color / mirror / Atom feed
From: Henning Schild <henning.schild@siemens.com>
To: Alexander Smirnov <asmirnov@ilbers.de>
Cc: <isar-users@googlegroups.com>, Claudius Heine <ch@denx.de>
Subject: Re: [PATCH v2] meta/classes/base: Fix showdata and liststasks after bitbake update
Date: Fri, 9 Feb 2018 19:16:10 +0100	[thread overview]
Message-ID: <20180209191610.161916ad@mmd1pvb1c.ad001.siemens.net> (raw)
In-Reply-To: <20180209172850.28852-1-asmirnov@ilbers.de>

Mhh this one rings a bell ;)

https://groups.google.com/d/msg/isar-users/Raq3ayrqtnc/oqTt6_sSBQAJ

Someone has to reread that and check whether my symlink approach was
rejected rightfully, or whether it was just one of the many
misunderstandings we had in the past.

Henning

Am Fri, 9 Feb 2018 20:28:50 +0300
schrieb Alexander Smirnov <asmirnov@ilbers.de>:

> From: Claudius Heine <ch@denx.de>
> 
> 8<--
> 
> Just added references to origin code in comment. Didn't touch the
> code.
> 
> 8<--
> 
> Those tasks were broken, this patchs fixes by implementing them
> similar to how openembedded-core does it.
> 
> Signed-off-by: Claudius Heine <ch@denx.de>
> ---
>  meta/classes/base.bbclass | 30 +++++++++++++++++++-----------
>  1 file changed, 19 insertions(+), 11 deletions(-)
> 
> diff --git a/meta/classes/base.bbclass b/meta/classes/base.bbclass
> index 7d44f72..ae84677 100644
> --- a/meta/classes/base.bbclass
> +++ b/meta/classes/base.bbclass
> @@ -37,25 +37,33 @@ bbfatal() {
>  	exit 1
>  }
>  
> +# Derived from bitbake: bitbake/classes/base.bbclass
>  addtask showdata
>  do_showdata[nostamp] = "1"
>  python do_showdata() {
> -	import sys
> -	# emit variables and shell functions
> -	bb.data.emit_env(sys.__stdout__, d, True)
> -	# emit the metadata which isnt valid shell
> -	for e in bb.data.keys(d):
> -		if bb.data.getVarFlag(e, 'python', d):
> -			sys.__stdout__.write("\npython %s ()
> {\n%s}\n" % (e, bb.data.getVar(e, d, 1)))
> +    for e in d.keys():
> +        if d.getVarFlag(e, 'python'):
> +            bb.plain("\npython %s () {\n%s}\n" % (e, d.getVar(e,
> True))) }
>  
> +# Derived from Open Embedded:
> openembedded-core/meta/classes/utility-tasks.bbclass addtask listtasks
>  do_listtasks[nostamp] = "1"
>  python do_listtasks() {
> -	import sys
> -	for e in bb.data.keys(d):
> -		if bb.data.getVarFlag(e, 'task', d):
> -			sys.__stdout__.write("%s\n" % e)
> +    taskdescs = {}
> +    maxlen = 0
> +    for e in d.keys():
> +        if d.getVarFlag(e, 'task'):
> +            maxlen = max(maxlen, len(e))
> +            if e.endswith('_setscene'):
> +                desc = "%s (setscene version)" %
> (d.getVarFlag(e[:-9], 'doc') or '')
> +            else:
> +                desc = d.getVarFlag(e, 'doc') or ''
> +            taskdescs[e] = desc
> +
> +    tasks = sorted(taskdescs.keys())
> +    for taskname in tasks:
> +        bb.plain("%s  %s" % (taskname.ljust(maxlen),
> taskdescs[taskname])) }
>  
>  do_fetch[dirs] = "${DL_DIR}"


  reply	other threads:[~2018-02-09 18:16 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-02-09 17:28 Alexander Smirnov
2018-02-09 18:16 ` Henning Schild [this message]
2018-02-09 18:42   ` Alexander Smirnov
2018-02-09 19:33     ` Henning Schild
2018-02-09 20:18       ` Alexander Smirnov
2018-02-12  7:44 ` Jan Kiszka
2018-02-12  9:14   ` Alexander Smirnov

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=20180209191610.161916ad@mmd1pvb1c.ad001.siemens.net \
    --to=henning.schild@siemens.com \
    --cc=asmirnov@ilbers.de \
    --cc=ch@denx.de \
    --cc=isar-users@googlegroups.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