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