Closed Bug 665978 Opened 13 years ago Closed 13 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: 13 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: