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 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 depende

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 depende

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

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 depende

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

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, vi

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 transl

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 repository

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 M

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. Rep

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 part

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 ma