Closed
Bug 794393
Opened 12 years ago
Closed 12 years ago
Remove package-xulrunner rules
Categories
(Firefox Build System :: General, defect)
Firefox Build System
General
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla18
People
(Reporter: glandium, Assigned: glandium)
References
Details
Attachments
(1 file)
AFAICT, they are unused, and a remainder of the old days when fennec was a xulrunner app.
|
Assignee | |
Comment 1•12 years ago
|
||
Attachment #664850 -
Flags: review?(mwu)
Attachment #664850 -
Flags: review?(blassey.bugs)
|
||
Comment 2•12 years ago
|
||
Comment on attachment 664850 [details] [diff] [review]
Remove package-xulrunner rules
Review of attachment 664850 [details] [diff] [review]:
-----------------------------------------------------------------
Oleg, is this needed for anything nokia is doing?
Attachment #664850 -
Flags: review?(romaxa)
|
|
||
Updated•12 years ago
|
Attachment #664850 -
Flags: review?(blassey.bugs) → review+
|
||
Comment 3•12 years ago
|
||
Comment on attachment 664850 [details] [diff] [review]
Remove package-xulrunner rules
I don't know this code..
Attachment #664850 -
Flags: review?(mwu)
|
|
Assignee | |
Comment 4•12 years ago
|
||
(In reply to Michael Wu [:mwu] from comment #3)
> Comment on attachment 664850 [details] [diff] [review]
> Remove package-xulrunner rules
>
> I don't know this code..
I guess this means b2g doesn't use it, right? (and it's only there because it was copied from mobile)
|
|
||
Comment 5•12 years ago
|
||
(In reply to Mike Hommey [:glandium] from comment #4)
> (In reply to Michael Wu [:mwu] from comment #3)
> > Comment on attachment 664850 [details] [diff] [review]
> > Remove package-xulrunner rules
> >
> > I don't know this code..
>
> I guess this means b2g doesn't use it, right? (and it's only there because
> it was copied from mobile)
Oh yeah, definitely. We don't use xulrunner at all. Lemme rubberstamp that part at least.
|
|
||
Comment 6•12 years ago
|
||
Comment on attachment 664850 [details] [diff] [review]
Remove package-xulrunner rules
r=me on the b2g part.
Attachment #664850 -
Flags: review+
|
||
Comment 7•12 years ago
|
||
> Oleg, is this needed for anything nokia is doing?
It supposed to be used for packaging xulrunner into debian N9 environment, and build some xulrunner apps on top of it, but not anymore..
I saw xulrunner package in debian distro... is that packaged differently?
|
|
||
Comment 8•12 years ago
|
||
Comment on attachment 664850 [details] [diff] [review]
Remove package-xulrunner rules
r=me, hope it will not break XUL fennec debian packaging accidentally
Attachment #664850 -
Flags: review?(romaxa) → review+
|
|
Assignee | |
Comment 9•12 years ago
|
||
|
||
Comment 10•12 years ago
|
||
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla18
|
||
Updated•7 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•