Le jeu. 30 mars 2017 à 16:56, Tim Sutton <[email protected]> a écrit :
> Hi > > On 30 Mar 2017, at 3:18 PM, Denis Rouzaud <[email protected]> wrote: > > Dear all, > > 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. > > A PR is pending for the documentation, but you can read it here > > https://github.com/3nids/QGIS-Documentation/blob/6e3a9fb6f98a2869aef27c01b7ab70eb0a4c7556/source/docs/developers_guide/codingstandards.rst#sip-bindings > (waiting to be merged). > > 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. > > > This is really great! A couple of questions: > > 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? > 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. > 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. > The goal is to remove the SIP files and have them created on build time. But: 1) I except a few errors in the script to arise (so better to have an easy access to the SIP files) 2) it needs to reach a significant point of ported headers. So, in a somehow close or far future, yes they'll be removed from source (hence no blacklist anymore only manual files such as core/conversions.sip).
_______________________________________________ Qgis-developer mailing list [email protected] List info: https://lists.osgeo.org/mailman/listinfo/qgis-developer Unsubscribe: https://lists.osgeo.org/mailman/listinfo/qgis-developer
