Skip to content

Add Closure compiler to emsdk.#403

Closed
juj wants to merge 1 commit intomasterfrom
closure_compiler
Closed

Add Closure compiler to emsdk.#403
juj wants to merge 1 commit intomasterfrom
closure_compiler

Conversation

@juj
Copy link
Collaborator

@juj juj commented Dec 7, 2019

Also drop Java.

To land this, please upload http://clb.confined.space/dump/google-closure-compiler-20191111.zip to Google emsdk hosting. The zip was generated via instructions by @dcodeIO at emscripten-core/emscripten#9962 (comment) .

@sbc100
Copy link
Collaborator

sbc100 commented Dec 7, 2019

What do you think about instead just adding to the package.json file at the top level of emscripten? Then we can just run npm install as a post install step and it will pull down whatever we need? This puts emscripten in control of its own dependencies.

And for non-emsdk-users, we just say run npm install ..

@juj
Copy link
Collaborator Author

juj commented Dec 7, 2019

That would work as well. I'll re-do the PR in that form tomorrow.

@juj
Copy link
Collaborator Author

juj commented Dec 8, 2019

Superceded by #404.

@juj juj closed this Dec 8, 2019
@sbc100 sbc100 deleted the closure_compiler branch March 25, 2021 15:23
vargaz pushed a commit to vargaz/emsdk that referenced this pull request Nov 22, 2023
emscripten-core#403)

* Update dependencies from https://github.com/dotnet/source-build-reference-packages build 20230807.1

Microsoft.SourceBuild.Intermediate.source-build-reference-packages
 From Version 8.0.0-alpha.1.23381.3 -> To Version 8.0.0-alpha.1.23407.1

* Update dependencies from https://github.com/dotnet/source-build-reference-packages build 20230808.2

Microsoft.SourceBuild.Intermediate.source-build-reference-packages
 From Version 8.0.0-alpha.1.23381.3 -> To Version 8.0.0-alpha.1.23408.2

---------

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants