RFR: 1329: incorrect hgupdate label for JDK 7 updates
Erik Joelsson
erikj at openjdk.java.net
Thu Jan 27 14:49:35 UTC 2022
On Thu, 27 Jan 2022 09:55:04 GMT, Magnus Ihse Bursie <ihse at openjdk.org> wrote:
>> This patch adds special handling of Backports for a couple of historic releases where we have 30 or more builds (and no BPRs).
>
> jbs/src/main/java/org/openjdk/skara/jbs/Backports.java line 320:
>
>> 318: if (resolvedInBuildNumber < 30) {
>> 319: ret.add(jdkVersion.feature());
>> 320: } else if (resolvedInBuildNumber < 60) {
>
> In the JBS comment you stated that the spec says BPR builds should only be < 39. Did you chose not to change the current formulation, or did you forget?
I initially changed this to 40, as the specification we have says BPR between 30 and 39. The problem is that this leaves a gap between 40 and 59 which is undefined. Builds 60 and up are special and should never be considered for hgupdate-sync, but after contemplating this, I agree with the original author's conclusion that the best way to deal with 40-59 is to assume they are also BPRs. In reality, I don't think we have any release with builds in that range, so it doesn't actually matter.
-------------
PR: https://git.openjdk.java.net/skara/pull/1280
More information about the skara-dev
mailing list