RFR: 7903837: apidiff: build.sh issues on Windows [v3]
Christian Stein
cstein at openjdk.org
Thu Sep 26 06:31:48 UTC 2024
On Wed, 25 Sep 2024 19:52:23 GMT, Jonathan Gibbons <jjg at openjdk.org> wrote:
>> Please review some `build.sh` and `Makefile` changes to address issues building on Windows.
>>
>> The primary Windows issue was missing use of `mixed_path` when passing some filenames into `make`.
>> There is some additional cleanup (for all platforms) to normalize the `DEPS_DIR` and add missing items to the `sanity` and `check_build_vars` targets.
>>
>> This has been tested on macOS. Can someone with access to a Windows machine test it on Windows, please?
>
> Jonathan Gibbons has updated the pull request incrementally with one additional commit since the last revision:
>
> fix test classpath when DAISYDIFF_JAR is empty (for Windows)
Changes requested by cstein (Committer).
test/junit/JUnitTests.gmk line 33:
> 31:
> 32: JUnitTest.classpath = \
> 33: $(BUILDTESTDIR)/JUnitTests/classes:$(APIDIFF_IMAGEDIR)/lib/apidiff.jar:$(JAVADIFFUTILS_JAR):$(HTMLCLEANER_JAR)
Suggestion:
$(BUILDTESTDIR)/JUnitTests/classes:$(APIDIFF_IMAGEDIR)/lib/apidiff.jar:$(JAVADIFFUTILS_JAR):$(HTMLCLEANER_JAR):$(EQUINOX_JAR)
-------------
PR Review: https://git.openjdk.org/apidiff/pull/12#pullrequestreview-2330234646
PR Review Comment: https://git.openjdk.org/apidiff/pull/12#discussion_r1776456144
More information about the apidiff-dev
mailing list