archives

« Bugzilla Issues Index

#3293 — 8.6.1 HostNormalizeModuleName needs parentName

• bug_id: 3293
• creation_ts: 2014-10-19 10:48:00 -0700
• short_desc: 8.6.1 HostNormalizeModuleName needs parentName
• delta_ts: 2015-07-10 08:35:02 -0700
• product: Draft for 6th Edition
• component: Modules
• version: Rev 28: October 14, 2014 Draft
• rep_platform: All
• op_sys: All
• bug_status: RESOLVED
• resolution: FIXED
• priority: Normal
• bug_severity: critical
• everconfirmed: true
• reporter: Guy Bedford
• assigned_to: Allen Wirfs-Brock
• cc: ["dherman", "erik.arvidsson", "jorendorff", "samth"]

---

• commentid: 10525
• comment_count: 0
• who: Guy Bedford
• bug_when: 2014-10-19 10:48:59 -0700

Modules have relative normalization:

app/main.js:
import './dep.js';

Should import "app/dep.js" not "dep.js" at the root-level.

We need to know the normalized name of the parent when normalizing in order to get this functionality:

HostNormalizeModuleName(unnormalizedName, parentNormalizedName)

---

• commentid: 10530
• comment_count: 1
• who: Erik Arvidsson
• bug_when: 2014-10-20 07:23:04 -0700

This is critical

---

• commentid: 10531
• comment_count: 2
• who: Allen Wirfs-Brock
• bug_when: 2014-10-20 09:19:33 -0700

(In reply to Guy Bedford from comment #0)
> Modules have relative normalization:
>
> app/main.js:
> import './dep.js';
>
> Should import "app/dep.js" not "dep.js" at the root-level.

Actually, this is host defined and not specified in the ES spec.

>
> We need to know the normalized name of the parent when normalizing in order
> to get this functionality:
>
> HostNormalizeModuleName(unnormalizedName, parentNormalizedName)

But I agree this hook is need to enable the host to define such an interpretation.

Fixed in rev29 editor's draft

Also need to updated the signature of NormalizeModuleName

---

• commentid: 10863
• comment_count: 3
• who: Allen Wirfs-Brock
• bug_when: 2014-12-07 14:35:02 -0800

fixed in rev29