<div dir="ltr"><div dir="ltr">On Tue, May 17, 2022 at 10:10 AM Albert Astals Cid <<a href="mailto:aacid@kde.org">aacid@kde.org</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">El dilluns, 16 de maig de 2022, a les 11:50:13 (CEST), Ben Cooksley va escriure:<br>
> On Mon, May 16, 2022 at 9:39 AM Albert Astals Cid <<a href="mailto:aacid@kde.org" target="_blank">aacid@kde.org</a>> wrote:<br>
> <br>
> > El diumenge, 15 de maig de 2022, a les 23:03:27 (CEST), David Faure va<br>
> > escriure:<br>
> > > On samedi 14 mai 2022 20:48:55 CEST Albert Astals Cid wrote:<br>
> > > > El diumenge, 8 de maig de 2022, a les 0:11:35 (CEST), David Faure va<br>
> > escriure:<br>
> > > > > On samedi 7 mai 2022 18:38:44 CEST Antonio Rojas wrote:<br>
> > > > > > El sábado, 7 de mayo de 2022 15:31:17 (CEST), David Faure escribió:<br>
> > > > > > > Dear packagers,<br>
> > > > > > ><br>
> > > > > > > KDE Frameworks 5.94.0 has been uploaded to the usual place.<br>
> > > > > > ><br>
> > > > > > > New frameworks: none this time.<br>
> > > > > ><br>
> > > > > > This should have been a reply to this mail of course<br>
> > > > > ><br>
> > > > > > >  tr tt ug uk and uz translations are missing from all tarballs.<br>
> > > > ><br>
> > > > > Thanks for noticing. My "update l10n" script did show an error about<br>
> > that<br>
> > > > > but I didn't see that error and moved on. I have now improved things<br>
> > so I<br>
> > > > > can't launch the next step if this step failed.<br>
> > > > ><br>
> > > > > I now uploaded new tarballs (also including<br>
> > > > > <a href="https://invent.kde.org/frameworks/" rel="noreferrer" target="_blank">https://invent.kde.org/frameworks/</a><br>
> > plasma-framework/-/merge_requests/523)<br>
> > > > ><br>
> > > > > I hope it's all fine now, let me know otherwise.<br>
> > > ><br>
> > > > I've just realized that the packages dependencies have not been<br>
> > increased to<br>
> > > > 5.94 and they are just at 5.93. I guess that's not that bad, but just<br>
> > want<br>
> > > > to make sure that doesn't mean some other change may be also missing?<br>
> > ><br>
> > > Good catch. Fortunately nothing else should be missing. The reason this<br>
> > didn't happen is that I run<br>
> > ><br>
> > > cd ../src && ./kdesrc-build --src-only && cd ../release-tools &&<br>
> > ./list_frameworks.sh && ./update_l10n.sh &&<br>
> > ./increase_frameworks_version.sh step2<br>
> > ><br>
> > > and update_l10n failed because anonsvn threw me out in the middle of it.<br>
> > > I didn't notice, ran the next step (./make_rc_tag.sh) and this is how we<br>
> > got tags with missing languages.<br>
> > > Then I fixed l10n and tagged again, completely missing that<br>
> > "increase_frameworks_version.sh step2" was never done.<br>
> > ><br>
> > > Sigh, I wish <a href="http://svn.kde.org" rel="noreferrer" target="_blank">svn.kde.org</a> would work for the way we're using it at<br>
> > release time.<br>
> ><br>
> > One thing we may try is moving Frameworks to the "po injection" test group.<br>
> ><br>
> > "po injection" = scripty commits po files back to git<br>
> ><br>
> > Plasma Mobile Gear has been using it and as far as I understand it is<br>
> > working relatively well.<br>
> ><br>
> > Example:<br>
> > <a href="https://invent.kde.org/plasma-mobile/alligator/-/commits/master/po" rel="noreferrer" target="_blank">https://invent.kde.org/plasma-mobile/alligator/-/commits/master/po</a><br>
> ><br>
> > This way you can remove any special handling of l10n altogether from the<br>
> > KDE Frameworks release scripts<br>
> ><br>
> <br>
> Is there any particular reason why we can't rollout po injection to all<br>
> repositories?<br>
<br>
We need to adapt the release scripts not to fetch l10n, we also have various release scripts that inject ki18n_install/kdoctools_install when they do the fecth, so we need to properly add those calls to the root CMakeLists.txt </blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Not a huge blocker, but it needs to be decided that we should do it and then do it.<br></blockquote><div><br></div><div>Given the number of benefits it has to release managers (no more tooling needed to download translations and integrate them), as well as build processes for nightly builds and for builds on developer systems I think we've a fairly compelling reason to switch it on globally. It will also ensure good CI coverage of the CMake files which should stop issues around ki18n_install macros which have happened a few times.</div><div><br></div><div>It'll also reduce the load on Leptone (the server for <a href="http://invent.kde.org">invent.kde.org</a> and <a href="http://svn.kde.org">svn.kde.org</a>) at release time.</div><div><br></div><div>Cheers,</div><div>Ben</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Cheers,<br>
  Albert<br>
<br>
> <br>
> <br>
> ><br>
> > Cheers,<br>
> >   Albert<br>
> ><br>
> <br>
> Cheers,<br>
> Ben<br>
> <br>
> <br>
> ><br>
> > ><br>
> > > I have now added more error checking, including at the beginning of<br>
> > make_rc_tag.sh,<br>
> > > as well as a reminder to run "step2" after fixing the l10n problems.<br>
> > ><br>
> > ><br>
> ><br>
> ><br>
> ><br>
> ><br>
> ><br>
> <br>
<br>
<br>
<br>
<br>
</blockquote></div></div>