[jdk17u-dev] RFR: 8284772: GHA: Use GCC Major Version Dependencies Only
Aleksey Shipilev
shade at openjdk.org
Mon Aug 21 10:16:33 UTC 2023
On Fri, 18 Aug 2023 16:16:29 GMT, Andrew John Hughes <andrew at openjdk.org> wrote:
> GHA regularly breaks because we specify a very explicit GCC version, even down to the release versioning of the Ubuntu package.
>
> Rather than bumping this yet again like https://github.com/openjdk/jdk17u/commit/260f28761a87553fd634615df5ba3db5b76ae692, this PR suggests dropping the specific version as we did some time ago in 8u and I've also proposed to fix the same breakage in 11u (https://github.com/openjdk/jdk11u-dev/pull/2087). The requirement still specifies a specific major version of GCC. It just means the dependency isn't broken every time Ubuntu bumps to a new minor release or even just makes a minor change to the package alone.
>
> Note that the current setup does not guarantee sticking with an exact version of GCC anyway, because - as seen by recent GHA breakage - older versions get removed from the package repository. All we get from this exact version requirement is sporadic breakage. If we truly want a static version of GCC, we need to provide our own as we do with the JDK.
>
> This is based on https://github.com/openjdk/jdk17u-dev/pull/1595 to allow https://github.com/openjdk/jdk17u-dev/pull/1672 to proceed.
Nope, not for 17u.
While the actual removal of minor GCC version tracking might be okay, it is wrong to keep effectively _forward-porting_ the GHA change from 8u -> 11u -> 17u. You need to do this in mainline, argue the point on minor GCC version removals (in other words, the reversal of [JDK-8256393](https://bugs.openjdk.org/browse/JDK-8256393)), and then argue for backports. Otherwise, you will keep doing this for every subsequent XXu.
-------------
Changes requested by shade (Reviewer).
PR Review: https://git.openjdk.org/jdk17u-dev/pull/1673#pullrequestreview-1586671811
More information about the jdk-updates-dev
mailing list