Open
Bug 455940
Opened 16 years ago
Updated 2 years ago
Blow away the dist directory for top level makes
Categories
(Firefox Build System :: General, defect)
Tracking
(Not tracked)
REOPENED
People
(Reporter: sdwilsh, Unassigned)
References
Details
Attachments
(1 file)
436 bytes,
patch
|
benjamin
:
review+
|
Details | Diff | Splinter Review |
if you backout a file, the tinderboxes need to be clobbered to get rid of it, and it can cause all sorts of strange issues. It'd be nice if it just worked.
Reporter | ||
Comment 1•16 years ago
|
||
This is thankfully simple enough.
Attachment #340275 -
Flags: review?(ted.mielczarek)
Reporter | ||
Updated•16 years ago
|
Whiteboard: [has patch][needs review ted]
Reporter | ||
Updated•16 years ago
|
Attachment #340275 -
Flags: review?(ted.mielczarek) → review?(benjamin)
Reporter | ||
Comment 2•16 years ago
|
||
Comment on attachment 340275 [details] [diff] [review] v1.0 Ted thinks this would be better reviewed by bsmedgberg
Reporter | ||
Updated•16 years ago
|
Whiteboard: [has patch][needs review ted] → [has patch][needs review bsmedberg]
Comment 3•16 years ago
|
||
Comment on attachment 340275 [details] [diff] [review] v1.0 please remove all of dist/bin
Attachment #340275 -
Flags: review?(benjamin) → review+
Reporter | ||
Comment 4•16 years ago
|
||
Addressed on checkin: http://hg.mozilla.org/mozilla-central/rev/2039bfc05310
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Summary: Blow away the components directory for top level makes → Blow away the dist directory for top level makes
Whiteboard: [has patch][needs review bsmedberg]
Comment 5•16 years ago
|
||
Did you really want to remove $(DIST)/dist/bin or just $(DIST)/bin?
Addressed comment 5 in http://hg.mozilla.org/mozilla-central/rev/541fc3b73d52 since the windows builds were staying orange.
Backed out completely: http://hg.mozilla.org/mozilla-central/rev/cb3240bf1d9b since it turned the Windows boxes (builders, unit testers) red.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
For a log, see http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1222463527.1222463564.6262.gz&fulltext=1
I relanded with $(DIST)/bin/components in the hope that that will remove the file we need to remove right now to get Windows green again without having to poke at the Windows machines (and end up with color depth problems, etc., etc.): http://hg.mozilla.org/mozilla-central/rev/1c2231730ff9
Also, if we remove dist/bin, we should remove dist/Firefox.app or whatever the appropriate *.app directory is.
Comment 11•16 years ago
|
||
Shawn: are you interested in investigating/fixing this, or would you rather I took it?
Reporter | ||
Comment 12•16 years ago
|
||
It might be better for someone more knowledgeable about the build system to take this. At this point, I'd be asking you all sorts of questions to fix it... (at least to address comment 10)
Reporter | ||
Comment 13•16 years ago
|
||
(In reply to comment #11) > Shawn: are you interested in investigating/fixing this, or would you rather I > took it? I'd rather you take this ted.
Assignee: sdwilsh → ted.mielczarek
Comment 14•16 years ago
|
||
Note that blowing away $DIST/bin/components is part of the problem in bug 468285.
Blocks: 478061
Updated•13 years ago
|
Assignee: ted.mielczarek → nobody
Updated•6 years ago
|
Product: Core → Firefox Build System
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•