Closed Bug 818186 Opened 12 years ago Closed 10 years ago

Some loader bugs

Categories

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

Product:
Add-on SDK Graveyard
Component:
General
Platform:
x86
macOS
Type:
defect

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: glind, Assigned: irakli)

Details

1.  Have a "" mapping by default in the mappings, so that all paths resolve.
2.  Fix resolve to this, so that 'bare imports' 'do the right thing':

let resolve1 = function(id, requirer) {
    console.log("R1",id,requirer);
    let ans = id[0] === "." ? resolve(id, requirer) : id
    console.log('R1 answer:',ans);
    return ans
}
Assignee: nobody → rFobic
Gregg, I'm not sure what this solves, or if it's quite outdated (we do have "" mappings now), do you have a use-case in mind? Is this still valid?
Flags: needinfo?(glind)
I believe that was fixed in 2013 in this rev:

commit a7ed5c079a876e48c6ff4432811afee385c64864
Author: Alexandre Poirot <poirot.alex@gmail.com>
Date:   Tue Feb 19 16:48:42 2013 +0100

    Bug 793925: part 8 - Fix handling of relative path for system/out-of-manifest modules

--> +  if (!isRelative(id)) return id;
Status: NEW → RESOLVED
Closed: 10 years ago
Flags: needinfo?(glind)
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.