RFR: 8349751: AIX build failure after upgrade pipewire to 1.3.81 [v3]

Alexander Zvegintsev azvegint at openjdk.org
Tue Feb 11 11:45:11 UTC 2025


On Tue, 11 Feb 2025 08:33:09 GMT, Julian Waters <jwaters at openjdk.org> wrote:

>> Alexander Zvegintsev has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   move fix to spa/utils/endian.h
>
> src/java.desktop/unix/native/libpipewire/include/spa/param/audio/raw.h line 16:
> 
>> 14: #if !defined(AIX)
>> 15: #include <spa/utils/endian.h>
>> 16: #endif
> 
> The ifdef conditional blocks could be merged if I am not mistaken?

Yes, at first I kept it as it was in the original fix.
But after revisiting it, I think that it is belongs in the new `spa/utils/endian.h` file.

@MBaesken could you please recheck if it still works for you?

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/23543#discussion_r1950694146


More information about the client-libs-dev mailing list