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/fd057cbd...f5aeddd8 With the latest patch, running `make test TEST=docs_all TEST_DEPS=docs-jdk` works as intended but ` make test-docs_all TEST_DEPS=docs-jdk` does not. I see that there is a function `DeclareRunTestRecipe` used to add prefixes, but all `test-$1` targets have the same two dependencies so it might be difficult to add custom dependencies when using that syntax ` $(foreach t, $(filter-out test-make%, $(ALL_TEST_TARGETS)), $(eval $t: jdk-image test-image))` ------------- PR Comment: https://git.openjdk.org/jdk/pull/21272#issuecomment-2416776000
