<div dir="ltr">On Tue, May 21, 2013 at 12:27 PM, Diego <span dir="ltr"><<a href="mailto:diego.ml@zoho.com" target="_blank">diego.ml@zoho.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
In data marted́ 21 maggio 2013 11:34:54, Otavio Salvador ha scritto:<br>
<div class="im">...<span style="color:rgb(34,34,34)"> </span></div></blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">> > +++ b/recipes-graphics/vivante-gpu-sdk/<a href="http://vivante-gpu-sdk_1.00.bb" target="_blank">vivante-gpu-sdk_1.00.bb</a><br>
</div><div><div class="h5">
> > @@ -0,0 +1,35 @@<br>
> > +SUMMARY = "Vivante GPU SDK Samples"<br>
> > +DESCRIPTION = "Freescale's adaptation of Jeff Molofee demos"<br>
><br>
> Is this description accurate?<br>
><br>
<br>
</div></div>These are the first lines of every source file:<br>
/*<br>
 * This code was created by Jeff Molofee '99<br>
 * (ported to Linux by Ti Leggett '01)<br>
 * (ported to i.mx51, i.mx31 and x11 by Freescale '10)<br>
 * If you've found this code useful, please let him know.<br>
 *<br>
 * Visit Jeff at <a href="http://nehe.gamedev.net/" target="_blank">http://nehe.gamedev.net/</a><br>
 *<br>
 */<br>
<br>
I know the description is not the best one can produce, but this is not an<br>
established project, nor a one man/enterprise work. At least the X11 part is<br>
not Vivante's nor Freescale's specific at all.</blockquote><div><br></div><div style>What about:</div><div style><br></div><div style>"Set of sample applications compatible with Vivante GPU"</div><div> </div>
<div>?</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">...<span style="color:rgb(34,34,34)"> </span></div></blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">> > +SRC_URI[md5sum] = "344c9a260b46a00f86eb5cb73bf729e0"<br></div><div class="im">
> > +SRC_URI[sha256sum] =<br>
> > "0f0a576ce1a08719ee1018832ae808ee723fad27a534c9739b5feec6d7435c57"<br>
> > +<br>
> > +S = "${WORKDIR}/${PN}-${PV}"<br>
> > +GLES11DIR = "${S}/Samples/GLES1.1"<br>
><br>
> You can avoid this setting:<br>
><br>
> WORKDIR = "${S}/Samples/GLES1.1"<br>
><br>
<br>
</div>I'll have multiple "workdirs", as this initial version is just for GLES1.1,<br>
but Samples folder has three subfolders:<br>
GLES1.1<br>
GLES2.0<br>
OpenVG<br>
each one has its own Makefile, and no common Makefile. I'm not interested in<br>
creating a common Makefile with a patch, as they might even get packaged<br>
separately in the future according to chipset support.<br>
According to that, would you still use just:<br>
<div class="im">WORKDIR = "${S}/Samples/GLES1.1"<br></div></blockquote><div><br></div><div style>I agree here; you can try to get the patch already supporting them. I'd use a 'for' loop to handle it, if possible.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
> > +VIVANTE_INSTALL_DIR = "/opt/${PN}"<br>
><br>
> > +VIVANTE_INSTALL_DEST_DIR = "${D}${VIVANTE_INSTALL_DIR}"<br>
><br>
><br>
> I'd use these explicitly as the variable name is bigger than the content of<br>
> it ;-)<br>
><br>
<br>
</div>Shorter code is not necessarily better or clearer code. However if you want<br>
I'll follow your suggestion.<br></blockquote><div> </div><div style>The problem with this is someone looking in the recipe needs to go there and check what is the value of it. I think it is an indirection which does not make it easier to read. We shouldn't keep changing this values so an indirection here does not add much I think.</div>
<div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">...<span style="color:rgb(34,34,34)"> </span></div></blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">> > +       install -d "${VIVANTE_INSTALL_DEST_DIR}"<br></div></blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">

> > +       oe_runmake -f Makefile.x11 install<br>
> > +       cp -r bin/ "${VIVANTE_INSTALL_DEST_DIR}"<br>
><br>
> Uh? make install does not handle this?<br>
><br>
<br>
</div>This is not in any way an "established project which installs in standard<br>
paths". It's more a "bunch of samples patched in the years, started quite some<br>
time ago, and intended just to be compiled and started locally by the learning<br>
developer".</blockquote><div><br></div><div style>Arrg! I see.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
> > +}<br>
> > +<br>
> > +FILES_${PN} += "${VIVANTE_INSTALL_DIR}"<br>
> > +FILES_${PN}-dbg = "${VIVANTE_INSTALL_DIR}/*/*/.debug"<br>
<br>
</div>Thanks for your review. If you can clear out the WORKDIR vs multiple Makefiles<br>
in subdirs issue I'll post the updated patch.<br></blockquote><div><br></div><div style>Thanks by working on this; this will be of great value for users :-)</div><div style><br></div><div style>Regards,</div></div><div>
<br></div>-- <br>Otavio Salvador                             O.S. Systems<br><a href="http://www.ossystems.com.br" target="_blank">http://www.ossystems.com.br</a>        <a href="http://projetos.ossystems.com.br" target="_blank">http://projetos.ossystems.com.br</a><br>
Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750<br>
</div></div>