Closed
Bug 658664
Opened 13 years ago
Closed 13 years ago
stop recommending arrays-of-paths for package.json 'lib' and 'tests'
Categories
(Add-on SDK Graveyard :: Documentation, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
1.0
People
(Reporter: warner, Assigned: warner)
Details
Attachments
(1 file)
|
1.15 KB,
patch
|
myk
:
review+
|
Details | Diff | Splinter Review |
Our package.json syntax permits some fields (specifically 'lib') to contain lists of pathnames, not just a single one. But the recent linker changes I made probably broke the multi-path case, because many of the URI-mapping tools assume that there is just one path in this array. I don't think anyone was using this feature, and it's pretty obscure, so I'd like to remove it. At the very least, I'd like to change the docs to stop claiming that it can tolerate an array, so we aren't teaching users to do something that might not work in the future.
|
Assignee | |
Comment 1•13 years ago
|
||
Attachment #534115 -
Flags: review?(myk)
|
|
Assignee | |
Updated•13 years ago
|
Assignee: nobody → warner-bugzilla
|
||
Updated•13 years ago
|
Attachment #534115 -
Flags: review?(myk) → review+
|
|
Assignee | |
Comment 2•13 years ago
|
||
landed in https://github.com/mozilla/addon-sdk/commit/d07233646ccaac5b2513ebb92687d31b5a509b0a
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
•