stealjs / steal

Gets JavaScript
https://stealjs.com
MIT License
1.37k stars 521 forks source link

package.json parse jspm field #550

Closed wclr closed 8 years ago

wclr commented 8 years ago

Also this could be parsed:

"jspm": {
  "main": "socket.io.js",
}

https://github.com/socketio/socket.io-client/blob/master/package.json#L14

matthewp commented 8 years ago

There's an issue in system-npm: https://github.com/stealjs/system-npm/issues/34

wclr commented 8 years ago

You could currently add support for steal, as some people use this, like in soket.io-client case

matthewp commented 8 years ago

Not sure what you mean. That issue is to add it to Steal. Should be easier to do.

wclr commented 8 years ago

I mean maybe to parse this field if it in package.json and take its main instead of package.json main.

matthewp commented 8 years ago

Yes, that is how it would work. This the same thing with system.main in the package.json.

wclr commented 8 years ago

So this doesn't wok now. Steal does not consider it.

matthewp commented 8 years ago

I know, that's why https://github.com/stealjs/system-npm/issues/34 exists.

wclr commented 8 years ago

Why is this responsiblity of SystemJs?

matthewp commented 8 years ago

It's not, system-npm is the plugin we use to load dependencies from node_modules, it's Steal's plugin, we control it. We just develop it in a separate project.

wclr commented 8 years ago

Ah sorry I din't notice that it is system-npm)