Vertispan / j2clmavenplugin

Maven plugin to launch new J2CL compilation
https://vertispan.github.io/j2clmavenplugin/
Apache License 2.0
53 stars 26 forks source link

Remove jszip config arg, set it up as a dependency instead #173

Closed niloc132 closed 2 years ago

niloc132 commented 2 years ago

Some closure features like typedast do not work with jszips. This patch mostly treats jszips as any normal jar, except it excludes them from compilation via javac/j2cl. This is necessary to do since the jre.js.zip from j2cl itself includes .java sources (for the sake of sourcemaps), yet is missing .native.js files, so fails to compile, and potentially wouldn't yield the correct results anyway.

The config property for jszips has been removed, but they can still be set via maven configuration, which will create extra project/dependency setups for them instead of passing just a path to the archive.

Debug logging for the complete structure of a project's dependencies also gets some cleanup, to avoid other logging messing up the tree.