Skip to content

HelloWorld sample source maps support #10

Open

Description

@axls

Does the current version of j2cl support sourcemaps?

Chrome's DevTool doesn't show HelloWorld.java in sources list. Also I see next log output in terminal when launching dev server through ibazel:

INFO: 750µs /0:0:0:0:0:0:0:1:38796 404 GET /HelloWorld.js.map

It seems like reference to the sources presents in js, but the web server cannot find requested resource.

Activity

  1. jDramaix commented on Nov 15, 2018

    @jDramaix
    Member

    Yes, J2CL has sourcemaps support. The web server used in the example is serving correctly the file. We will fix that.

  2. self-assigned this
    on Nov 15, 2018
  3. gkdn commented on Nov 16, 2018

    @gkdn
    Member

    @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.

  4. gkdn commented on Nov 8, 2019

    @gkdn
    Member

    As an update; we are actively working switching from zip files to TreeArtifacts so after that this should be easier to address.

  5. niloc132 commented on Nov 8, 2019

    @niloc132
    Contributor

    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.

  6. treblereel commented on Oct 24, 2022

    @treblereel
    Contributor

    @gkdn maybe you have any updates? Thanks

  7. niloc132 commented on Oct 24, 2022

    @niloc132
    Contributor

    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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions