diff against openjdk/jdk head

[Johan Vos]

Thanks for looking into this.
As for the next steps:
I have a PR that reverts the existing changes, and that brings
openjdk/mobile 100% on par with openjdk/jdk:
https://github.com/openjdk/mobile/pull/25
I tried to create a branch in openjdk/mobile but I don't seem to have
permission to do that. A branch would be a good point to have a copy of the
current mobile repository. If a branch is not possible, a tag should be
sufficient. Once PR-25 is reviewed and approved, I will add a tag to the
current HEAD (e.g. mobile_classic or mobile_gen1) and integrate PR-25
immediately after that.

thoughts?

- Johan


On Thu, May 30, 2024 at 9:09 PM Magnus Ihse Bursie <
magnus.ihse.bursie at oracle.com> wrote:

> On 2024-05-14 16:21, Johan Vos wrote:
>
> > Hi,
> >
> > The current master branch of openjdk/mobile has some changes that are
> > not in upstream openjdk/jdk, but not all of them are (still) relevant
> > and there is also a major syntax error.
> > Rather than adding mobile-specific changes on top of the current
> > master branch, I started from the head of openjdk/jdk to find out what
> > changes I had to make to build both ios and android versions.
> > I did this in my personal fork, in a branch named android-may-2024:
> > https://github.com/johanvos/jdk/tree/android-may-2024
> >
> > There are 4 commits in this branch:
> > * changes required to build Android
> > * changes required to build iOS
> > * header files I added for the hotspot build for Android
> > * header files I added for the hotspot build for iOS
> >
> > With these commits, I can build the static libs and VM for ios and
> > android using
> > make static-libs-image
>
> These changes look very clean. I think it would be much better to
> restart from jdk mainline and apply these patches. Have you had any
> progress in figuring out how to best achieve that?
>
> /Magnus
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.org/pipermail/mobile-dev/attachments/20240531/50aa6285/attachment.htm>