Closed Bug 452426 Opened 16 years ago Closed 16 years ago

application.ini should supply mercurial changeset

Categories

(Firefox Build System :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: benjamin, Assigned: ted)

References

Details

Attachments

(1 file, 1 obsolete file)

Parallel to bug 407319, which was only fixed on CVS for date stamps.
Stealing, hope you don't mind!
Assignee: benjamin → ted.mielczarek
Status: NEW → ASSIGNED
Attachment #338238 - Flags: review?(benjamin)
I think this would be super helpful for lots of reasons, not least of all Talos. Thinking about other apps a bit, each could implement something similar in their own application.ini for however many repositories they need. But perhaps we could put a mozilla-central revision into platform.ini ? Then apps using XULRunner/The Platform need only specify their own repo's in application.ini.
This would help a lot when finding possible culprits for regressions. Note also bug 448155.
Comment on attachment 338238 [details] [diff] [review] put changeset id and repo URL in application.ini >+SOURCE_STAMP = $(shell hg identify -i $(topsrcdir)) 1) Use := to avoid calling out to this several times 2) Redirect stderr to /dev/null so that a tarball build (or RPM build, or whatever) doesn't show an unnecessary error message? >diff --git a/browser/app/application.ini b/browser/app/application.ini >+#ifdef MOZ_SOURCE_REPO >+SourceRepository=@MOZ_SOURCE_REPO@ >+#endif >+SourceStamp=@MOZ_SOURCE_STAMP@ Need an ifdef for SourceStamp too, in case someone's building from a tarball. r=me with those changes
Attachment #338238 - Flags: review?(benjamin) → review+
Attached patch patch for checkin (deleted) — Splinter Review
Made the changes that bsmedberg suggested, and added a little sed s/^ssh:/http:/ on the repo path, just to make that nicer. I tested this with a hg repo and with moving my .hg dir out of the way, and both ways looked good. I'm going to land this version of the patch.
Attachment #338238 - Attachment is obsolete: true
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Blocks: 494498
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: