Review Request 129900: Re-introduce the missing databases.
Shaheed Haque
srhaque at theiet.org
Sat Jan 28 19:37:00 UTC 2017
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129900/
-----------------------------------------------------------
(Updated Jan. 28, 2017, 7:36 p.m.)
Review request for Extra Cmake Modules.
Repository: extra-cmake-modules
Description
-------
This involved:
1. Restoring the ability for rules_engine.py to be run so as to generate
it own documentation, and support for the following rule databases:
- Unexposed
- Typecode
2. Restoring the corresponding support in sip_generator.py.
3. Improved diagnostics by making each Rule and AbstractCompiledCodeDb
have a useful names, and then using the results when tracing rule
execution (and usage dumping). This should make it MUCH easier to
debug rule-related issues.
4. Unit tests for the new functionality.
Diffs
-----
find-modules/.gitignore PRE-CREATION
find-modules/Qt5Ruleset.py 2d8cfaacd1a6e15afe533bc15b6177c49441a852
find-modules/rules_engine.py 955a8eb69115723a5c527d5067dd0d82bb7f968c
find-modules/sip_generator.py ba9dd636aeb1bf7345f87e5f05bcf530f53f2e2c
tests/GenerateSipBindings/cpplib.h a91b54864a5d3601a37f8f788fc4f3e9bb8ba323
tests/GenerateSipBindings/rules_SipTest.py c570a03988e63bf6a03641fa21035ee678dcad22
tests/GenerateSipBindings/testscript.py e79706bcb2d337484e8ce2098eaacd95c15cae9b
Diff: https://git.reviewboard.kde.org/r/129900/diff/
Testing (updated)
-------
1. Added a bunch of unit tests to ECM and then:
Test project .../extra-cmake-modules
Start 1: GenerateSipBindings
1/49 Test #1: GenerateSipBindings ................................... Passed 17.26 sec
Start 2: ExecuteCoreModules
2/49 Test #2: ExecuteCoreModules .................................... Passed 0.39 sec
Start 3: ExecuteKDEModules
3/49 Test #3: ExecuteKDEModules ..................................... Passed 0.41 sec
Start 4: KDEInstallDirsTest.relative_or_absolute
4/49 Test #4: KDEInstallDirsTest.relative_or_absolute ............... Passed 0.40 sec
Start 5: KDEInstallDirsTest.vars_in_sync_no_args
...
2. Verified against KGuiAddons (at cb82278f0b4da446a7859873f3e99ca64c470552).
Test project .../kguiaddons
Start 1: appstreamtest
1/6 Test #1: appstreamtest .................... Passed 0.02 sec
Start 2: Py3TestKGuiAddons
2/6 Test #2: Py3TestKGuiAddons ................ Passed 0.24 sec
Start 3: Py2TestKGuiAddons
3/6 Test #3: Py2TestKGuiAddons ................ Passed 0.23 sec
Start 4: kwordwraptest
4/6 Test #4: kwordwraptest .................... Passed 0.18 sec
Start 5: kcolorutilstest
5/6 Test #5: kcolorutilstest .................. Passed 0.53 sec
3. Reviewed against regression with bulk tools:
(From module-generator.py)
INFO: Summary: 0 processing errors for 2315 files in 170 modules
(From module-compiler.py)
ERROR: Summary: 140 processing errors for 165 modules
(Modules built)
$ find tmp2/python/ -name *.so|wc -l
25
Thanks,
Shaheed Haque
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-buildsystem/attachments/20170128/ab8a03e2/attachment-0001.html>
More information about the Kde-buildsystem
mailing list