Closed
Bug 807756
Opened 12 years ago
Closed 12 years ago
Bad path for Services.jsm in DebuggerUI.jsm
Categories
(DevTools :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 19
People
(Reporter: jimm, Assigned: jimm)
References
Details
Attachments
(1 file, 1 obsolete file)
|
(deleted),
patch
|
Gavin
:
review+
|
Details | Diff | Splinter Review |
|
Assignee | |
Comment 1•12 years ago
|
||
Assignee: nobody → jmathies
|
|
Assignee | |
Updated•12 years ago
|
Attachment #677514 -
Flags: review?(rcampbell)
|
|
Assignee | |
Comment 2•12 years ago
|
||
Hmm, I think FileUtils is wrong as well.
|
|
Assignee | |
Comment 3•12 years ago
|
||
Attachment #677514 -
Attachment is obsolete: true
Attachment #677514 -
Flags: review?(rcampbell)
Attachment #677516 -
Flags: review?(rcampbell)
|
|
Assignee | |
Updated•12 years ago
|
Attachment #677516 -
Attachment is patch: true
|
||
Updated•12 years ago
|
Attachment #677516 -
Flags: review?(rcampbell) → review+
|
|
Assignee | |
Comment 4•12 years ago
|
||
|
||
Comment 5•12 years ago
|
||
Status: NEW → RESOLVED
Closed: 12 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → Firefox 19
|
||
Comment 6•12 years ago
|
||
(In reply to Jim Mathies [:jimm] from comment #3)
> Created attachment 677516 [details] [diff] [review]
> fix
Thanks for noticing and fixing this. I wonder why it worked without before though. Is the //modules path aliased by /gre/modules?
|
||
Comment 7•12 years ago
|
||
(In reply to Victor Porof [:vp] from comment #6)
> (In reply to Jim Mathies [:jimm] from comment #3)
> > Created attachment 677516 [details] [diff] [review]
> > fix
>
> Thanks for noticing and fixing this. I wonder why it worked without before
> though. Is the //modules path aliased by /gre/modules?
It is in Firefox, but not necessarily in other embeddings.
|
|
Assignee | |
Comment 8•12 years ago
|
||
(In reply to Victor Porof [:vp] from comment #6)
> (In reply to Jim Mathies [:jimm] from comment #3)
> > Created attachment 677516 [details] [diff] [review]
> > fix
>
> Thanks for noticing and fixing this. I wonder why it worked without before
> though. Is the //modules path aliased by /gre/modules?
Currently platform and app resources end up in the same dist/bin directory for fx builds. This will change with the work in bug 755724. The metro repo on Elm has bug 755724's experimental work in it, so when bug 707302 merged over some tests broke. Hence the filing of this bug.
|
||
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•