[jdk17u-dev] RFR: 8273979: move some os time related functions to os_posix for POSIX platforms
Radim Vansa
rvansa at openjdk.org
Thu Sep 5 10:04:57 UTC 2024
On Thu, 5 Sep 2024 07:27:00 GMT, Goetz Lindenmaier <goetz at openjdk.org> wrote:
>> Hi all,
>>
>> This pull request contains a backport of commit[ 1c6fa113](https://github.com/openjdk/jdk/commit/1c6fa113bd06585945e077f0d2004a291c27946d) from the [openjdk/jdk](https://git.openjdk.org/jdk) repository.
>>
>> The commit being backported was authored by Matthias Baesken on 23 Sep 2021 and was reviewed by David Holmes and Lutz Schmidt.
>>
>> The backport applies cleanly, was created by the /backport command.
>>
>> Thanks!
>
> Hi @rvansa
> I am not so happy about this backport.
> Simplifying another project is not an eligible reason to backport things.
> Also, you did not test macos and expecially not aix. Does that still build?
Hi @GoeLin , understood, though I am rather trying to be honest than making up a synthetic argument. Since it was accepted in the mainline I could say that it is an improvement in code quality.
MacOS should be tested here in GHA, regrettably I don't know where to get AIX for testing.
-------------
PR Comment: https://git.openjdk.org/jdk17u-dev/pull/2831#issuecomment-2331118243
More information about the jdk-updates-dev
mailing list