You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
@jDramaix making source map work is not a small project. web_library in rules_closure needs to understand and serve files from zip files first.
Let's leave this to contributors for now. If nobody takes care of it by the time and eventually that part of the problem will go away with the tree artifacts.
Reacted by Julien Dramaix and Sergio Triana Escobedo
Worth pointing out perhaps that closure-compiler's BUNDLE mode presently doesn't handle sourcemaps at all, see google/closure-compiler#3247. In that issue, I link the patch we're using in our closure-compiler fork so that j2cl-maven-plugin can provide sourcemaps in dev mode. This is not a perfect solution, but probably will suffice until BUNDLE gets to be more efficient or is replaced in the maven plugin.
For what its worth, with only very minor changes, our patch to closure-compiler has cleanly rebased over the last several years so that this always works in maven/gradle. Bazel adoption seems light, due at least to this and j2cl unit tests still being no-ops.
Activity
jDramaix commentedon Nov 15, 2018
Yes, J2CL has sourcemaps support. The web server used in the example is serving correctly the file. We will fix that.
gkdn commentedon Nov 16, 2018
@jDramaix making source map work is not a small project. web_library in rules_closure needs to understand and serve files from zip files first.
Let's leave this to contributors for now. If nobody takes care of it by the time and eventually that part of the problem will go away with the tree artifacts.
gkdn commentedon Nov 8, 2019
As an update; we are actively working switching from zip files to TreeArtifacts so after that this should be easier to address.
niloc132 commentedon Nov 8, 2019
Worth pointing out perhaps that closure-compiler's BUNDLE mode presently doesn't handle sourcemaps at all, see google/closure-compiler#3247. In that issue, I link the patch we're using in our closure-compiler fork so that j2cl-maven-plugin can provide sourcemaps in dev mode. This is not a perfect solution, but probably will suffice until BUNDLE gets to be more efficient or is replaced in the maven plugin.
treblereel commentedon Oct 24, 2022
@gkdn maybe you have any updates? Thanks
niloc132 commentedon Oct 24, 2022
For what its worth, with only very minor changes, our patch to closure-compiler has cleanly rebased over the last several years so that this always works in maven/gradle. Bazel adoption seems light, due at least to this and j2cl unit tests still being no-ops.