Review Request 128968: Convert checkxml5 into a qt executable for cross plattform support.

Ralf Habacker ralf.habacker at freenet.de
Wed Sep 21 13:59:23 UTC 2016



> On Sept. 21, 2016, 12:38 nachm., Luigi Toscano wrote:
> > Would it make more sense to change meinproc5 to work like checkXML5 if invoked with that name (hardlink, copy, etc, a-la busybox)?
> 
> Ralf Habacker wrote:
>     Sorry, I do not know the reasons why this tool has been created and has not been integrated into meinproc5. You should ask the original author.
> 
> Luigi Toscano wrote:
>     The original author (Laurent Montel, https://websvn.kde.org/?view=revision&revision=505145) don't remember much, and I guess it's not relevant here. My question was: from a multiplatform prospective, would this solution have any drawback? Where "this solution" is: have a checkXML5(.exe) executable which is an hardlink (or a copy, I guess, on Windows) to meinproc5, but exposing the behavior of the wrapper script?

Pro's:
- no additional source/cmake target/binary to maintain 

Neg's:
- increase the complexibility of meinproc5 caused by the need to support of additional 'checkXML5' use case
- more efforts required to maintain the related man pages caused by the dualism  e.g. 'Do I need to update "meinproc5" or "checkXML5" man page 
- duplicated executable on windows caused by pysical copy (hardlinks are not supported by cmake as far as I know)

Seeing this I would stay with the checkXML5 executable introduced with this request.


- Ralf


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


On Sept. 21, 2016, 11:35 vorm., Ralf Habacker wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128968/
> -----------------------------------------------------------
> 
> (Updated Sept. 21, 2016, 11:35 vorm.)
> 
> 
> Review request for Documentation, KDE Frameworks and Luigi Toscano.
> 
> 
> Repository: kdoctools
> 
> 
> Description
> -------
> 
> Convert checkxml5 into a qt executable for cross plattform support.
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt 8674faaba6346156907aa5fbbea1ee2b38865eda 
>   checkXML5.in.cmake d7a57c78ec768a4eb9d3c669b8fe3bbef2991c93 
>   src/CMakeLists.txt 6f903b542bc9ba256dd379275567d5ff2127fe39 
>   src/checkXML5.cpp PRE-CREATION 
> 
> Diff: https://git.reviewboard.kde.org/r/128968/diff/
> 
> 
> Testing
> -------
> 
> compiled at https://build.opensuse.org/package/show/home:rhabacker:branches:KDE:Frameworks526/kdoctools
> 
> 
> Thanks,
> 
> Ralf Habacker
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20160921/14778481/attachment.html>


More information about the Kde-frameworks-devel mailing list