<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><br></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><br></div><div class="gmail_extra"><br><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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi Salvatore,<br><div><div class="gmail_extra"><br><div class="gmail_quote"><div class="im">On Sat, Aug 31, 2013 at 4:50 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">Hi Larry,<br><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">Hi Salvatore,<br><div><div class="gmail_extra"><br><div class="gmail_quote"><div>
On Fri, Aug 30, 2013 at 2:52 PM, 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">Hi Larry,<br><div class="gmail_extra"><br><div class="gmail_quote">2013/8/30 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 dir="ltr">Hi,<br></div></div></div></div></blockquote></div></div></div>
</blockquote></div></div></div></div></div></blockquote></div></div></div></blockquote></div><div>snip --------8<------ <br></div><div class="im"><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 class="gmail_extra"><div class="gmail_quote"><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 class="gmail_extra">
<div class="gmail_quote"><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 class="gmail_extra"><div class="gmail_quote"><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 dir="ltr"><br>Should GDAL-OGR-1.9.1 be updated to 1.10 for QGIS 2.0 release?<br>
</div></div></div></div></blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"></blockquote><div><br></div>I would like update to the 1.10.0 version and add the GEOS-3.4.2, I have prepared those api files so if there are not objections here I could push they to master in order to add the new APIs to the pyqgis.pap file.<br>
</div></div></div></blockquote><div><br></div></div><div>Sounds good to me. I'm done mucking about with the setup, so please make the changes you see fit. Looks like the default pyqgis.pap I committed is missing the new 'qgis.core.NULL' attribute. Must of missed that somehow. Shows up fine when making qsci-pap-master, though. Please test for its completion tip in console before uploading another +1 MB .pap file (thanks).<br>
</div></div></div></div></div></blockquote><br>the pyqgis.pap file seems not to have the PyQGIS.api file, I done some adjustment to the cmake file [0] in order to point the generates_api script to the right directory. Please could you verify that it is working for you as well?<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 class="gmail_extra"><div class="gmail_quote"><div></div></div></div></div></div></blockquote></div></div></div></blockquote><div><br></div></div><div>The point of the qsci-pap-src target is to update the default pyqgis.pap, in the source tree. Once that is updated, then the regular CMakelists.txt [0] takes care of installing it. This update could be done every once in a while, but should always be done right before a release (i.e. like now). The target is not part of ALL, otherwise building would take that much longer. So, keeping a fairly recent version of pyqgis.pap updated in the source tree is a good idea.<br>
<br></div><div>The qsci-pap-master target helps developers when they are testing out new bindings and want the console to show them right away. Ideally, once a new binding is committed, the qsci-pap-src would be run to update the default pyqgis.pap. However, I do not expect this to happen very often; and, it seems rather excessive to upload a new pyqgis.pap (> 1MB) so often. It should be done a regular basis, though.<br>
<br></div><div>It may be possible to just remove the qsci-pap-master target. But, my feeling is that if many developers are updating the default pyqgis.pap, it will lead to problems with it.<br></div><div><br></div><div>
In other words, your edits should not be committed. But, you did find a bug. :^) Indeed, the PyQGIS.api file was not being picked up when making qsci-pap-src. I have adjusted the custom commands and the script. GitHub is offline right now for maintenance, but I'll commit it ASAP, along with a fixed default pyqgis.pap.</div>
<div class="im">
<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 class="gmail_extra"><div class="gmail_quote"><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 class="gmail_extra"><div class="gmail_quote"><div>Missing is the QScintilla2 API file, but I think that will just bloat the default completions with items most users will not need. IMO, we should keep the default .pap as slim as possible, for the same reasons, i.e. just core PyQGIS, OSGeo and Python/PyQt modules. That is, until there is on-the-fly object inspection support, which would auto-filter the results.<br>
</div></div></div></div></div></blockquote><div><br></div><div>ok, we can leave out the QScintilla APIs, but what about to add the updated file (QScintilla-2.7.2) into qsci_apis dir ? or should we drop all the old versions (GDAL-OGR-1.10 will replace GDAL-OGR-1.9.1 and so on) ?<br>
</div></div></div></div></blockquote><div><br></div></div><div>I think the .api files in QGIS source/install should reflect the 'base and most likely' versions available on supported platforms. It is common to have a much older QScintilla installed, which is why the older 2.4.5 api is in there. Since QGIS is working with much newer OSGeo libs (e.g. gdal-1.10) in core, I think it would be better to have more up-to-date .api version for those.<br>
<br>IMO, the idea here is to ensure new users and plugin developers don't get used to using a newer API for something that might not be available for all users. However, your custom API loader in console should take care of those users who do want to use the latest.<br>
</div><div><br><br></div><div>Regards,<br><br></div><div>Larry<br></div><div class="im"><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 class="gmail_extra"><div class="gmail_quote"><div>Thanks,<br><br></div><div>-SL<br><br>[0] - <a href="https://gist.github.com/slarosa/6397516" target="_blank">https://gist.github.com/slarosa/6397516</a><br> <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 class="gmail_extra"><div class="gmail_quote"><div>
<br></div><div>Regards,<br></div><div><br></div><div>Larry<br></div><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 class="gmail_extra">
<div class="gmail_quote"></div><div class="gmail_quote">I am going to test your cool commit....:-)<br></div><div class="gmail_quote"><div><br></div><div>Thanks a lot!<br><br></div><div>Best Regards,<br><br></div><div>-SL<br>
</div><div> </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><br><div><div><br>[0] <a href="https://github.com/qgis/Quantum-GIS/commit/dc462859" target="_blank">https://github.com/qgis/Quantum-GIS/commit/dc462859</a><br clear="all"><div><div><div><div><br></div><div>Regards,<br>
</div><div><br>
Larry</div>
</div></div></div></div></div></div></div>
<br></blockquote></div></div></div></blockquote></div></div></div></div></div></blockquote></div></div></div></blockquote></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>