-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112545/
-----------------------------------------------------------

(Updated Sept. 10, 2013, 4:52 p.m.)


Review request for KDE Frameworks and Albert Astals Cid.


Changes
-------

implement david's suggestions.


Description
-------

Now we only have the one, which can be compiled without KArchive by ifdef'ing 
meinproc.cpp.

I'm unsure that KArchive it's that big of a dependency, to make it optional. 
Opinions?


Diffs (updated)
-----

  staging/kdoctools/src/CMakeLists.txt 72bb06e 
  staging/kdoctools/src/meinproc.cpp 5d476f6 
  staging/kdoctools/src/meinproc_simple.cpp 334f13f 

Diff: http://git.reviewboard.kde.org/r/112545/diff/


Testing
-------

Builds, both ways.


Thanks,

Aleix Pol Gonzalez

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

Reply via email to