<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Mar 30, 2017 at 5:26 PM, Alessandro Pasotti <span dir="ltr"><<a href="mailto:apasotti@gmail.com" target="_blank">apasotti@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"><span class="gmail-"><div class="gmail_extra"><div class="gmail_quote">On Thu, Mar 30, 2017 at 5:02 PM, Denis Rouzaud <span dir="ltr"><<a href="mailto:denis.rouzaud@gmail.com" target="_blank">denis.rouzaud@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"><br><br><div class="gmail_quote"><span><div dir="ltr">Le jeu. 30 mars 2017 à 16:56, Tim Sutton <<a href="mailto:tim@kartoza.com" target="_blank">tim@kartoza.com</a>> a écrit :<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 style="overflow-wrap: break-word;" class="gmail-m_-381313482829047574m_-4485998777950292603gmail_msg">Hi<div class="gmail-m_-381313482829047574m_-4485998777950292603gmail_msg"><br class="gmail-m_-381313482829047574m_-4485998777950292603gmail_msg"><div class="gmail-m_-381313482829047574m_-4485998777950292603gmail_msg"></div></div></div><div style="overflow-wrap: break-word;" class="gmail-m_-381313482829047574m_-4485998777950292603gmail_msg"><div class="gmail-m_-381313482829047574m_-4485998777950292603gmail_msg"><div class="gmail-m_-381313482829047574m_-4485998777950292603gmail_msg"><blockquote type="cite" class="gmail-m_-381313482829047574m_-4485998777950292603gmail_msg"><div class="gmail-m_-381313482829047574m_-4485998777950292603gmail_msg">On 30 Mar 2017, at 3:18 PM, Denis Rouzaud <<a href="mailto:denis.rouzaud@gmail.com" class="gmail-m_-381313482829047574m_-4485998777950292603gmail_msg" target="_blank">denis.rouzaud@gmail.com</a>> wrote:</div><br class="gmail-m_-381313482829047574m_-4485998777950292603m_-7903493363487032234Apple-interchange-newline gmail-m_-381313482829047574m_-4485998777950292603gmail_msg"><div class="gmail-m_-381313482829047574m_-4485998777950292603gmail_msg"><div dir="ltr" class="gmail-m_-381313482829047574m_-4485998777950292603gmail_msg">Dear all,<div class="gmail-m_-381313482829047574m_-4485998777950292603gmail_msg"><br class="gmail-m_-381313482829047574m_-4485998777950292603gmail_msg"></div><div class="gmail-m_-381313482829047574m_-4485998777950292603gmail_msg">I'd like to raise the attention on a new workflow I just merged to automatically generate SIP files using the header as source only.</div><div class="gmail-m_-381313482829047574m_-4485998777950292603gmail_msg"><br class="gmail-m_-381313482829047574m_-4485998777950292603gmail_msg"></div><div class="gmail-m_-381313482829047574m_-4485998777950292603gmail_msg">A PR is pending for the documentation, but you can read it here</div><div class="gmail-m_-381313482829047574m_-4485998777950292603gmail_msg"><a href="https://github.com/3nids/QGIS-Documentation/blob/6e3a9fb6f98a2869aef27c01b7ab70eb0a4c7556/source/docs/developers_guide/codingstandards.rst#sip-bindings" class="gmail-m_-381313482829047574m_-4485998777950292603gmail_msg" target="_blank">https://github.com/3nids/QGIS-<wbr>Documentation/blob/6e3a9fb6f98<wbr>a2869aef27c01b7ab70eb0a4c7556/<wbr>source/docs/developers_guide/c<wbr>odingstandards.rst#sip-binding<wbr>s</a><br class="gmail-m_-381313482829047574m_-4485998777950292603gmail_msg"></div><div class="gmail-m_-381313482829047574m_-4485998777950292603gmail_msg">(waiting to be merged).</div><div class="gmail-m_-381313482829047574m_-4485998777950292603gmail_msg"><br class="gmail-m_-381313482829047574m_-4485998777950292603gmail_msg"></div><div class="gmail-m_-381313482829047574m_-4485998777950292603gmail_msg">For the moment, only 3 files are generated (qgsattributes, qgsfeature and qgsvectorlayer). One can starts adding new files to the process by adapting their header and removing them from python/auto_sip.blacklist.</div></div></div></blockquote><div class="gmail-m_-381313482829047574m_-4485998777950292603gmail_msg"><br class="gmail-m_-381313482829047574m_-4485998777950292603gmail_msg"></div></div></div></div><div style="overflow-wrap: break-word;" class="gmail-m_-381313482829047574m_-4485998777950292603gmail_msg"><div class="gmail-m_-381313482829047574m_-4485998777950292603gmail_msg"><div class="gmail-m_-381313482829047574m_-4485998777950292603gmail_msg"><div class="gmail-m_-381313482829047574m_-4485998777950292603gmail_msg">This is really great! A couple of questions:</div><div class="gmail-m_-381313482829047574m_-4485998777950292603gmail_msg"><br class="gmail-m_-381313482829047574m_-4485998777950292603gmail_msg"></div><div class="gmail-m_-381313482829047574m_-4485998777950292603gmail_msg">1) the blacklist implies that any new .h files added to the source tree will automatically have sip bindings generated unless you explicitly blacklist them?</div></div></div></div></blockquote><div><br></div></span><div>As soon as you add the include in python/core/core.sip (or gui, analysis), the SIP is considered as automatically generated from header except if you blacklist it.</div><span><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 style="overflow-wrap: break-word;" class="gmail-m_-381313482829047574m_-4485998777950292603gmail_msg"><div class="gmail-m_-381313482829047574m_-4485998777950292603gmail_msg"><div class="gmail-m_-381313482829047574m_-4485998777950292603gmail_msg"><div class="gmail-m_-381313482829047574m_-4485998777950292603gmail_msg">2) are you thinking to maintain the sip files in Git still or remove the from revision in favour of having them as build time artifacts only? I ask this because I often direct people to use the sip files as a reference guide for the python API if they want to dig under the hood a bit more.</div></div></div></div></blockquote><div><br></div></span><div>The goal is to remove the SIP files and have them created on build time.</div><br></div></div></blockquote></div><br></div></span><div class="gmail_extra">I surely missed something: how can you deal with annotations?<span class="gmail-HOEnZb"><font color="#888888"><br><br></font></span></div><span class="gmail-HOEnZb"><font color="#888888"><div class="gmail_extra"><br></div></font></span></div></blockquote><div><br>Oh, now I see the docs:<br> <a href="https://patch-diff.githubusercontent.com/raw/qgis/QGIS-Documentation/pull/1788.diff">https://patch-diff.githubusercontent.com/raw/qgis/QGIS-Documentation/pull/1788.diff</a><br><br clear="all"></div></div>Smart solution indeed! <br><br>Thanks!<br><br></div><div class="gmail_extra">-- <br><div class="gmail_signature">Alessandro Pasotti<br>w3:   <a href="http://www.itopen.it" target="_blank">www.itopen.it</a></div>
</div></div>