On Tue, 15 Oct 2024 18:45:37 GMT, Nizar Benalla <nbena...@openjdk.org> wrote:
>> Please review this change that adds a new test root `docs` dedicated to >> testing the documentation, which has been a work in progress for a while. >> Tests for links, encoding, HTML, accessibility will be later added in >> following PRs. >> >> We also define a new make target `test-docs` meant for local use and depends >> on the docs. >> This also adds the necessary configurations needed at Oracle. >> >> This patch includes a test `TestDocs` which serves to show developers how >> they are meant to resolve the docs to test them, I want to include it >> temporarily until better tests are added later. >> >> TIA > > Nizar Benalla has updated the pull request with a new target base due to a > merge or a rebase. The incremental webrev excludes the unrelated changes > brought in by the merge/rebase. The pull request contains 11 additional > commits since the last revision: > > - - add explicit env declaration to `run-test-prebuilt-docs` profile. > > - add TEST-DEPS solution > - Merge branch 'master' into 8340818-jtreg-test-root > - remove headers > - - Add tidy as a dependency to `run-test-prebuilt-docs` profile > - remove deprecated syntax in Main.gmk > - add (C) > - drop new test-docs target, left for future PR > - drop list of keys in TEST.ROOT > - Revert "Add tidy as a dependency to `run-test-prebuilt-docs` and `docs` > profiles" > > This reverts commit f706a420b86196e56683eac6f37818edf97a7770. > - Add tidy as a dependency to `run-test-prebuilt-docs` and `docs` profiles > - - Add code block that was accidentally removed. > - Simplify code > - typos - remove a few mentions of doccheck > - update after review comments > - (C) > - ... and 1 more: https://git.openjdk.org/jdk/compare/a24256cd...f5aeddd8 make/RunTestsPrebuiltSpec.gmk line 71: > 69: BUILD_JAVA_FLAGS := $(JAVA_FLAGS_BIG) > 70: > 71: # Express that testing needs additional dependencies You don't need to set this. In fact, it makes it impossible to set it from the command line. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/21272#discussion_r1802513671
