-
-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
libicu: Bumping to 73.1 #16222
Comments
shouldn't you have actually waited for libicu to finish building before pushing the separate revdeps? i think they're downloading the old one |
@landfillbaby CI does not work that way. The new version of |
oh so it's just being built 5 times then? |
Exactly. |
ah you're right i just checked. still, waiting for it to build once before pushing the revdeps would have been better i think. for next time |
That would increase the chance that the aforementioned incosistencies could happen. For example, thorough upgrading after |
So I tried to synchronize the upload timing of revdeps as much as possible, preferebly within 30 minutes. Other maintainers are not expected to do that though. I'm aware it is kind of overkill. |
I assume that by "bindist" you mean aptly's apt repository on the server. And anyways, this is not a problem at all, we have been having parallel commits modifying various packages and having uploads, usually only the upload step of the CI fails that too only if by coincidence multiple upload jobs happen at same time (which is rare), which can be re-run, and things work as expected without any inconsistencies |
@thunder-coding See what happened to arm build in https://github.com/termux/termux-packages/actions/runs/4709004821. |
Excerpt from the build log:
We suffer from this from time to time. |
As you see this "interference" by other pushes only happens when starting build of a package, and does not happen after the build of the last package in CI starts successfully, which now is the case with the CIs for So maintainers are now safe to push commits, even though not all CIs are finished. Thanks. |
Hi. I was told to report this here (from the Telegram group):
It happens when trying to upgrade nodejs-lts. |
@DarremMolko Yes, this is a typical instance of inconsistencies. Please wait for a while. Thanks. |
CI has completed. Note that it will take some time for mirrors to reflect the change. Please use the main server (temporarily) if necessary. Thanks. Keeping this open for a while. |
a way to avoid this issue in future would be a refactor to separate downloading in the builds to a separate job, and mess with the github actions concurrency so as well as the current way of only allowing one upload at a time, the uploads and downloads are mutually exclusive. |
libicu
is now bumping to 73.1. Revdeps are split into several pushes and are being rebuilt. Inconsistencies in bindist can happen but are expected to be resolved within few hours in the main bindist server. Please refrain from filing individual bugs for that. Those bug reports that do not mention this issue will be immedately closed as duplicates.Also please refrain from pushing commits that modify bindist while CIs for
libicu
and revdeps are running, to avoid build failure in CI.Sorry for your inconvenience and thanks for your understanding.
The text was updated successfully, but these errors were encountered: