> On Sept. 21, 2016, 1:52 p.m., Luigi Toscano wrote: > > src/CMakeLists.txt, line 173 > > <https://git.reviewboard.kde.org/r/128969/diff/1/?file=477241#file477241line173> > > > > docbookl10nhelper is not meant to be installed > > Ralf Habacker wrote: > To cross compile kdoctools for windows on e.g. linux native running tools > like meinproc5, docbookl10nhelper and checkXML5 are required. On > bootstrapping cross compile these native tools are created and packaged > together in a packages named mingwXX-cross-kf5-tools like similar cross > helper tools for example mingw32-cross-gcc. > > Luigi Toscano wrote: > Uhm, but normally we don't need it - can't this installation be done > under some condition, so that the default is not changed? > > Ralf Habacker wrote: > Sonnet for example installs similar tools 'parsetrigrams' and > 'gentrigrams' also unconditional. On opensuse they are packaged as part of > the development package. see > https://build.opensuse.org/package/view_file/KDE:Frameworks5/sonnet/sonnet.spec?expand=1 > line 134,135. > > If this is an absolute nogo there would be still the choice to use a > specific configure time switch like INSTALL_ALL or similar ?
docbookl10nhelper only exists because I was unable to do the same tasks with cmake (or maybe it's just not possible). So it's really part of the compilation system. My comment is not an "absolute no go" to this change, it's a "I'm fine if it does not change the default behavior". For example mainproc5 can be compiled without bz2 support *only* for internal usage on our server for documentation, but it's not the default and it's clearly marked as such. - Luigi ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128969/#review99351 ----------------------------------------------------------- On Sept. 21, 2016, 11:43 a.m., Ralf Habacker wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/128969/ > ----------------------------------------------------------- > > (Updated Sept. 21, 2016, 11:43 a.m.) > > > Review request for Documentation, KDE Frameworks and Luigi Toscano. > > > Repository: kdoctools > > > Description > ------- > > Add cross compiling support for docbookl10nhelper. > > > Diffs > ----- > > src/CMakeLists.txt 6f903b542bc9ba256dd379275567d5ff2127fe39 > > Diff: https://git.reviewboard.kde.org/r/128969/diff/ > > > Testing > ------- > > compiled at > https://build.opensuse.org/package/show/home:rhabacker:branches:KDE:Frameworks526/kdoctools > > > Thanks, > > Ralf Habacker > >
