Closed Bug 901898 Opened 7 years ago Closed 7 years ago

Codesighs is never run by buildbot, but is still enabled in mozconfigs

Categories

(Firefox Build System :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
mozilla26

People

(Reporter: standard8, Assigned: emorley)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Codesighs got removed from buildbot as part of bug 803736. However, its still enabled in the mozconfig files, and still being built.

If we're not using it, we should remove the option from the mozconfig files, and maybe even remove its code as well (similar to bug 823915).
Attached patch Patch v1Splinter Review
Attachment #787065 - Flags: review?(gps)
Assignee: nobody → emorley
Status: NEW → ASSIGNED
No longer blocks: 823915
Duplicate of this bug: 823915
Comment on attachment 787065 [details] [diff] [review]
Patch v1

Review of attachment 787065 [details] [diff] [review]:
-----------------------------------------------------------------

I don't know the full history of codesighs and don't know if it is appropriate to remove all of its code. We may no longer run codesighs on automation. However, automation is only one consumer of the tree. Is anyone else running codesighs? Does it deserve to die regardless?

Anyway, I just asked that question in #build and Ms2ger says it can die. That's enough of a second opinion for me, so, r+.
Attachment #787065 - Flags: review?(gps) → review+
https://hg.mozilla.org/mozilla-central/rev/2654301601dd
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla26
FWIW, codesighs's advantages over the tool in bug 659950 are:

 * it shows useful diffs explaining what changed

 * it adds up the data prior to some things that cause quantization, i.e., parts of the library having their sizes rounded up to some rounder size (although I'm not sure how big of an effect that is these days)
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.