RFR: 8255883: Avoid multiple GeneratedMethodAccessor for same NativeMethod… [v6]

Jorn Vernee jvernee at openjdk.java.net
Sat Nov 14 12:14:55 UTC 2020


On Sat, 14 Nov 2020 07:55:35 GMT, Hui Shi <hshi at openjdk.org> wrote:

>> Notice that the `/test` command is currently un-available (some implementation concerns are still under consideration). As an alternative, GitHub actions can be used to do basic automatic testing when pushes occur.
>> 
>> However, pushes to the master branch are ignored. The tests didn't run for this PR since it was created from the `master` branch instead of a feature branch (this is ill-advised https://github.com/openjdk/jdk/pull/1070#issuecomment-722108502).
>> 
>> @huishi-hs My advice for now would be to push the changes for this PR to a separate new branch in your fork, for which the GitHub actions workflow should run, and then simply include a link to the workflow here (if you want to share the results).
>
> @JornVernee How can "ready" label be added to this PR? This PR has been reviewed. Would you help on this?

@huishi-hs See the "Integration blocker" section in the PR body; you have a JBS issue and PR title mismatch.

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

PR: https://git.openjdk.java.net/jdk/pull/1070


More information about the core-libs-dev mailing list