RFR: Fix issue with missing JSON classes in make build [v2]

Athijegannathan Sundararajan sundar at openjdk.org
Mon Sep 9 04:16:16 UTC 2024


On Fri, 6 Sep 2024 20:47:52 GMT, Jorn Vernee <jvernee at openjdk.org> wrote:

>> Currently, the `make` build is broken, since we added a new package, but the make build doesn't build the source files in that package. 
>> 
>> This patch adds compilation of the new sources, and adds the missing `--add-exports` flags when running tests. Instead of listing each package in the javac command, I've switched to module source path-based compilation. So, if we ever change the packages in the future, we should run into the same problem again.
>
> Jorn Vernee has updated the pull request incrementally with one additional commit since the last revision:
> 
>   use module-based compilation

LGTM

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

Marked as reviewed by sundar (Committer).

PR Review: https://git.openjdk.org/jextract/pull/259#pullrequestreview-2288782065


More information about the jextract-dev mailing list