Closed
Bug 541774
Opened 15 years ago
Closed 15 years ago
idl-parser cache and compiled python files are not cleaned up by make clean
Categories
(Firefox Build System :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla1.9.3a5
People
(Reporter: glandium, Assigned: glandium)
Details
Attachments
(3 files)
881 bytes,
patch
|
ted
:
review+
|
Details | Diff | Splinter Review |
499 bytes,
patch
|
ted
:
review+
|
Details | Diff | Splinter Review |
480 bytes,
patch
|
ted
:
review+
|
Details | Diff | Splinter Review |
There are .pyc leftovers in other-licenses/ply/ply and xpcom/idl-parser, as well as cache files in the latter.
The attached patch moves the cache files to another directory, and removes the .pyc files.
Assignee | ||
Updated•15 years ago
|
Attachment #423213 -
Attachment is patch: true
Attachment #423213 -
Attachment mime type: application/octet-stream → text/plain
Attachment #423213 -
Flags: review?(benjamin)
Updated•15 years ago
|
Attachment #423213 -
Flags: review?(benjamin) → review?(ted.mielczarek)
Updated•15 years ago
|
Assignee: nobody → mh+mozilla
Updated•15 years ago
|
Attachment #423213 -
Flags: review?(ted.mielczarek) → review+
Updated•15 years ago
|
Status: NEW → ASSIGNED
Keywords: checkin-needed
Assignee | ||
Comment 1•15 years ago
|
||
Additional patch, required in some cases. I haven't looked deeply where that comes from, but seamonkey builds trigger the generation of the file.
The patch needs to be applied on top of the first one.
Attachment #427335 -
Flags: review?(ted.mielczarek)
Comment 2•15 years ago
|
||
it comes from yacc, we specify it as a debugfile. And we do not override the default |debug=1| in yacc when we init that.
This file seems like it should also be present in the Firefox build too.
Updated•15 years ago
|
Attachment #427335 -
Flags: review?(ted.mielczarek) → review+
Comment 3•15 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.3a4
Comment 4•15 years ago
|
||
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 5•15 years ago
|
||
This comes on top of the 2 others and should solve the problem.
Attachment #435130 -
Flags: review?(ted.mielczarek)
Updated•15 years ago
|
Attachment #435130 -
Flags: review?(ted.mielczarek) → review+
Assignee | ||
Updated•15 years ago
|
Keywords: checkin-needed
Status: REOPENED → RESOLVED
Closed: 15 years ago → 15 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: mozilla1.9.3a4 → mozilla1.9.3a5
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
•