[GitHub] csantanapr commented on issue #2243: Compile Swift (& Java?) binaries once

[gi...@git.apache.org]

csantanapr commented on issue #2243: Compile Swift (& Java?) binaries once
URL: https://github.com/apache/incubator-openwhisk/issues/2243#issuecomment-306339913
 
 
   Yeah I think it would be a package instead, with an action per runtime, since the action would need the runtime anyway to compile, might as well be in the target language
   ```
   whisk.system/compile/nodejs
   whisk.system/compile/java
   whisk.system/compile/swift
   whisk.system/compile/python
   ```
   
   There is no need for parameter kind
   
   I would make the action accept two modes.
   Give it a git info to get the source code from, or a zip
   
   @pritidesai 
   I don't follow why you do a `npm install` to create the zip to send to the action, that's what the action is suppose to do so any binaries from npm install are done in the correct runtime.
   
   There should be specific parameters for each kind.
   For example for nodejs, I might now that there is no binaries and is all pure js, so using webpack or something similar should be allow, maybe allow for a custom npm script like instead of npm install and zip you could run a npm run script.
   
   
 
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services