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