Closed Bug 665978 Opened 14 years ago Closed 14 years ago

Delete old .gcda / .pgd files during NSPR PROFILE_GENERATE phase

Categories

(NSPR :: NSPR, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: justin.lebar+bug, Assigned: justin.lebar+bug)

References

Details

(Whiteboard: [fixed in nspr])

Attachments

(1 file)

Bug 659942 fixed this for everywhere save NSPR, but we should also fix it for NSPR, even though I doubt that fixing this bug will fix bug 653961. Patch in a moment.
Attached patch Patch v1Splinter Review
This is kind of a hack; it always deletes both .gcda and .pgd files, because I wasn't sure how to properly detect GCC (GNU_CC is not defined). I don't think that's a big deal, but I'm happy to change it.
Attachment #540823 - Flags: review?(ted.mielczarek)
Assignee: wtc → justin.lebar+bug
OS: Linux → All
Hardware: x86_64 → All
Comment on attachment 540823 [details] [diff] [review] Patch v1 Review of attachment 540823 [details] [diff] [review]: ----------------------------------------------------------------- ::: nsprpub/config/rules.mk @@ +382,5 @@ > +# Clean up profiling data during PROFILE_GENERATE phase > +export:: > + -$(RM) *.pgd *.gcda > +endif > + This should be fine, since $(RM) is "rm -f", so removing things that aren't there isn't harmful.
Attachment #540823 - Flags: review?(ted.mielczarek) → review+
Keywords: checkin-needed
Whiteboard: [needs checkin to NSPR]
Checking in config/rules.mk; /cvsroot/mozilla/nsprpub/config/rules.mk,v <-- rules.mk new revision: 3.81; previous revision: 3.80 done
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Target Milestone: --- → 4.9
Target Milestone: 4.9 → 4.8.9
Keywords: checkin-needed
Whiteboard: [needs checkin to NSPR] → [fixed in nspr]
Depends on: 675084
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: