-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/117795/#review56670
-----------------------------------------------------------


This review has been submitted with commit 
8ea94caa0ad7efc94d6aea3688af3b40ae99ea86 by Aurélien Gâteau to branch master.

- Commit Hook


On April 27, 2014, 9:40 a.m., Aurélien Gâteau wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/117795/
> -----------------------------------------------------------
> 
> (Updated April 27, 2014, 9:40 a.m.)
> 
> 
> Review request for KDE Frameworks and Aurélien Gâteau.
> 
> 
> Repository: kapidox
> 
> 
> Description
> -------
> 
> The dependency diagram code used to extract the framework name from the name 
> of the yaml file, but that is not possible anymore. We can't use the name of 
> the directory containing the code because build.kde.org stores the framework 
> code under different names. Therefore I decided to use the name of the 
> project() from the toplevel CMakeLists.txt. I rewrote parse_fancyname() to 
> get the info from the CMakeLists.txt instead of README.md for consistency.
> 
> That patch is unfortunately quite large as the changes cascaded quite a lot.
> 
> 
> Diffs
> -----
> 
>   src/kgenapidox 8fbffb1 
>   src/kapidox/data/dependencies.md.mustache 5d34042 
>   src/kapidox/__init__.py 61e6a13 
>   src/depdiagram-prepare 320502f 
>   src/kgenframeworksapidox 101f9b3 
> 
> Diff: https://git.reviewboard.kde.org/r/117795/diff/
> 
> 
> Testing
> -------
> 
> - Regenerated all the diagrams.
> - Ran kgenframeworksapidox on all the frameworks.
> - Ran kgenapidox on one framework.
> 
> 
> Thanks,
> 
> Aurélien Gâteau
> 
>

_______________________________________________
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Reply via email to