deps does not exist

[justinbmeyer]

If you have a meta with an init function, it breaks because deps does not exist here:

https://github.com/systemjs/systemjs/blob/master/lib/extension-global.js#L67

You can see that deps is not in the scope.

[justinbmeyer]

cc @matthewp

[guybedford]

Ok yes that is a bug. PRs welcome.

Note that the init function will be removed in the next major version though.

[justinbmeyer]

Why will it be removed? It's super useful.

Sent from my iPhone

On Oct 29, 2014, at 3:58 AM, Guy Bedford notifications@github.com wrote:

Ok yes that is a bug. PRs welcome.

Note that the init function will be removed in the next major version though.

—
Reply to this email directly or view it on GitHub.

[guybedford]

I think its use cases are replaced by #190.

[justinbmeyer]

Globals can depend on non-globals. For example jQueryUI might depend on jQuery.

Also, without init, I am unable to deal with "document ready" type problems. For example, it's common to want to defer both jQuery's ready event and QUnit from running until all modules have been loaded. This is done in RequireJS's init.

Sent from my iPhone

On Oct 30, 2014, at 7:38 AM, Guy Bedford notifications@github.com wrote:

I think its use cases are replaced by #190.

—
Reply to this email directly or view it on GitHub.

[justinbmeyer]

Also, we've been trying to convince people to upgrade from RequireJS to steal/system. The more similar the config, the easier it is.

Sent from my iPhone

On Oct 30, 2014, at 7:38 AM, Guy Bedford notifications@github.com wrote:

I think its use cases are replaced by #190.

—
Reply to this email directly or view it on GitHub.

[guybedford]

I'm keen to get deps working for all module formats in the next release.

[guybedford]

Finally fixed.