closure-compiler icon indicating copy to clipboard operation
closure-compiler copied to clipboard

Include LICENSE.txt in the released jar files

Open brad4d opened this issue 2 years ago • 2 comments

I've created this as a pull request instead of a normal Google-internal change because it was easier to work on the OSS build rules from a git repository.

brad4d avatar Feb 02 '24 00:02 brad4d

I think the title might have a typo - if it is "unshaded", then it should not include dependencies, by including dependencies, the jar then would be shaded.

Of course there is also the distinction between "the shaded classes are rebased/repackaged" and just merging all of the jars, but it would hurt our use cases if the "unshaded" jars must then contain dependencies instead of relying on dependency management tools to look them up as needed.

niloc132 avatar Feb 02 '24 01:02 niloc132

I've just pushed a new version of this fix that I think is more satisfactory.

brad4d avatar Feb 03 '24 01:02 brad4d

@gkdn fixed the underlying problem that this hacky PR was trying to fix, so it is no longer necessary.

brad4d avatar Mar 06 '24 00:03 brad4d