[jdk21u-dev] RFR: 8341881: [REDO] java/nio/file/attribute/BasicFileAttributeView/CreationTime.java#tmp fails on alinux3 [v2]
SendaoYan
syan at openjdk.org
Fri Dec 20 09:57:18 UTC 2024
> Hi all,
>
> This pull request contains a backport of commit [f56a1541](https://github.com/openjdk/jdk/commit/f56a154132f7e66b1b65adfa2aa937119999b14a) from the [openjdk/jdk](https://git.openjdk.org/jdk) repository.
>
> The commit being backported was authored by SendaoYan on 14 Oct 2024 and was reviewed by Chen Liang, Severin Gehwolf, Magnus Ihse Bursie and Brian Burkhalter.
>
> The original PR use FFM API to call native library, but we can't use FFM API in jdk21u directly, and the original PR dependency the file `test/lib/native/export.h`, this file not exists in jdk21 repository. So we use JNI instead of FFM API. Before this PR, the file `test/jdk/java/nio/file/attribute/BasicFileAttributeView/libCreationTimeHelper.c` and `test/jdk/java/nio/file/attribute/BasicFileAttributeView/CreationTimeHelper.java` already exists cause this PR can't backport cleanly.
>
> Additional testing:
>
> - [x] linux-x64 build at alinux3, and run the test passed.
> - [x] linux-x64 build at centos7 docker container, and run the test passed.
> - [x] linux-64 compile libCreationTimeHelper.c at `centos6 docker container(glibc2.12)` by gcc 11.2.0 use below command
>
> `libCreationTimeHelper.c` compile test command:
>
> gcc test/jdk/java/nio/file/attribute/BasicFileAttributeView/libCreationTimeHelper.c -Ibuild/linux-x86_64-server-release/jdk/include -I build/linux-x86_64-server-release/jdk/include/linux -c
>
>
> Thanks!
SendaoYan has updated the pull request incrementally with one additional commit since the last revision:
add patch of JDK-8342145.patch
-------------
Changes:
- all: https://git.openjdk.org/jdk21u-dev/pull/1068/files
- new: https://git.openjdk.org/jdk21u-dev/pull/1068/files/846f1b57..b17d88eb
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk21u-dev&pr=1068&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk21u-dev&pr=1068&range=00-01
Stats: 11 lines in 1 file changed: 8 ins; 1 del; 2 mod
Patch: https://git.openjdk.org/jdk21u-dev/pull/1068.diff
Fetch: git fetch https://git.openjdk.org/jdk21u-dev.git pull/1068/head:pull/1068
PR: https://git.openjdk.org/jdk21u-dev/pull/1068
More information about the jdk-updates-dev
mailing list