litegraph.js icon indicating copy to clipboard operation
litegraph.js copied to clipboard

Change global compiler.jar reference to local

Open kriffe opened this issue 7 years ago • 4 comments

Since google-closure-compiler is included in package.json as a dependency the global reference still remaining in the old python build script can be removed (if python build is still beeing used?)

https://github.com/jagenjo/litegraph.js/blob/master/utils/builder.py#L7

Should be replacable with reference to local node_modules. Needs testing first

compiler_path = "node_modules/google-closure-compiler/compiler.jar"

kriffe avatar Mar 23 '18 17:03 kriffe

all my scripts rely on that version (and I like to leave users the freedom to not use grunt if they dont want to), maybe when executing the grunt file it could add it to the system somehow so it doesnt matter where it is located.

jagenjo avatar Mar 23 '18 18:03 jagenjo

@jagenjo But if its a specific version of the compiler that is important it should be specified in the package.json as a locked down version, otherwise we might get different output on each build (https://github.com/jagenjo/litegraph.js/blob/master/package.json#L32)

Since my latest pull request i bundled grunt internally so the user wont need to install it (another devdependency.. sorry). But everything is hidden inside npm now, so you only need "npm run build" to handle the process. Or do you mean that the user should not need to use "npm install" at all in the project?

Im unsure if its possible to make global installs from npm in a safe/structured way. I think its better that you require a npm install and that your python scripts point locally (your other projects will be unaffected of that)

kriffe avatar Mar 23 '18 18:03 kriffe

The python build script not work on windows I can use npm run build, but it comes with a lot of diff, see #67 @kriffe

NateScarlet avatar Apr 24 '19 09:04 NateScarlet

the diff could be because my python script uses the /utils/deploy_files.txt files list to pack together, if the npm build uses an old copy then the included files wont be the same.

jagenjo avatar Apr 24 '19 10:04 jagenjo