Closed
Bug 587541
Opened 14 years ago
Closed 14 years ago
Stop trying to package FormNotifier.js, since it doesn't exist
Categories
(Firefox Build System :: General, defect)
Firefox Build System
General
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla4.0b5
People
(Reporter: philor, Assigned: philor)
References
Details
Attachments
(1 file)
(deleted),
patch
|
khuey
:
review+
Gavin
:
approval2.0+
|
Details | Diff | Splinter Review |
Well, strictly speaking I think it still exists in Weave-as-an-extension, but it doesn't exist in dist/bin/components since bug 487558 landed.
Attachment #466195 -
Flags: review?(me)
Comment on attachment 466195 [details] [diff] [review]
rm
I should hurry up and make packager warnings fatal.
Attachment #466195 -
Flags: review?(me) → review+
Assignee | ||
Comment 2•14 years ago
|
||
Problem is, the ones you can easily make fatal are the uninteresting ones - I can't remember the last time someone accidentally made a file disappear, but I can easily remember the last time someone forgot to package something that did exist. This mostly matters because Nick's going to see it while checking packaging for the next beta, and will have to make sure it was okay that it didn't exist.
Comment 3•14 years ago
|
||
I dunno what we can do about the general problem, except educate developers that they should be calling 'make package', or testing their builds from try, rather than using dist/bin.
Assignee | ||
Comment 4•14 years ago
|
||
Persuade them to also build --disable-tests, so the output of make package-compare is close to usable; teach package-compare about the things it shouldn't mind seeing, and make everything else fatal; move everything we don't want to ship out of dist/bin and package everything. Nothing short-range I know of, other than me building more often on more platforms, since my build scripts do finish with make package-compare.
Assignee | ||
Updated•14 years ago
|
Attachment #466195 -
Flags: approval2.0?
Comment 5•14 years ago
|
||
(In reply to comment #4)
> move everything we don't
> want to ship out of dist/bin and package everything.
Yes, please.
Updated•14 years ago
|
Attachment #466195 -
Flags: approval2.0? → approval2.0+
Assignee | ||
Comment 6•14 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → Firefox 4.0b5
Updated•6 years ago
|
Component: Build Config → General
Product: Firefox → Firefox Build System
Updated•6 years ago
|
Target Milestone: Firefox 4.0b5 → mozilla4.0b5
You need to log in
before you can comment on or make changes to this bug.
Description
•