-
Notifications
You must be signed in to change notification settings - Fork 223
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Prepare Release 3.9.1 #2813
Comments
Note: I'm merely opening this now without any previous discussion to have a sane place to list reminders for the release announcement. I think it makes sense to keep this release small. I haven't checked, but I think we haven't had anything translation-related, so we might be able to skip that and make it a real bugfix/very-small-feature release (as the versioning implies). |
Oops, I forgot about the new nb_NO app translations. Is there any way to use the script for a single language? EDIT: I've raised the issue by hand but can't assign the issue to @comradekingu - presumably because they're not in "translators"? |
@pljones I (just one person) can help fix issues in more languages if need be :) |
If you want to liaise with the people assigned to the other languages, please do. If you want to change the source text, please raise an appropriate pull request. Thanks. |
@jamulussoftware/maindevelopers I think we should open the website translation issues as soon as possible (once we've confirmed the docs are up to date) |
This is why they've not been opened. |
@emlynmac could you please build the RC? |
The accelerator keys should be rechecked as we had a translation update |
@henkdegroot suggested to integrate Weblate into the release process a bit more: #2758 (comment) |
3.9.1 GUI version works fine on my debian 11 machine. Sound is good. |
Windows ASIO and JACK also seem to work fine. |
macOS Legacy also seems ok |
macOS non legacy seems also ok. |
After that we used to delete next-release or force push to it... |
Can someone please clarify: 3.9.1 released in October and is the default download on http://jamulus.io, but we're nearing release of an update that will have the same version number (rather than be 3.9.2)? I need to install a new server but may wait for the update if it's likely to get out in the next week or so. Thanks. |
3.9.1 is the version which just got released. 3.9.2 or 3.10.0 isn't even planned yet. |
As written in the chat, I think we shouldn't have changed the order of steps from the release check list regarding finalizing all builds (signed Mac build) and updating the download links on the website. The website currently provides an unsigned build for Mac users, which is a regression and might show unexpected warnings to users (we've already got one enquiry at team@). To rectify that, I've now pushed the 3.9.1 tag to @emlynmac's repo. I'll upload the artifact to our release package once done. @chrisrimple To clarify further: 3.9.1 has been tagged and frozen. The only thing which will change (binary-wise) is that the Mac build will be replaced by the signed version shortly. It's just that the 3.9.1 release process is unusually stretched this time (most of the steps were usually done only some hours apart, while it's now days). |
Well, I haven't been successful so far, the build gets stuck. The problem seems to be somewhere around the |
I might have found the underlying issue. I'm documenting it here: #2925. A 3.9.1 build with a fix included is currently running here: https://github.com/emlynmac/jamulus/actions/runs/3285054847/jobs/5411736285#step:10:1887 |
The fix works and makes the build complete. @emlynmac did a successful test. I've therefore taken the build from @emlynmac's repo and replaced the unsigned one in our 3.9.1 release. @pljones @ann0see: Should I modify the r3_9_1 tag to include #2927? I think we had a similar case with debian a few releases ago and I think we hadn't modified the tag there. |
According to the check list and to git, |
We are talking about two kind of "latest" tags here though:
The status quo for 3.9.1 is:
It might not have been optimal for now, but I guess we'll now keep that as-is?
See #2813 (comment) -- I had already done that in emlynmac's repo yesterday. My question about the r3_9_1 tag was solely about jamulussoftware/jamulus, as the tag currently points to a different commit (pre-signing fix) than in emlynmac's repo (which is post-signing fix). Unless we want to be super-exact about aligning the r3_9_1 tag and replacing artifacts again (I wouldn't do that as it would imply that there are multiple files named "3.9.1" with different hashes floating around; might even make Windows acceptance harder). People already downloaded 3.9.1 as the Github release creation causes notifications and as the website has already been updated to link to 3.9.1. So, my suggestion would be to continue with the check list and start publishing the announcement. |
Yes, as we've got further than I thought. |
"latest" was what was being pulled by "Simon's Script" when it built from source, IIRC, before it installed the .deb. I think it's also something was quite common: "latest" referencing the latest stable build... Such usage is generally now deprecated, in favour of taking control of dependencies and knowing what your versions are. Whether that makes life simpler for more people or not, I don't know. The problem on Github comes down to the version in
All of that lot needs to happen more or less "as soon as it can" in order. But everyone involved needs to know it's happening and where we are in that flow. |
Yep. Regarding 3.9.1 again: |
On its way.
Still waiting for someone to confirm the website translators - I've not been able to mange to find out how. It's worth double-checking the contributors.
Best after the annoucement, indeed, as it's more for those who don't read the announcement anyway. |
@pljones just for reference: https://hosted.weblate.org/projects/jamulus/#reports shows the translators |
I don't see where you got that on the site. Plain text would be easier for editing the announcement. |
I think you need to be logged in.
It doesn't seem to be filtered for the website only. |
You need to be an admin to get those details too. |
Ok. Good to know. Thanks! |
OK, for this release announcement we'll bundle app and website. That gives: 이정희 |
Are all the website issues for 3.9.1 getting closed? |
Not yet. I think the translation period is not yet over? |
Well, for 3.9.1 it is. Anything now is "after release". |
The website translation still has some time since it started a lot later than expected. So there will be a post release update. |
Website is now updated |
@emlynmac @ann0see @gilgongo I've unpinned this and am about to close -- assuming there were no changes as an outcome of
and
|
OK, closing. |
The Mac stuff might need updating. |
OK, we can have an issue targetting 3.10.0 if it does. |
Target timeline
Current state: Released
Checklist
needs documentation
label for any outstanding PRs flagged for this release and remove that label if done.next-release
to release, set it as "Draft", sanity check for conflicts and any obvious problems.next-release
andrelease
branch. No changes should be made from now on to ensure translators don't have to work twice.tools/create-translation-issues.sh
. Make sure issue text is up-to-date. Add any URLs that will need localisation into the "New/Changed screenshots" section.tools/create-translation-issues.sh
usingweb
argument (see notes in script)..ts
files in master vialupdate
tools/create-translation-issues.sh
is up-to-datetools/create-translation-issues.sh
usingapp
argument.tools/checkkeys.pl
).qm
files vialrelease Jamulus.pro
tools/get_release_contributors.py
, see reminders at the end of this issue.Jamulus.pro
and add the release date to the Changelog header and commitr3_y_z
latest
and push._config.yml
innext-release
release
branch by clicking on "Edit" on the Branches page and adding a_
behindrelease
.next-release
intorelease
release
branch after the site and the.po
files are published by removing the_
from the branch protection rule you edited on the Branches page.Jamulus.pro
(dev
suffix) and ChangeLog (add a header) for the next releaseReminders for the release announcement
The text was updated successfully, but these errors were encountered: