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/4928e3b0...f5aeddd8 make/conf/jib-profiles.js line 1008: > 1006: > testOnlyProfilesPrebuiltDocs["run-test-prebuilt-docs"].dependencies.push("docs.doc_api_spec", > "tidy"); > 1007: > testOnlyProfilesPrebuiltDocs["run-test-prebuilt-docs"].environment["DOCS_IMAGE_DIR"] > = input.get("docs.doc_api_spec", "install_path"); > 1008: > testOnlyProfilesPrebuiltDocs["run-test-prebuilt-docs"].environment["TIDY_HOME"] > = input.get("tidy", "home_path") + "/bin/tidy"; This should be all you need for supplying `TIDY` to the makefiles when running tests without configure. Suggestion: testOnlyProfilesPrebuiltDocs["run-test-prebuilt-docs"].environment["TIDY"] = input.get("tidy", "home_path") + "/bin/tidy"; ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/21272#discussion_r1801806290