Review Request 124331: New proxy: KExtraColumnsProxyModel, allows to add columns to an existing model.

2015-07-12 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124331/ --- Review request for KDE Frameworks, Jesper Pedersen and Stephen Kelly.

Re: Review Request 124328: Man pages for kapidox

2015-07-12 Thread Albert Astals Cid
On jul. 12, 2015, 4:45 p.m., Kevin Krammer wrote: I thought our man pages are written in docbook and then translated by the usual docbook workflow? Albert Astals Cid wrote: Correct. Luigi Toscano wrote: Yes, because they can be translated. But that means a dependency on

Re: Review Request 124328: Man pages for kapidox

2015-07-12 Thread Scott Kitterman
On July 12, 2015, 4:45 p.m., Kevin Krammer wrote: I thought our man pages are written in docbook and then translated by the usual docbook workflow? Albert Astals Cid wrote: Correct. Luigi Toscano wrote: Yes, because they can be translated. But that means a dependency on

Re: Review Request 124331: New proxy: KExtraColumnsProxyModel, allows to add columns to an existing model.

2015-07-12 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124331/#review82440 --- autotests/kextracolumnsproxymodeltest.cpp (line 1)

Re: Review Request 124226: Delete old highlighter

2015-07-12 Thread Lasse Liehu
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124226/ --- (Updated July 12, 2015, 8:12 a.m.) Status -- This change has been

Re: Review Request 124328: Man pages for kapidox

2015-07-12 Thread Alex Merry
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124328/#review82378 --- Ship it! For all that troff is dark magic, the important

Re: Review Request 124328: Man pages for kapidox

2015-07-12 Thread Allen Winter
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124328/#review82394 --- hmm.. I sorta against committing roff code into our

Re: Review Request 124328: Man pages for kapidox

2015-07-12 Thread Scott Kitterman
On July 12, 2015, 4:31 p.m., Allen Winter wrote: hmm.. I sorta against committing roff code into our repository. Since very few people will be able to understand it in this day and age. perhaps you can write the source for the man pages in pod and convert them to roff using

Re: Review Request 124328: Man pages for kapidox

2015-07-12 Thread Scott Kitterman
On July 12, 2015, 4:45 p.m., Kevin Krammer wrote: I thought our man pages are written in docbook and then translated by the usual docbook workflow? Albert Astals Cid wrote: Correct. Luigi Toscano wrote: Yes, because they can be translated. But that means a dependency on

Re: Review Request 124328: Man pages for kapidox

2015-07-12 Thread Albert Astals Cid
On jul. 12, 2015, 4:45 p.m., Kevin Krammer wrote: I thought our man pages are written in docbook and then translated by the usual docbook workflow? Correct. - Albert --- This is an automatically generated e-mail. To reply, visit:

Re: Review Request 124328: Man pages for kapidox

2015-07-12 Thread Luigi Toscano
On Lug. 12, 2015, 6:45 p.m., Kevin Krammer wrote: I thought our man pages are written in docbook and then translated by the usual docbook workflow? Albert Astals Cid wrote: Correct. Yes, because they can be translated. But that means a dependency on kdoctools. On the other side,

Re: Review Request 124328: Man pages for kapidox

2015-07-12 Thread Kevin Krammer
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124328/#review82397 --- I thought our man pages are written in docbook and then

Re: Review Request 124328: Man pages for kapidox

2015-07-12 Thread Scott Kitterman
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124328/ --- (Updated July 12, 2015, 4 p.m.) Review request for KDE Frameworks, Alex