Closed
Bug 322293
Opened 20 years ago
Closed 20 years ago
pull cross-commit by default
Categories
(Firefox Build System :: General, defect)
Firefox Build System
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: myk, Assigned: myk)
Details
Attachments
(1 file)
|
1.37 KB,
patch
|
benjamin
:
review+
benjamin
:
approval-branch-1.8.1+
|
Details | Diff | Splinter Review |
cross-commit automates checking into multiple branches simultaneously. To be easily accessible to developers who could take advantage of it, it should get pulled automatically just as other tools (f.e. trace-malloc) are.
| Assignee | ||
Comment 1•20 years ago
|
||
This patch adds mozilla/tools/cross-commit to MODULES_core in client.mk, so users who pull any product get the cross-commit utility. Benjamin, the purpose of this change is to provide this tool, which lets you commit to multiple branches simultaneously, to developers whose work has to land on both the trunk and the MOZILLA_1_8_BRANCH per Brendan's trunk/branch plan (http://wiki.mozilla.org/Global:1.9_Trunk_1.8_Branch_Plan).
Chase suggested this solution, but he also suggested I ask your opinion on this approach. Is this how I should be doing it, or is there another, better way?
Updated•20 years ago
|
Attachment #207662 -
Flags: review?(benjamin) → review+
| Assignee | ||
Comment 2•20 years ago
|
||
Thanks Benjamin.
Checking in client.mk;
/cvsroot/mozilla/client.mk,v <-- client.mk
new revision: 1.254; previous revision: 1.253
done
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
| Assignee | ||
Comment 3•20 years ago
|
||
Comment on attachment 207662 [details] [diff] [review]
patch v1: adds cross-commit to client.mk's MODULES_core
This one-liner makes client.mk pull tools/cross-commit so developers get it by default. cross-commit is a script that simultaneously checks in patches to both the trunk and the 1.8 branch, and it's useful for the many cases these days where developers need to check the same patch in to both places.
Attachment #207662 -
Flags: approval1.8.1?
Comment 4•20 years ago
|
||
Is cross-commit on the 1.8 branch? And why would developers need this on the branch when in general people are going to be commiting from trunk trees?
| Assignee | ||
Comment 5•20 years ago
|
||
> Is cross-commit on the 1.8 branch?
It is.
> And why would developers need this on the branch when in general people are
> going to be commiting from trunk trees?
For those cases where they are committing from branch trees.
Updated•20 years ago
|
Attachment #207662 -
Flags: approval1.8.1? → branch-1.8.1+
Updated•8 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•