<div dir="ltr"><div class="gmail_default" style="font-family:times new roman,serif">Kieran,</div><div class="gmail_default" style="font-family:times new roman,serif">Looks good, thanks for shepherding that in.</div><div class="gmail_default" style="font-family:times new roman,serif">/Brian</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Dec 6, 2021 at 10:22 AM Kieran Bingham <<a href="mailto:kieran.bingham@ideasonboard.com">kieran.bingham@ideasonboard.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Brian,<br>
<br>
Quoting Brian Olson (2021-12-06 14:52:28)<br>
> Found it, in short:<br>
> chmod +x ./utils/tracepoints/gen-tp-header.py<br>
<br>
Oh, that not what I was expecting, but good find.<br>
<br>
> It looks like other python build steps already have +x and the right things<br>
> happen, but this one was missed and the build system cleverly runs it under<br>
> python (but the wrong one).<br>
> <br>
> Patch attached, or it's probably easier for someone with committer to just<br>
> run that chmod line.<br>
<br>
Can I add the following to the commit that does that?<br>
<br>
utils: tracepoints: gen-tp-header: Add execute permission<br>
<br>
The gen-tp-header script does not have execute permissions set on the<br>
file.<br>
<br>
Add the execute permission to ensure that meson runs this script<br>
consistently in the same configuration as other scripts, without<br>
invoking python itself separately.<br>
<br>
Signed-off-by: Brian Olson <<a href="mailto:icic@bolson.org" target="_blank">icic@bolson.org</a>><br>
<br>
<br>
to that?<br>
<br>
<br>
> On Sun, Dec 5, 2021 at 8:05 AM Brian Olson <<a href="mailto:icic@bolson.org" target="_blank">icic@bolson.org</a>> wrote:<br>
> <br>
> > $ ninja -C build install<br>
> > ...<br>
> > /usr/bin/python3<br>
> > /blah/blah/blah/libcamera/utils/tracepoints/./gen-tp-header.py ...<br>
> > Traceback (most recent call last):<br>
> >   File "/blah/blah/blah/libcamera/utils/tracepoints/./gen-tp-header.py",<br>
> > line 10, in <module><br>
> >     import jinja2<br>
> > ModuleNotFoundError: No module named 'jinja2'<br>
> ><br>
> > Yeah, I don't have it installed at the root level, but I have jinja2<br>
> > installed in the venv I was running ninja from. I don't know how to say<br>
> > "use this python3" other than putting it in the PATH, which it was, but<br>
> > ninja ignored that and went with /usr/bin/python3, and that failed.<br>
> ><br>
</blockquote></div>