Closed
Bug 663706
Opened 13 years ago
Closed 13 years ago
Allow require with file extensions
Categories
(Add-on SDK Graveyard :: General, defect, P2)
Add-on SDK Graveyard
General
Tracking
(Not tracked)
RESOLVED
FIXED
Future
People
(Reporter: irakli, Assigned: irakli)
Details
Attachments
(1 file)
I run into some packages that require modules via string containing `.js` file extension: require('foo/bar.js') which works perfectly fine in other platforms like node.js but in jetpack it fails as .js is appended blindly. Would be very nice to fix this.
Assignee | ||
Comment 1•13 years ago
|
||
Pointer to Github pull-request
Assignee | ||
Comment 2•13 years ago
|
||
Comment on attachment 538764 [details] Pointer to Github pull request: https://github.com/mozilla/addon-sdk/pull/188 Warner can you take a look at this ?
Attachment #538764 -
Flags: review?(warner-bugzilla)
Assignee | ||
Updated•13 years ago
|
Assignee: nobody → rFobic
Updated•13 years ago
|
OS: Mac OS X → All
Priority: -- → P2
Hardware: x86 → All
Target Milestone: --- → Future
Comment 3•13 years ago
|
||
Comment on attachment 538764 [details] Pointer to Github pull request: https://github.com/mozilla/addon-sdk/pull/188 current version of the pull request (with the docs.md fix) looks ok to me.
Attachment #538764 -
Flags: review?(warner-bugzilla) → review+
Assignee | ||
Comment 4•13 years ago
|
||
https://github.com/mozilla/addon-sdk/commit/3b972447b699dab4d1f3033c61f934e3fb561027
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•