<div dir="ltr"><div><div>Hi Larry,<br></div></div><div class="gmail_extra"><br><div class="gmail_quote">2013/8/31 Larry Shaffer <span dir="ltr"><<a href="mailto:larrys@dakotacarto.com" target="_blank">larrys@dakotacarto.com</a>></span><br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><div><div><div><div>Hi Salvatore,<br><br></div>Sorry to keep messing with this, but it was causing a build error. I reverted adding of qsci-pap-master target, leaving it similar to as before, excepting the convenience target of qsci-pap-src.<br>
<br></div>WITH_QSCIAPI is now marked as advanced, to keep casual users from thinking they need to regenerate the .pap file as part of building.<br><br>Here is why I think this is the simplest, most versatile solution:<br>
<br></div>*Casual user/dev who compiles/installs QGIS*<br></div>Everything works as before with the default pre-prepared pyqgis.pap in source tree used. No generation of .pap occurs. PyQGIS.api is still built by default and the target for qsci-pap-src is just created.<br>
<br></div>*QGIS dev working on bindings*<br><div><div><div><div>Making qsci-pap-src will update the default .pap in source tree, which will be used when running from build directory or install during binding testing. If the dev is ready to commit, the already-updated .pap includes the latest completions, and everyone benefits. We can use git to revert any fouled-up .pap that may make its way into the repo. This assumes the committing dev actually ran qsci-pap-src. If not, then any other dev can easily do so, updating their source tree, then commit the new default .pap.<br>
<br></div></div></div></div></div></blockquote><div><br><span lang="en"><span>this</span> <span>I was not clear</span> <span>in my previous</span> <span>email about</span> <span>the update</span> <span>in the source tree</span><span>, but now it does</span> <span>make much sense.</span></span><br>
<br>No idea for a better solution at the moment, I agree with your
solution. <br>For now we can go ahead keeping your last commit, just after
on Sept 1st. we can think for a better solution (whether it there is)
IMO.<br><br><br><div>Best Regards,<br></div><br>-SL<br> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><div><div><div></div>
<div>I am really done messing with this now. :^) If you have better solution, please let me know.<br></div><div><br></div><div>Regards,<br><br></div><div>Larry<br><br></div><div><div><div><div><br><div><div class="gmail_extra">
<div class="gmail_quote">On Sat, Aug 31, 2013 at 12:37 PM, Larry Shaffer <span dir="ltr"><<a href="mailto:larrys@dakotacarto.com" target="_blank">larrys@dakotacarto.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<div dir="ltr">Hi Salvatore,<br><div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Aug 31, 2013 at 7:58 AM, Salvatore Larosa <span dir="ltr"><<a href="mailto:lrssvtml@gmail.com" target="_blank">lrssvtml@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><div>ok Larry, your last commit works fine only I was wonder might be a problem to edit the source tree file pyqgis.pap by running the qsci-pap-src target ? just curiosity!<br>
</div></div></div></blockquote><div><br></div><div>I'm not sure I understand your question. Doing periodic updates to the pyqgis.pap in source keeps from forcing everyone who compiles the source to also generate that file. There is no simple means for checking when the API has changed (except maybe SHA comparisons of the API file). Without such a check, the generation of that file may often be unnecessary, but still done.<br>
<br>Everyone benefits if that file is kept up to date in the source tree. For example, the one I replaced with the initial commit was 11 months old, missing many completions for new functions.<br><br></div><div>Alternatively, we can set it up similar to what you were suggesting. Let me know what you think would be a better solution.<br>
</div><div><br></div><div>Regards,<br><br></div><div>Larry<br></div><div><br> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">
<div><div></div>Tonight I'll push the updated API files (GDAL/OGR-1.10.0,QScintilla-2.7.2 and GEOS-3.4.2 ) and will add the GEOS-3.4.2 APIs to the default .pap file.<br>
<br></div>Thanks for the clarification !<br><br></div></blockquote></div></div></div></div>
</blockquote></div><br></div></div></div></div></div></div></div></div></div></div>
</blockquote></div><br><br clear="all"><br>-- <br>Salvatore Larosa<br>linkedIn: <a href="http://linkedin.com/in/larosasalvatore" target="_blank">http://linkedin.com/in/larosasalvatore</a><br>twitter: @lrssvt<br>skype: s.larosa<br>
IRC: lrssvt on freenode<br>
</div></div>