<div>Using split() will break if the path contains a space if that is a concern.<br></div><div><br><div class="gmail_quote"><div dir="ltr">On Fri, Dec 7, 2018 at 8:04 AM <<a href="mailto:richard.purdie@linuxfoundation.org">richard.purdie@linuxfoundation.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Thu, 2018-12-06 at 21:27 -0600, Joshua Watt wrote:<br>
> On Thu, Dec 6, 2018 at 4:05 PM Richard Purdie<br>
> <<a href="mailto:richard.purdie@linuxfoundation.org" target="_blank">richard.purdie@linuxfoundation.org</a>> wrote:<br>
> > On Thu, 2018-12-06 at 12:55 -0600, Joshua Watt wrote:<br>
> > > It is occasionally useful to pass additional flags to fakeroot<br>
> > > (pseudo)<br>
> > > either for debugging or for behavioral system changes. Add a<br>
> > > variable<br>
> > > called "FAKEROOTARGS" that can be used for this purpose.<br>
> > <br>
> > Why wouldn't you just append to FAKEROOT/FAKEROOTCMD ?<br>
> <br>
> You can't currently add arguments to FAKEROOTCMD because it is passed<br>
> as an array member to subprocess.Popen sort of like:<br>
> <br>
> subprocess.Popen([d.getVar('FAKEROOTCMD'), 'blah', 'blah', ...<br>
> <br>
> In order to pass arguments, it would probably have to have split into<br>
> an array i.e. :<br>
> <br>
> subprocess.Popen(d.getVar('FAKEROOTCMD').split() + ['blah', 'blah',<br>
> ...<br>
> <br>
> I wasn't sure if that was desirable or if it would break anything, so<br>
> I went with the lower risk approach of adding a new variables. I can<br>
> certainly make a change to modify the usage of FAKEROOTCMD as<br>
> described above and simply append the flags to it if that is<br>
> preferred.<br>
<br>
I'd prefer to fix this to work than add more variables...<br>
<br>
Cheers,<br>
<br>
Richard<br>
<br>
-- <br>
_______________________________________________<br>
bitbake-devel mailing list<br>
<a href="mailto:bitbake-devel@lists.openembedded.org" target="_blank">bitbake-devel@lists.openembedded.org</a><br>
<a href="http://lists.openembedded.org/mailman/listinfo/bitbake-devel" rel="noreferrer" target="_blank">http://lists.openembedded.org/mailman/listinfo/bitbake-devel</a><br>
</blockquote></div></div>