All of lore.kernel.org
 help / color / mirror / Atom feed
* INHERIT += "image-buildinfo" doesn't parse
@ 2016-06-04 22:55 Trevor Woerner
  2016-06-05  7:20 ` Richard Purdie
  0 siblings, 1 reply; 6+ messages in thread
From: Trevor Woerner @ 2016-06-04 22:55 UTC (permalink / raw)
  To: OE-core

Any config with
	INHERIT += "image-buildinfo"
doesn't parse on master (updated moments ago):

	ERROR: Error in compiling python function in /z/layerindex-master/layers/meta-poky/meta/classes/image-buildinfo.bbclass, line 27:

	The code lines resulting in this error were:
	SyntaxError: invalid syntax (image-buildinfo.bbclass, line 35)
	ERROR: Error parsing configuration files
	Traceback (most recent call last):
	  File "/z/layerindex-master/layers/meta-poky/bitbake/lib/bb/parse/parse_py/BBHandler.py", line 145, in handle(fn='/z/layerindex-master/layers/meta-poky/meta/classes/image-buildinfo.bbclass', d=<bb.data_smart.DataSmart object at 0x7fb5df5367f0>, include=True):
		 try:
	    >        statements.eval(d)
		 except bb.parse.SkipRecipe:
	  File "/z/layerindex-master/layers/meta-poky/bitbake/lib/bb/parse/ast.py", line 38, in StatementGroup.eval(data=<bb.data_smart.DataSmart object at 0x7fb5df5367f0>):
		     for statement in self:
	    >            statement.eval(data)
	     
	  File "/z/layerindex-master/layers/meta-poky/bitbake/lib/bb/parse/ast.py", line 187, in PythonMethodNode.eval(data=<bb.data_smart.DataSmart object at 0x7fb5df5367f0>):
		     text = '\n'.join(self.body)
	    >        bb.methodpool.insert_method(self.modulename, text, self.filename, self.lineno - len(self.body) - 1)
		     data.setVarFlag(self.function, "func", 1)
	  File "/z/layerindex-master/layers/meta-poky/bitbake/lib/bb/methodpool.py", line 27, in insert_method(modulename='image-buildinfo.bbclass', code='def get_layer_git_status(path):\n    import subprocess\n    try:\n        subprocess.check_output("cd %s; PSEUDO_UNLOAD=1 git diff --quiet --no-ext-diff" % path,\n                                shell=True,\n                                stderr=subprocess.STDOUT)\n        return ""\n    except subprocess.CalledProcessError, ex:\n        # Silently treat errors as "modified", without checking for the\n        # (expected) return code 1 in a modified git repo. For example, we get\n        # output and a 129 return code when a layer isn\'t a git repo at all.\n        return "-- modified"\n', fn='/z/layerindex-master/layers/meta-poky/meta/classes/image-buildinfo.bbclass', lineno=27):
		 """
	    >    comp = better_compile(code, modulename, fn, lineno=lineno)
		 better_exec(comp, None, code, fn)
	  File "/z/layerindex-master/layers/meta-poky/bitbake/lib/bb/utils.py", line 330, in better_compile(text='def get_layer_git_status(path):\n    import subprocess\n    try:\n        subprocess.check_output("cd %s; PSEUDO_UNLOAD=1 git diff --quiet --no-ext-diff" % path,\n                                shell=True,\n                                stderr=subprocess.STDOUT)\n        return ""\n    except subprocess.CalledProcessError, ex:\n        # Silently treat errors as "modified", without checking for the\n        # (expected) return code 1 in a modified git repo. For example, we get\n        # output and a 129 return code when a layer isn\'t a git repo at all.\n        return "-- modified"\n', file='image-buildinfo.bbclass', realfile='/z/layerindex-master/layers/meta-poky/meta/classes/image-buildinfo.bbclass', mode='exec', lineno=27):
		     e = bb.BBHandledException(e)
	    >        raise e
	     
	bb.BBHandledException: invalid syntax (image-buildinfo.bbclass, line 35)


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: INHERIT += "image-buildinfo" doesn't parse
  2016-06-04 22:55 INHERIT += "image-buildinfo" doesn't parse Trevor Woerner
@ 2016-06-05  7:20 ` Richard Purdie
  2016-06-05 13:02   ` Trevor Woerner
  0 siblings, 1 reply; 6+ messages in thread
From: Richard Purdie @ 2016-06-05  7:20 UTC (permalink / raw)
  To: Trevor Woerner, OE-core

On Sat, 2016-06-04 at 18:55 -0400, Trevor Woerner wrote:
> Any config with
> 	INHERIT += "image-buildinfo"
> doesn't parse on master (updated moments ago):
> 
> 	ERROR: Error in compiling python function in /z/layerindex
> -master/layers/meta-poky/meta/classes/image-buildinfo.bbclass, line
> 27:
> 
> 	The code lines resulting in this error were:
> 	SyntaxError: invalid syntax (image-buildinfo.bbclass, line 35)
> 
> 	bb.BBHandledException: invalid syntax (image-buildinfo.bbclass,
> line 35)

I've tweaked line 35 which used an obsolete python syntax so was an
obvious/trivial fix. I've not tested it though...

Cheers,

Richard


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: INHERIT += "image-buildinfo" doesn't parse
  2016-06-05  7:20 ` Richard Purdie
@ 2016-06-05 13:02   ` Trevor Woerner
  2016-06-05 13:04     ` Trevor Woerner
  0 siblings, 1 reply; 6+ messages in thread
From: Trevor Woerner @ 2016-06-05 13:02 UTC (permalink / raw)
  To: Richard Purdie; +Cc: OE-core

On Sun 2016-06-05 @ 08:20:11 AM, Richard Purdie wrote:
> On Sat, 2016-06-04 at 18:55 -0400, Trevor Woerner wrote:
> > Any config with
> > 	INHERIT += "image-buildinfo"
> > doesn't parse on master (updated moments ago):
> > 
> > 	ERROR: Error in compiling python function in /z/layerindex
> > -master/layers/meta-poky/meta/classes/image-buildinfo.bbclass, line
> > 27:
> > 
> > 	The code lines resulting in this error were:
> > 	SyntaxError: invalid syntax (image-buildinfo.bbclass, line 35)
> > 
> > 	bb.BBHandledException: invalid syntax (image-buildinfo.bbclass,
> > line 35)
> 
> I've tweaked line 35 which used an obsolete python syntax so was an
> obvious/trivial fix. I've not tested it though...

I've pulled your change (6e7d9dd9b5cbf46b955036876a25353fafbf264c) but it's
still not happy:

ERROR: Error parsing configuration files
Traceback (most recent call last):
  File "/z/layerindex-master/layers/meta-poky/bitbake/lib/bb/parse/parse_py/BBHandler.py", line 145, in handle(fn='/z/layerindex-master/layers/meta-poky/meta/classes/image-buildinfo.bbclass', d=<bb.data_smart.DataSmart object at 0x7f90650ee828>, include=True):
         try:
    >        statements.eval(d)
         except bb.parse.SkipRecipe:
  File "/z/layerindex-master/layers/meta-poky/bitbake/lib/bb/parse/ast.py", line 38, in StatementGroup.eval(data=<bb.data_smart.DataSmart object at 0x7f90650ee828>):
             for statement in self:
    >            statement.eval(data)

  File "/z/layerindex-master/layers/meta-poky/bitbake/lib/bb/parse/ast.py", line 187, in PythonMethodNode.eval(data=<bb.data_smart.DataSmart object at 0x7f90650ee828>):
             text = '\n'.join(self.body)
    >        bb.methodpool.insert_method(self.modulename, text, self.filename, self.lineno - len(self.body) - 1)
             data.setVarFlag(self.function, "func", 1)
  File "/z/layerindex-master/layers/meta-poky/bitbake/lib/bb/methodpool.py", line 27, in insert_method(modulename='image-buildinfo.bbclass', code='def get_layer_git_status(path):\n    import subprocess\n    try:\n        subprocess.check_output("cd %s; PSEUDO_UNLOAD=1 git diff --quiet --no-ext-diff" % path,\n                                shell=True,\n                                stderr=subprocess.STDOUT)\n        return ""\n    except subprocess.CalledProcessError, ex:\n        # Silently treat errors as "modified", without checking for the\n        # (expected) return code 1 in a modified git repo. For example, we get\n        # output and a 129 return code when a layer isn\'t a git repo at all.\n        return "-- modified"\n', fn='/z/layerindex-master/layers/meta-poky/meta/classes/image-buildinfo.bbclass', lineno=27):
         """
    >    comp = better_compile(code, modulename, fn, lineno=lineno)
         better_exec(comp, None, code, fn)
  File "/z/layerindex-master/layers/meta-poky/bitbake/lib/bb/utils.py", line 330, in better_compile(text='def get_layer_git_status(path):\n    import subprocess\n    try:\n        subprocess.check_output("cd %s; PSEUDO_UNLOAD=1 git diff --quiet --no-ext-diff" % path,\n                                shell=True,\n                                stderr=subprocess.STDOUT)\n        return ""\n    except subprocess.CalledProcessError, ex:\n        # Silently treat errors as "modified", without checking for the\n        # (expected) return code 1 in a modified git repo. For example, we get\n        # output and a 129 return code when a layer isn\'t a git repo at all.\n        return "-- modified"\n', file='image-buildinfo.bbclass', realfile='/z/layerindex-master/layers/meta-poky/meta/classes/image-buildinfo.bbclass', mode='exec', lineno=27):
             e = bb.BBHandledException(e)
    >        raise e

bb.BBHandledException: invalid syntax (image-buildinfo.bbclass, line 35)


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: INHERIT += "image-buildinfo" doesn't parse
  2016-06-05 13:02   ` Trevor Woerner
@ 2016-06-05 13:04     ` Trevor Woerner
  2016-06-05 13:06       ` Trevor Woerner
  0 siblings, 1 reply; 6+ messages in thread
From: Trevor Woerner @ 2016-06-05 13:04 UTC (permalink / raw)
  To: Richard Purdie; +Cc: OE-core

oh... wait, sorry my mistake I didn't test properly, let me try again...


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: INHERIT += "image-buildinfo" doesn't parse
  2016-06-05 13:04     ` Trevor Woerner
@ 2016-06-05 13:06       ` Trevor Woerner
  2016-06-05 13:15         ` Trevor Woerner
  0 siblings, 1 reply; 6+ messages in thread
From: Trevor Woerner @ 2016-06-05 13:06 UTC (permalink / raw)
  To: Richard Purdie; +Cc: OE-core

On Sun 2016-06-05 @ 09:04:12 AM, Trevor Woerner wrote:
> oh... wait, sorry my mistake I didn't test properly, let me try again...

yes, that fixes it :-D

(i was using git://git.yoctoproject.org/poky's meta instead of
git://git.openembedded.org/openembedded-core.git's)


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: INHERIT += "image-buildinfo" doesn't parse
  2016-06-05 13:06       ` Trevor Woerner
@ 2016-06-05 13:15         ` Trevor Woerner
  0 siblings, 0 replies; 6+ messages in thread
From: Trevor Woerner @ 2016-06-05 13:15 UTC (permalink / raw)
  To: Richard Purdie; +Cc: OE-core

On Sun 2016-06-05 @ 09:06:50 AM, Trevor Woerner wrote:
> On Sun 2016-06-05 @ 09:04:12 AM, Trevor Woerner wrote:
> > oh... wait, sorry my mistake I didn't test properly, let me try again...
> 
> yes, that fixes it :-D
> 
> (i was using git://git.yoctoproject.org/poky's meta instead of
> git://git.openembedded.org/openembedded-core.git's)

... and now that I check, I see that you've already pushed this to poky as
well. Sorry for the confusion, both work (my build was using one, I updated
the other, it didn't seem to work).


^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2016-06-05 13:15 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-06-04 22:55 INHERIT += "image-buildinfo" doesn't parse Trevor Woerner
2016-06-05  7:20 ` Richard Purdie
2016-06-05 13:02   ` Trevor Woerner
2016-06-05 13:04     ` Trevor Woerner
2016-06-05 13:06       ` Trevor Woerner
2016-06-05 13:15         ` Trevor Woerner

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.