Re: Date not changed in docbook of lokalize

2018-08-15 Thread Yuri Chornoivan 
середа, 15 серпня 2018 р. 17:39:13 EEST Freek de Kruijf написано:
> One of the latest changes in templates/docmessages/kdesdk/lokalize.pot is
> the change from the date 2018-07-31 to 2018-08-14. This is reflected in nl/
> docmessages/kdesdk/lokalize.po
> 
> However after the generation of nl/docs/kdesdk/lokalize/index.docbook this
> date is not changed.

We have some problems with anongit.kde.org (at least here). So your system 
cannot update the files in your /documentation folder and cannot sync the docs 
with the current file.

Best regards,
Yuri

Date not changed in docbook of lokalize

2018-08-15 Thread Freek de Kruijf 
One of the latest changes in templates/docmessages/kdesdk/lokalize.pot is the 
change from the date 2018-07-31 to 2018-08-14. This is reflected in nl/
docmessages/kdesdk/lokalize.po

However after the generation of nl/docs/kdesdk/lokalize/index.docbook this 
date is not changed.

-- 
fr.gr.

vertaler van KDE
Freek de Kruijf

D14817: Convert manual install test to an autotest

2018-08-15 Thread Ralf Habacker 
habacker added a comment.


  In D14817#309388 , @habacker wrote:
  
  > Agreed, as long as there is Qt on Windows not patched to also use unix 
style path layout, which is the case on OBS
  
  
  but this case should be covered by ECM module 'KDEInstallDirs'.
  
  If KDE_INSTALL_XXX always return a relative dir , it should be save to use

REPOSITORY
  R238 KDocTools

REVISION DETAIL
  https://phabricator.kde.org/D14817

To: habacker, ltoscano
Cc: bcooksley, kde-frameworks-devel, ltoscano, kde-doc-english, michaelh, 
ngraham, bruns, skadinna

D14817: Convert manual install test to an autotest

2018-08-15 Thread Ralf Habacker 
habacker added a comment.


  In D14817#309370 , @bcooksley 
wrote:
  
  > On Windows the correct path to use is $prefix/bin/data/... as that is what 
QStandardPaths expects.
  >  The location $prefix/share/ won't work on Windows, so the test will need 
to handle that case.
  
  
  Agreed, as long as there is Qt on Windows not patched to also use unix style 
path layout, which is the case on OBS

REPOSITORY
  R238 KDocTools

REVISION DETAIL
  https://phabricator.kde.org/D14817

To: habacker, ltoscano
Cc: bcooksley, kde-frameworks-devel, ltoscano, kde-doc-english, michaelh, 
ngraham, bruns, skadinna

D14817: Convert manual install test to an autotest

2018-08-15 Thread Ralf Habacker 
habacker added a comment.


  another option would be to force the install path in the cmake call
  
add_test(kdoctools_install ${CMAKE_CTEST_COMMAND}
  --build-and-test
  "${CMAKE_CURRENT_SOURCE_DIR}/kdoctools-install"
  "${CMAKE_CURRENT_BINARY_DIR}/kdoctools-install"
  --build-generator ${CMAKE_GENERATOR}
  --build-makeprogram ${CMAKE_MAKE_PROGRAM}
  --build-target install
  --build-options
  "-DCMAKE_MODULE_PATH=${CMAKE_MODULE_PATH}"
  
"-DCMAKE_INSTALL_PREFIX:PATH=${CMAKE_CURRENT_BINARY_DIR}/kdoctools-install/destdir"
  "-DKF5DocTools_DIR=${CMAKE_BINARY_DIR}/cmake"
  "-DKDOCTOOLS_CUSTOMIZATION_DIR=${KDOCTOOLS_CUSTOMIZATION_DIR}"
  "-DKDOCTOOLS_ENABLE_HTMLHANDBOOK=1"
 ...
"-DKDE_INSTALL_MANDIR=share/man"
"-DKDE_INSTALL_DOCBUNDLEDIR=share/doc/HTML"

REPOSITORY
  R238 KDocTools

REVISION DETAIL
  https://phabricator.kde.org/D14817

To: habacker, ltoscano
Cc: bcooksley, kde-frameworks-devel, ltoscano, kde-doc-english, michaelh, 
ngraham, bruns, skadinna

D14817: Convert manual install test to an autotest

2018-08-15 Thread Ralf Habacker 
habacker added a comment.


  Because test.qmake do not anything about KDE  install path layout those 
pathes need to be specified in the ctest config in autotests/CMakeLists.txt I 
guess
  
--test-command ${CMAKE_COMMAND}
"-DKDE_INSTALL_MANDIR=${KDE_INSTALL_MANDIR}"
"-DKDE_INSTALL_DOCBUNDLEDIR=${KDE_INSTALL_DOCBUNDLEDIR}"

"-DCMAKE_INSTALL_PREFIX:PATH=${CMAKE_CURRENT_BINARY_DIR}/kdoctools-install/destdir"
-P "${CMAKE_CURRENT_SOURCE_DIR}/kdoctools-install/test.cmake")
  
  and in test.qmake
  
...
set(MANPATH ${CMAKE_INSTALL_PREFIX}/${KDE_INSTALL_MANDIR})
set(HTMLPATH ${CMAKE_INSTALL_PREFIX}/${KDE_INSTALL_DOCBUNDLEDIR})
 ...

REPOSITORY
  R238 KDocTools

REVISION DETAIL
  https://phabricator.kde.org/D14817

To: habacker, ltoscano
Cc: bcooksley, kde-frameworks-devel, ltoscano, kde-doc-english, michaelh, 
ngraham, bruns, skadinna

D14817: Convert manual install test to an autotest

2018-08-15 Thread Ben Cooksley 
bcooksley added a comment.


  On Windows the correct path to use is $prefix/bin/data/... as that is what 
QStandardPaths expects.
  The location $prefix/share/ won't work on Windows, so the test will need to 
handle that case.

REPOSITORY
  R238 KDocTools

REVISION DETAIL
  https://phabricator.kde.org/D14817

To: habacker, ltoscano
Cc: bcooksley, kde-frameworks-devel, ltoscano, kde-doc-english, michaelh, 
ngraham, bruns, skadinna

D14817: Convert manual install test to an autotest

2018-08-15 Thread Ralf Habacker 
habacker added a comment.


  In D14817#309332 , @bcooksley 
wrote:
  
  > The Windows failure looks like a problem with handling of DESTDIR - 
probably due to reliance on it being handled by $DESTDIR instead of %DESTDIR% 
(which is the correct syntax for Windows)
  
  
  No , it is a install path mismatch.  According to 
https://build.kde.org/job/Frameworks%20kdoctools%20kf5-qt5%20WindowsMSVCQt5.10/39/console
 the files are installed below <...>/bin/data 
(https://cgit.kde.org/extra-cmake-modules.git/tree/kde-modules/KDEInstallDirs.cmake)
 but test.qmake expects them in <...>/share 
(https://cgit.kde.org/kdoctools.git/tree/autotests/kdoctools-install/test.cmake)

REPOSITORY
  R238 KDocTools

REVISION DETAIL
  https://phabricator.kde.org/D14817

To: habacker, ltoscano
Cc: bcooksley, kde-frameworks-devel, ltoscano, kde-doc-english, michaelh, 
ngraham, bruns, skadinna

D14817: Convert manual install test to an autotest

2018-08-15 Thread Ralf Habacker 
habacker added a comment.


  In D14817#309303 , @bcooksley 
wrote:
  
  > Try removing that and the error it has should show up on your local system
  
  
  I see. 
  Since this problem has been fixed in kdoctools git repo and kdoctools has 
been rebuilt on KDE CI, packages like kio, which depend on kdoctools, are 
automatically rebuilt, as is the case with OBS?

REPOSITORY
  R238 KDocTools

REVISION DETAIL
  https://phabricator.kde.org/D14817

To: habacker, ltoscano
Cc: bcooksley, kde-frameworks-devel, ltoscano, kde-doc-english, michaelh, 
ngraham, bruns, skadinna

D14817: Convert manual install test to an autotest

2018-08-15 Thread Ben Cooksley 
bcooksley added a comment.


  The Windows failure looks like a problem with handling of DESTDIR - probably 
due to reliance on it being handled by $DESTDIR instead of %DESTDIR% (which is 
the correct syntax for Windows)

REPOSITORY
  R238 KDocTools

REVISION DETAIL
  https://phabricator.kde.org/D14817

To: habacker, ltoscano
Cc: bcooksley, kde-frameworks-devel, ltoscano, kde-doc-english, michaelh, 
ngraham, bruns, skadinna

D14817: Convert manual install test to an autotest

2018-08-15 Thread Ralf Habacker 
habacker added a comment.


  In D14817#309295 , @habacker wrote:
  
  > According to https://cgit.kde.org/kdoctools.git/tree/src/meinproc.cpp#n51 
uses meinproc5  qCCritical(KDocToolsLog) - How can this be enabled on KDE CI ?
  
  
  Got it - see 
https://phabricator.kde.org/R238:81a3d951169b7f575c0a09c2cf19eb911fab9882

REPOSITORY
  R238 KDocTools

REVISION DETAIL
  https://phabricator.kde.org/D14817

To: habacker, ltoscano
Cc: bcooksley, kde-frameworks-devel, ltoscano, kde-doc-english, michaelh, 
ngraham, bruns, skadinna

D14817: Convert manual install test to an autotest

2018-08-15 Thread Ben Cooksley 
bcooksley added a comment.


  You aren't seeing the error the CI system sees because your build prefix 
still exists. Try removing that and the error it has should show up on your 
local system

REPOSITORY
  R238 KDocTools

REVISION DETAIL
  https://phabricator.kde.org/D14817

To: habacker, ltoscano
Cc: bcooksley, kde-frameworks-devel, ltoscano, kde-doc-english, michaelh, 
ngraham, bruns, skadinna

D14817: Convert manual install test to an autotest

2018-08-15 Thread Ralf Habacker 
habacker added a comment.


  In D14817#309281 , @bcooksley 
wrote:
  
  > Dependency Builds triggered to roll it out to all the non Frameworks jobs 
which have already picked this up.
  
  
  How to do this ?

REPOSITORY
  R238 KDocTools

REVISION DETAIL
  https://phabricator.kde.org/D14817

To: habacker, ltoscano
Cc: bcooksley, kde-frameworks-devel, ltoscano, kde-doc-english, michaelh, 
ngraham, bruns, skadinna

D14817: Convert manual install test to an autotest

2018-08-15 Thread Ralf Habacker 
habacker added a comment.


  According to https://cgit.kde.org/kdoctools.git/tree/src/meinproc.cpp#n51 
uses meinproc5  qCCritical(KDocToolsLog) - How can this be enabled on KDE CI ?

REPOSITORY
  R238 KDocTools

REVISION DETAIL
  https://phabricator.kde.org/D14817

To: habacker, ltoscano
Cc: bcooksley, kde-frameworks-devel, ltoscano, kde-doc-english, michaelh, 
ngraham, bruns, skadinna

D14817: Convert manual install test to an autotest

2018-08-15 Thread Ralf Habacker 
habacker added a comment.


  In D14817#309285 , @habacker wrote:
  
  > Funny, that this works with Qt 5.9 
https://build.kde.org/job/Frameworks%20kdoctools%20kf5-qt5%20SUSEQt5.9/ and Qt 
5.11 (locally tested) - it only happens with Qt 5.10
  
  
  Because I cannot reproduce this issue on a local build there is the question 
how

REPOSITORY
  R238 KDocTools

REVISION DETAIL
  https://phabricator.kde.org/D14817

To: habacker, ltoscano
Cc: bcooksley, kde-frameworks-devel, ltoscano, kde-doc-english, michaelh, 
ngraham, bruns, skadinna

D14817: Convert manual install test to an autotest

2018-08-15 Thread Ralf Habacker 
habacker added a comment.


  In D14817#309283 , @habacker wrote:
  
  > This is the remaining issue on non Windows KDE CI Qt 5.10
  >
  > 
https://build.kde.org/job/Frameworks%20kdoctools%20kf5-qt5%20SUSEQt5.10/49/consoleFull
  >
  >   07:02:43 cd "/home/jenkins/workspace/Frameworks kdoctools kf5-qt5 
SUSEQt5.10/autotests/kdoctools-install" && "/home/jenkins/workspace/Frameworks 
kdoctools kf5-qt5 SUSEQt5.10/build/bin/meinproc5" --check --cache 
/home/jenkins/workspace/Frameworks\ kdoctools\ kf5-  qt5\ 
SUSEQt5.10/build/autotests/kdoctools-install/po/es/docs/foobar/index.cache.bz2 
po/es/docs/foobar/index.docbook
  >   07:02:43 gmake[2]: *** 
[CMakeFiles/po-es-docs-foobar-index-cache-bz2.dir/build.make:65: 
po/es/docs/foobar/index.cache.bz2] Error 1
  
  
  Funny, that this works with Qt 5.9 
https://build.kde.org/job/Frameworks%20kdoctools%20kf5-qt5%20SUSEQt5.9/ and Qt 
5.11 (locally tested) - it only happens with Qt 5.10

REPOSITORY
  R238 KDocTools

REVISION DETAIL
  https://phabricator.kde.org/D14817

To: habacker, ltoscano
Cc: bcooksley, kde-frameworks-devel, ltoscano, kde-doc-english, michaelh, 
ngraham, bruns, skadinna

D14817: Convert manual install test to an autotest

2018-08-15 Thread Ralf Habacker 
habacker added a comment.


  In D14817#309281 , @bcooksley 
wrote:
  
  > This change has caused CI wide breakage and makes kdoctools unrelocatable.
  
  
  This has been fixed with commit  
https://phabricator.kde.org/R238:4e48476f47414f1077a181ef56a19f7d9289d50d
  
  See other comments for the remaining issues.

REPOSITORY
  R238 KDocTools

REVISION DETAIL
  https://phabricator.kde.org/D14817

To: habacker, ltoscano
Cc: bcooksley, kde-frameworks-devel, ltoscano, kde-doc-english, michaelh, 
ngraham, bruns, skadinna

D14817: Convert manual install test to an autotest

2018-08-15 Thread Ralf Habacker 
habacker added a comment.


  This is the remaining issue on non Windows KDE CI Qt 5.10
  
  
https://build.kde.org/job/Frameworks%20kdoctools%20kf5-qt5%20SUSEQt5.10/49/consoleFull
  
07:02:43 cd "/home/jenkins/workspace/Frameworks kdoctools kf5-qt5 
SUSEQt5.10/autotests/kdoctools-install" && "/home/jenkins/workspace/Frameworks 
kdoctools kf5-qt5 SUSEQt5.10/build/bin/meinproc5" --check --cache 
/home/jenkins/workspace/Frameworks\ kdoctools\ kf5-  qt5\ 
SUSEQt5.10/build/autotests/kdoctools-install/po/es/docs/foobar/index.cache.bz2 
po/es/docs/foobar/index.docbook
07:02:43 gmake[2]: *** 
[CMakeFiles/po-es-docs-foobar-index-cache-bz2.dir/build.make:65: 
po/es/docs/foobar/index.cache.bz2] Error 1

REPOSITORY
  R238 KDocTools

REVISION DETAIL
  https://phabricator.kde.org/D14817

To: habacker, ltoscano
Cc: bcooksley, kde-frameworks-devel, ltoscano, kde-doc-english, michaelh, 
ngraham, bruns, skadinna

D14817: Convert manual install test to an autotest

2018-08-15 Thread Ben Cooksley 
bcooksley added a comment.


  This change has caused CI wide breakage and makes kdoctools unrelocatable. It 
needs to be reverted urgently, and Dependency Builds triggered to roll it out 
to all the non Frameworks jobs which have already picked this up.

REPOSITORY
  R238 KDocTools

REVISION DETAIL
  https://phabricator.kde.org/D14817

To: habacker, ltoscano
Cc: bcooksley, kde-frameworks-devel, ltoscano, kde-doc-english, michaelh, 
ngraham, bruns, skadinna