Bootstrap build of Portola on Alpine

[David Holmes]

On 18/05/2017 4:58 PM, Josh Graham wrote:
> OK Updated the gist, the extra args to config no longer needed :-)
>
> The symlink of sys/poll.h is because of this:
>
>
> In file included from /usr/include/alsa/asoundlib.h:38:0,

Ah! Not much we can do about ancient system header files :(

The warning actually comes from the musl headers:

http://git.musl-libc.org/cgit/musl/commit/?id=54446d730cfb17c5f7bcf57f139458678f5066cc

Will need to disable warning-are-errors in jdk/make/lib/SoundLibraries.gmk

David

>                  from
> /code/jdk/src/java.desktop/unix/native/libjsound/PLATFORM_API_LinuxOS_ALSA_CommonUtils.h:26,
>                  from
> /code/jdk/src/java.desktop/unix/native/libjsound/PLATFORM_API_LinuxOS_ALSA_CommonUtils.c:29:
> /usr/include/sys/poll.h:1:2: error: #warning redirecting incorrect #include
> <sys/poll.h> to <poll.h> [-Werror=cpp]
>  #warning redirecting incorrect #include <sys/poll.h> to <poll.h>
>   ^
> In file included from /usr/include/alsa/asoundlib.h:38:0,
>                  from
> /code/jdk/src/java.desktop/unix/native/libjsound/PLATFORM_API_LinuxOS_ALSA_MidiIn.c:32:
> /usr/include/sys/poll.h:1:2: error: #warning redirecting incorrect #include
> <sys/poll.h> to <poll.h> [-Werror=cpp]
>  #warning redirecting incorrect #include <sys/poll.h> to <poll.h>
>   ^
> cc1: all warnings being treated as errors
> SoundLibraries.gmk:151: recipe for target
> '/code/build/linux-x86_64-normal-server-release/support/native/java.desktop/libjsoundalsa/PLATFORM_API_LinuxOS_ALSA_CommonUtils.o'
> failed
> make[3]: ***
> [/code/build/linux-x86_64-normal-server-release/support/native/java.desktop/libjsoundalsa/PLATFORM_API_LinuxOS_ALSA_CommonUtils.o]
> Error 1
> make[3]: *** Waiting for unfinished jobs....
> In file included from /usr/include/alsa/asoundlib.h:38:0,
>                  from
> /code/jdk/src/java.desktop/unix/native/libjsound/PLATFORM_API_LinuxOS_ALSA_MidiOut.c:31:
> /usr/include/sys/poll.h:1:2: error: #warning redirecting incorrect #include
> <sys/poll.h> to <poll.h> [-Werror=cpp]
>  #warning redirecting incorrect #include <sys/poll.h> to <poll.h>
>   ^
> cc1: all warnings being treated as errors
> cc1: all warnings being treated as errors
> SoundLibraries.gmk:151: recipe for target
> '/code/build/linux-x86_64-normal-server-release/support/native/java.desktop/libjsoundalsa/PLATFORM_API_LinuxOS_ALSA_MidiIn.o'
> failed
> make[3]: ***
> [/code/build/linux-x86_64-normal-server-release/support/native/java.desktop/libjsoundalsa/PLATFORM_API_LinuxOS_ALSA_MidiIn.o]
> Error 1
> make[3]: ***
> [/code/build/linux-x86_64-normal-server-release/support/native/java.desktop/libjsoundalsa/PLATFORM_API_LinuxOS_ALSA_MidiOut.o]
> Error 1
> SoundLibraries.gmk:151: recipe for target
> '/code/build/linux-x86_64-normal-server-release/support/native/java.desktop/libjsoundalsa/PLATFORM_API_LinuxOS_ALSA_MidiOut.o'
> failed
> make[2]: *** [java.desktop-libs] Error 1
> make/Main.gmk:221: recipe for target 'java.desktop-libs' failed
>
>
>
>
> On Wed, May 17, 2017 at 9:54 AM, Mikael Vidstedt <mikael.vidstedt at oracle.com
>> wrote:
>
>>
>> Josh,
>>
>> Did you have a chance to look at which file generates the warning/error?
>>
>> Also, with the latest changes in the portola forest you should (hopefully)
>> be able to drop the —build and —host configure arguments. If you happen to
>> have a chance to try that let me know how it works out for you!
>>
>> Cheers,
>> Mikael
>>
>>> On Apr 26, 2017, at 6:42 PM, Mikael Vidstedt <mikael.vidstedt at oracle.com>
>> wrote:
>>>
>>>
>>>> On Apr 26, 2017, at 6:14 PM, Josh Graham <josh at grahamis.com> wrote:
>>>>
>>>> G'day Mikael,
>>>>
>>>> Out of curiosity, why did you have to do the poll.h softlink
>> workaround? I
>>>>> was naively hoping I had fixed all of the incorrect includes, but
>> maybe I
>>>>> missed something?
>>>>>
>>>>
>>>> With warnings-as-errors the content of `sys/poll.h` emits a warning,
>> thus
>>>> raises an error. I suppose that flag could be turned off. There's a few
>>>> -Xlint warnings, too, which may warrant further investigation.
>>>
>>> Right, but I thought this change would have removed all the references
>> to sys/poll.h (in favor of poll.h without the sys/ prefix):
>>>
>>> http://mail.openjdk.java.net/pipermail/portola-dev/2017-
>> April/000009.html <http://mail.openjdk.java.net/
>> pipermail/portola-dev/2017-April/000009.html>
>>>
>>> http://hg.openjdk.java.net/portola/portola/hotspot/rev/95d3f5d5ca24
>>> http://hg.openjdk.java.net/portola/portola/jdk/rev/e5f4488e151b <
>> http://hg.openjdk.java.net/portola/portola/jdk/rev/e5f4488e151b>
>>>
>>> Which file is generating the warning/error?
>>>
>>>>
>>>>
>>>>
>>>>> bash ./configure --host=x86_64-unknown-linux-musl
>>>>> --build=x86_64-unknown-linux-musl <…>
>>>>>
>>>>
>>>> Trying that now (along with a `make clean`). The new `--openjdk-target`
>>>> argument looks interesting, I'll play with that too at some point.
>>>
>>> Indeed. Erik/Magnus, can you comment on how the —openjdk-target
>> could/should be used here..?
>>>
>>>> After I get a good build, I'll iterate from fresh and incorporate the
>> `sed`
>>>> idea and fewer package installs. Then it's on to runtime testing!
>>>
>>> Sounds great, let me know!
>>>
>>> Cheers,
>>> Mikael
>>>
>>
>>