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