translation updates in Krusader stable branch

[Albert Astals Cid]

El diumenge, 23 d’octubre de 2022, a les 18:12:43 (CEST), Luigi Toscano va 
escriure:
> Nikita Melnichenko ha scritto:
> > Hi Albert and Luigi,
> > 
> > Thank you for your responses. I tried the old script
> > (create_tarball_kf5.rb) and the output seems to be similar to what I had
> > seen before except for the expected source code fetch errors, however it
> > didn't download any translations.
> > 
> > I found that Albert deleted the script from the repo with the message
> > <https://invent.kde.org/sdk/kde-dev-scripts/-/commit/1e2f51fbffc838308dd84
> > 3f92d35219e58485fce> it no longer works and suggestion to migrate to
> > releaseme. I tried this: /$ ./tarme.rb --origin stable --version
> > 2.8.0-beta1 krusader//
> > //           !! Make sure to read the wiki :) !!//
> > //     https://community.kde.org/ReleasingSoftware//
> > //INFO -- ReleaseMe::Release: Getting CI states.//
> > //INFO -- ReleaseMe::Release: Getting source (git -
> > git at invent.kde.org:utilities/krusader [stable])//
> > //INFO -- ReleaseMe::Release:  Getting translations...//
> > //INFO -- ReleaseMe::L10n: Found existing translations: ru, sr at latin, ca,
> > pl, bg, de, et, pa, zh_TW, hr, nn, mai, ka, uk, gl, es, ja,
> > sr at ijekavianlatin, ro, eu, sr, ar, tr, ug, ca at valencia, en_GB, sk, zh_CN,
> > nb, pt, da, nds, ga, lt, bs, sv, fr, sl, it, pt_BR, ia, el, fi, eo, hu,
> > sr at ijekavian, ko, mr, cs, nl// //INFO -- ReleaseMe::Release:  Getting
> > documentation...//
> > //INFO -- ReleaseMe::DocumentationL10n: Downloading documentations for
> > krusader-2.8.0-beta1//
> > //INFO -- ReleaseMe::DocumentationL10n: *No translations for: as, az, af,
> > ast, ar, be, be at latin, bn, bg, bn_IN, bs, br, ca at valencia, crh, csb, cy,
> > da, el, en_GB, eo, eu, fa, et, fi, fy, ga, gd, gl, gu, ha, he, hne, hi,
> > hr, hsb, hu, hy, ia, id, is, ja, ka, kab, kk, km, kn, ko, lb, ku, lv,
> > mai, mk, ml, mr, ms, mt, my, nb, nds, ne, nn, nso, oc, pa, or, pl, ro,
> > ps, rw, se, si, sk, sl, sq, sr, sr at ijekavian, sr at ijekavianlatin,
> > sr at latin, ta, te, tg, th, tr, tn, tt, ug, uz, uz at cyrillic, vi, wa, xh,
> > zh_CN, zh_HK, zh_TW*//
> > //INFO -- ReleaseMe::Release: Archiving source (git -
> > git at invent.kde.org:utilities/krusader [stable])/
> > 
> > It didn't download any translations as well.
> 
> It didn't download any translation because the translations for messages are
> already inside the repository (see the first line "Found existing
> translations"). And then it tried to download the translation for the
> documentation, but those are inside the repository too, so it tried to get
> the missing ones, and they are not available ("No translations for: " lists
> languages for which there is no fully translated docbook - you may see that
> it and fr, just to name two, are not listed).
> 
> > Since the scripts are not working, would it be possible to enable
> > injection
> > into the stable branch any time soon? I was planning to release in 4
> > weeks,
> > but the exact date is not set yet, so I'm flexible. If injection is hard
> > to
> > setup in a reasonable time, could you please help looking into why it
> > fails to download?
> 
> The scripts are working, the problem here is that this is the fist project
> which had a stable branch created from master after the injection started,
> so yes, we probably need to enable the injection just for it to start with.

I have enabled po injection for krusader in the stable branch now.

Cheers,
  Albert

> > In addition, a quick question. I see that the tarme script adds this
> > snippet to CMakeLists.txt:
> > /  find_package(KF5DocTools CONFIG)//
> > //  if(KF5DocTools_FOUND)//
> > //    kdoctools_install(po)//
> > //  endif()/
> > 
> > Should I add this piece to the source code directly? We only have
> > ki18n_install(po) call in CMakeLists.txt.
> 
> You can and should add it, in master first and backport it.
> 
> Ciao