> On May 19, 2017, at 1:08 AM, Magnus Ihse Bursie > <magnus.ihse.bur...@oracle.com> wrote: > > > "Indirect exports" was a good term. I dropped the "Deps" part of the name and > it all became much clearer. > > Here's an updated webrev: > http://cr.openjdk.java.net/~ihse/JDK-8180480-docs-should-use-requires-transitive/webrev.02 >
+ $1_INDIRECT_EXPORTS := $$(call FindTransitiveIndirectExportsForModules, $$($1_MODULES)) Nit: “exported” term is for API but not for dependency. s/$1_INDIRECT_EXPORTS/$1_TRANSITIVE_MODULES s/FindTransitiveIndirectExportsForModules/FindTransitiveIndirectDepsForModules Otherwise, looks good. No need for a new webrev: Mandy