<div dir="ltr"><div dir="ltr">On Sat, May 21, 2022 at 8:38 PM 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 dimarts, 17 de maig de 2022, a les 8:53:16 (CEST), Ben Cooksley va escriure:<br>
> On Tue, May 17, 2022 at 10:10 AM Albert Astals Cid <<a href="mailto:aacid@kde.org" target="_blank">aacid@kde.org</a>> wrote:<br>
> <br>
> > El dilluns, 16 de maig de 2022, a les 11:50:13 (CEST), Ben Cooksley va<br>
> > 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<br>
> > 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<br>
> > 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<br>
> > tarballs.<br>
> > > > > > ><br>
> > > > > > > Thanks for noticing. My "update l10n" script did show an error<br>
> > about<br>
> > > > that<br>
> > > > > > > but I didn't see that error and moved on. I have now improved<br>
> > 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<br>
> > just<br>
> > > > want<br>
> > > > > > to make sure that doesn't mean some other change may be also<br>
> > missing?<br>
> > > > ><br>
> > > > > Good catch. Fortunately nothing else should be missing. The reason<br>
> > 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<br>
> > it.<br>
> > > > > I didn't notice, ran the next step (./make_rc_tag.sh) and this is<br>
> > 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<br>
> > 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<br>
> > 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<br>
> > various release scripts that inject ki18n_install/kdoctools_install when<br>
> > they do the fecth, so we need to properly add those calls to the root<br>
> > CMakeLists.txt<br>
> <br>
> <br>
> > Not a huge blocker, but it needs to be decided that we should do it and<br>
> > then do it.<br>
> ><br>
> <br>
> Given the number of benefits it has to release managers (no more tooling<br>
> needed to download translations and integrate them), as well as build<br>
> processes for nightly builds and for builds on developer systems I think<br>
> we've a fairly compelling reason to switch it on globally. <br>
<br>
I am not disagreeing and that's always been the plan, but someone needs to do the work for that to be possible :)<br>
<br>
I can only do so much, so help always welcome :)<br></blockquote><div><br></div><div>What sort of work is required here?</div><div><br></div><div>My understanding is everything on the scripty/infrastructure side is in place?</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></blockquote><div><br></div><div>Thanks,</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>
> It will also<br>
> ensure good CI coverage of the CMake files which should stop issues around<br>
> ki18n_install macros which have happened a few times.<br>
> <br>
> It'll also reduce the load on Leptone (the server for <a href="http://invent.kde.org" rel="noreferrer" target="_blank">invent.kde.org</a> and<br>
> <a href="http://svn.kde.org" rel="noreferrer" target="_blank">svn.kde.org</a>) at release time.<br>
> <br>
> Cheers,<br>
> Ben<br>
> <br>
> <br>
> ><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>
> ><br>
> <br>
<br>
<br>
<br>
<br>
</blockquote></div></div>