RFR: 8351277: Remove pipewire from AIX build [v2]
Alexander Zvegintsev
azvegint at openjdk.org
Thu Mar 20 22:07:09 UTC 2025
On Thu, 13 Mar 2025 13:06:38 GMT, Matthias Baesken <mbaesken at openjdk.org> wrote:
>> Seems Wayland is not supported on AIX, so we most likely do not need to build pipewire on AIX and can remove it from the build.
>> Reason is that pipewire updates can break the AIX build, so better avoid it if possible.
>>
>> To check if it works, I move the 'pipewire/include' folder away, and the build still worked on AIX.
>
> Matthias Baesken has updated the pull request incrementally with one additional commit since the last revision:
>
> remove AIX special handling from endian.h
src/java.desktop/unix/native/libawt_xawt/awt/screencast_pipewire.c line 1104:
> 1102: Java_sun_awt_screencast_ScreencastHelper_closeSession(JNIEnv *env, jclass cls) {
> 1103: }
> 1104:
Suggestion:
This doesn't seem necessary since this method should only be called if `loadPipewire` returns true.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24009#discussion_r2006524200
More information about the build-dev
mailing list