935109 - Loader changes to support node dependencies

Status: RESOLVED FIXED

(Add-on SDK Graveyard :: General, defect, P1)

Description

[Jordan Santell [:jsantell] [@jsantell]]

https://github.com/mozilla/addon-sdk/pull/1279

The loader change would be a change to how files are resolved when using Jetpack's require to accomodate node modules.

Comment 1

[Jordan Santell [:jsantell] [@jsantell]]

Attachment: GH PR 1291

Tests are passing, everything behind a flag, so this should be able to land to ease AOM migrations, as we optimize and continue to work behind a 'native' flag until we're ready to roll out.

Give a close eye to the components in sdk/test/ and things outside of toolkit/loader mainly

Comment 3

[Erik Vold [:erikvold] (please needinfo? me)]

(In reply to Jordan Santell [:jsantell] [@jsantell] from comment #0)
> https://github.com/mozilla/addon-sdk/pull/1279
> 
> The loader change would be a change to how files are resolved when using
> Jetpack's require to accomodate node modules.

Oops I reviewed this JEP, we should complete this and land it first.

Comment 4

[Erik Vold [:erikvold] (please needinfo? me)]

Comment on attachment 8346279 [details]
GH PR 1291

Nice work! this is awesome :)

I just mentioned some tests that should be included in the merge.

Comment 5

[Erik Vold [:erikvold] (please needinfo? me)]

We can make the // issue a separate bug.

Comment 6

[[github robot]]

Commits pushed to master at https://github.com/mozilla/addon-sdk

https://github.com/mozilla/addon-sdk/commit/3781fecb520e5c7fbe10ec2e89efd861467ef64c
Bug 935109 - Implement native loader capable of loading node-style
dependencies

https://github.com/mozilla/addon-sdk/commit/d7af5ad3e6a05bccec0bb3f370f48e9a7c8d7bae
Merge pull request #1291 from jsantell/nodeloader-imp

Bug 935109 Native Node Loader Implementation, r=@erikvold

Comment 7

[Jordan Santell [:jsantell] [@jsantell]]

Merged