March 2022 - jextract-dev

RFR: Testing GHA
by Jorn Vernee 28 Apr '22

28 Apr '22

Add a basic actions workflow that runs the tests on pull requests ------------- Commit messages: - Use Oracle JDK - Back to repo hosted acitons - Update test.yml - Change actions to run in fork instead - Remove java check from test run - Typo - Add missing blank lines - Checkout specific jtreg tag - Working Impl - Remove whitespace - ... and 2 more: https://git.openjdk.java.net/jextract/compare/e6345e3a...c3d97f9a Changes: https://git.openjdk.java.net/jextract/pull/6/files Webrev: https://webrevs.openjdk.java.net/?repo=jextract&pr=6&range=00 Stats: 64 lines in 1 file changed: 64 ins; 0 del; 0 mod Patch: https://git.openjdk.java.net/jextract/pull/6.diff Fetch: git fetch https://git.openjdk.java.net/jextract pull/6/head:pull/6 PR: https://git.openjdk.java.net/jextract/pull/6

2 13

test email - please ignore
by Maurizio Cimadamore 31 Mar '22

31 Mar '22

Please ignore Maurizio

1 0

RFR: Update Readme with macOs specifics
by Brice Dutheil 30 Mar '22

30 Mar '22

The macOs libclang location as well as build output differ enough to warrant specific information. In particular the use of the XCode based path to find the libclang is not an easy find. The output also makes use of jpackage with `app-image` type, but the macOs _.app_ layout different enough on macOs. ------------- Commit messages: - Update Readme with macOs specifics Changes: https://git.openjdk.java.net/jextract/pull/10/files Webrev: https://webrevs.openjdk.java.net/?repo=jextract&pr=10&range=00 Stats: 8 lines in 1 file changed: 6 ins; 0 del; 2 mod Patch: https://git.openjdk.java.net/jextract/pull/10.diff Fetch: git fetch https://git.openjdk.java.net/jextract pull/10/head:pull/10 PR: https://git.openjdk.java.net/jextract/pull/10

3 31

RFR: 7903136: Don't require jdk18_home to be set in jextract gradle build
by Jorn Vernee 30 Mar '22

30 Mar '22

Drop the requirement to set `jdk18_home` in the gradle build, and rely instead on `JAVA_HOME` being set to JDK 18. I've also updated the build instructions. The `-Pjtreg_home` flag seems to have been dropped by accident from the test command. I've re-added it. Also renamed `libclang_dir` -> `llvm_dir` which was missed in an earlier change. ------------- Commit messages: - libclang_dir -> llvm_dir in instructions - Use java.home instead of JAVA_HOME - reorder build instructions - Add missing jtreg_home property to test command - Use JDK 18 from Java home, and add a JDK version check Changes: https://git.openjdk.java.net/jextract/pull/9/files Webrev: https://webrevs.openjdk.java.net/?repo=jextract&pr=9&range=00 Issue: https://bugs.openjdk.java.net/browse/CODETOOLS-7903136 Stats: 43 lines in 2 files changed: 35 ins; 3 del; 5 mod Patch: https://git.openjdk.java.net/jextract/pull/9.diff Fetch: git fetch https://git.openjdk.java.net/jextract pull/9/head:pull/9 PR: https://git.openjdk.java.net/jextract/pull/9

4 21

RFR: 7903132: Replace casts with type test pattern
by Ethan McCue 28 Mar '22

28 Mar '22

Replaces a handful of casts following type tests with the equivalent pattern syntax. I cannot create a matching issue, as I do not have an account on the bug tracker. ------------- Commit messages: - Replace casts with type test pattern Changes: https://git.openjdk.java.net/jextract/pull/5/files Webrev: https://webrevs.openjdk.java.net/?repo=jextract&pr=5&range=00 Issue: https://bugs.openjdk.java.net/browse/CODETOOLS-7903132 Stats: 81 lines in 13 files changed: 2 ins; 22 del; 57 mod Patch: https://git.openjdk.java.net/jextract/pull/5.diff Fetch: git fetch https://git.openjdk.java.net/jextract pull/5/head:pull/5 PR: https://git.openjdk.java.net/jextract/pull/5

3 14

RFR: 7903133: add lp_solve sample
by Athijegannathan Sundararajan 28 Mar '22

28 Mar '22

Adding lp_solve jextract sample. This is more or less straight port of a C example. ------------- Commit messages: - removed trailing whitespaces in LpSolveDemo.java - 7903133: add lp_solve sample Changes: https://git.openjdk.java.net/jextract/pull/7/files Webrev: https://webrevs.openjdk.java.net/?repo=jextract&pr=7&range=00 Issue: https://bugs.openjdk.java.net/browse/CODETOOLS-7903133 Stats: 166 lines in 7 files changed: 166 ins; 0 del; 0 mod Patch: https://git.openjdk.java.net/jextract/pull/7.diff Fetch: git fetch https://git.openjdk.java.net/jextract pull/7/head:pull/7 PR: https://git.openjdk.java.net/jextract/pull/7

2 2

RFR: 7903134: Imrpove error reporting in jextract gradle build
by Jorn Vernee 28 Mar '22

28 Mar '22

Improve error reporting around missing LLVM paths in jextract gradle build. This patch tries to detect missing paths and throw a descriptive error, instead of a more cryptic one. I've also added the `.idea` folder to the `.gitignore` file. ------------- Commit messages: - ignore .idea folder - Improve error reporting around missing LLVM paths in the gradle build. Changes: https://git.openjdk.java.net/jextract/pull/8/files Webrev: https://webrevs.openjdk.java.net/?repo=jextract&pr=8&range=00 Issue: https://bugs.openjdk.java.net/browse/CODETOOLS-7903134 Stats: 19 lines in 2 files changed: 18 ins; 0 del; 1 mod Patch: https://git.openjdk.java.net/jextract/pull/8.diff Fetch: git fetch https://git.openjdk.java.net/jextract pull/8/head:pull/8 PR: https://git.openjdk.java.net/jextract/pull/8

2 5

RFR: 7903129: jextract does not handle @argfile
by Athijegannathan Sundararajan 25 Mar '22

25 Mar '22

Missed CommandLine.parse call in JextractTool.run method. ------------- Commit messages: - 7903129: jextract does not handle @argfile Changes: https://git.openjdk.java.net/jextract/pull/4/files Webrev: https://webrevs.openjdk.java.net/?repo=jextract&pr=4&range=00 Issue: https://bugs.openjdk.java.net/browse/CODETOOLS-7903129 Stats: 13 lines in 2 files changed: 13 ins; 0 del; 0 mod Patch: https://git.openjdk.java.net/jextract/pull/4.diff Fetch: git fetch https://git.openjdk.java.net/jextract pull/4/head:pull/4 PR: https://git.openjdk.java.net/jextract/pull/4

3 8

RFR: 7903128: Overhaul jextract README
by Maurizio Cimadamore 24 Mar '22

24 Mar '22

I've improved the README file, by putting the expanding the build section, and putting it more front and center. I have also streamlined the remaining contents a little, and removed the parts that were describing the foreign API (not really needed here). ------------- Commit messages: - Overhaul README Changes: https://git.openjdk.java.net/jextract/pull/3/files Webrev: https://webrevs.openjdk.java.net/?repo=jextract&pr=3&range=00 Issue: https://bugs.openjdk.java.net/browse/CODETOOLS-7903128 Stats: 116 lines in 1 file changed: 28 ins; 68 del; 20 mod Patch: https://git.openjdk.java.net/jextract/pull/3.diff Fetch: git fetch https://git.openjdk.java.net/jextract pull/3/head:pull/3 PR: https://git.openjdk.java.net/jextract/pull/3

3 16

RFR: 7903121: initial jextract tool code, tests and samples
by Athijegannathan Sundararajan 23 Mar '22

23 Mar '22

Adding initial standalone jextract code, tests and samples. ------------- Commit messages: - 7903121: initial jextract tool code, tests and samples - fixed whitespaces, tabs and executable permission. - initial commit Changes: https://git.openjdk.java.net/jextract/pull/2/files Webrev: https://webrevs.openjdk.java.net/?repo=jextract&pr=2&range=00 Issue: https://bugs.openjdk.java.net/browse/CODETOOLS-7903121 Stats: 34702 lines in 401 files changed: 34699 ins; 2 del; 1 mod Patch: https://git.openjdk.java.net/jextract/pull/2.diff Fetch: git fetch https://git.openjdk.java.net/jextract pull/2/head:pull/2 PR: https://git.openjdk.java.net/jextract/pull/2

3 9