Review Request 127984: Always update the Trader parser from y/l sources
Pino Toscano
pino at kde.org
Sun Jul 10 05:27:57 UTC 2016
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127984/
-----------------------------------------------------------
(Updated July 9, 2016, 10:27 p.m.)
Status
------
This change has been marked as submitted.
Review request for KDE Frameworks.
Changes
-------
Submitted with commit 0cf33ffbf88777a86635948ed03917253342569c by Pino Toscano to branch master.
Repository: kservice
Description
-------
Add Flex and Bison as required build dependencies, and use them to always regenerate at build time the Trader parser. This ensures that the parser does not rot, and there is no more need to rely on autogenerated sources added statically among the others.
Second commit: remove old generated files of Trader parser, and the helper regen.sh script.
Diffs
-----
CMakeLists.txt 7f9d4f03510c66f230e5c189e92d90a31beb2cf2
src/CMakeLists.txt cdcf88532cb8d7aba0cf7fbd24086d7f5905b6da
src/services/lex.c c811f67fc89a1a1ad0aef0de2feeba00ebd5d057
src/services/regen.sh c2af52b92767ca2dd0d1abe19eb6cf50642b5096
src/services/yacc.h 070ffa4bc895683b4a5b7502a031679c7d70c74a
src/services/yacc.c 675650a8d787c5d7e66ef5e2cf97bdeb65660f4b
Diff: https://git.reviewboard.kde.org/r/127984/diff/
Testing
-------
Builds fine with flex 2.6.0 and bison 3.0.4; `make test` passes too.
Thanks,
Pino Toscano
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20160710/6db6a314/attachment.html>
More information about the Kde-frameworks-devel
mailing list