Commit | Line | Data |
---|---|---|
51603dab GM |
1 | NOTES ON COMMITTING TO EMACS'S BAZAAR REPO -*- outline -*- |
2 | ||
3 | * Install changes only on one branch, let them get merged elsewhere if needed. | |
4 | In particular, install bug-fixes only on the release branch (if there | |
5 | is one) and let them get synced to the trunk; do not install them by | |
6 | hand on the trunk as well. E.g. if there is an active "emacs-23" branch | |
7 | and you have a bug-fix appropriate for the next Emacs-23.x release, | |
8 | install it only on the emacs-23 branch, not on the trunk as well. | |
9 | ||
10 | Installing things manually into more than one branch makes merges more | |
11 | difficult. | |
12 | ||
13 | http://lists.gnu.org/archive/html/emacs-devel/2010-03/msg01124.html | |
14 | ||
15 | * Backporting a bug-fix from the trunk to a branch (e.g. "emacs-23"). | |
16 | Label the commit as a backport, e.g. by starting the commit message with | |
17 | "Backport:". This is helpful for the person merging the release branch | |
18 | to the trunk. | |
19 | ||
20 | http://lists.gnu.org/archive/html/emacs-devel/2010-05/msg00262.html | |
21 | ||
22 | * Installing changes from your personal branches. | |
23 | If your branch has only a single commit, or many different real | |
24 | commits, it is fine to do a merge. If your branch has only a very | |
25 | small number of "real" commits, but several "merge from trunks", it is | |
26 | preferred that you take your branch's diff, apply it to the trunk, and | |
27 | commit directly, not merge. This keeps the history cleaner. | |
28 | ||
5739cdd2 EZ |
29 | In general, when working on some feature in a separate branch, it is |
30 | preferable not to merge from trunk until you are done with the | |
31 | feature. Unless you really need some change that was done on the | |
32 | trunk while you were developing on the branch, you don't really need | |
33 | those merges; just merge once, when you are done with the feature, and | |
34 | Bazaar will take care of the rest. Bazaar is much better in this than | |
35 | CVS, so interim merges are unnecessary. | |
36 | ||
51603dab GM |
37 | Or use shelves; or rebase; or do something else. See the thread for |
38 | yet another fun excursion into the exciting world of version control. | |
39 | ||
40 | http://lists.gnu.org/archive/html/emacs-devel/2010-04/msg00086.html | |
9aafb22b | 41 | |
9092d186 PE |
42 | * Installing changes from gnulib |
43 | Some of the files in Emacs are copied from gnulib. To synchronize | |
44 | these files from the version of gnulib that you have checked out into | |
45 | a sibling directory of your branch, type "make sync-from-gnulib"; this | |
46 | will check out the latest version of gnulib if there is no sibling | |
47 | directory already. It is a good idea to run "bzr status" afterwards, | |
48 | so that if a gnulib module added a file, you can record the new file | |
49 | using "bzr add". After synchronizing from gnulib, do a "make" in the | |
50 | usual way. | |
51 | ||
52 | To change the set of gnulib modules, change the GNULIB_MODULES | |
53 | variable in the top-level Makefile.in, and then run: | |
54 | ||
55 | ./config.status | |
56 | make sync-from-gnulib | |
57 | bzr status | |
58 | ||
59 | The last command will mention files that may need to be added using | |
60 | "bzr add". If you remove a gnulib module, or if a gnulib module | |
61 | removes a file, then remove the corresponding files by hand. | |
62 | ||
9aafb22b GM |
63 | * How to merge changes from emacs-23 to trunk |
64 | ||
65 | The following description uses bound branches, presumably it works in | |
66 | a similar way with unbound ones. | |
67 | ||
d0ba09dc GM |
68 | 0) (First time only) Get the bzr changelog_merge plugin: |
69 | ||
70 | cd ~/.bazaar/plugins | |
71 | bzr branch lp:bzr-changelog-merge | |
72 | mv bzr-changelog-merge changelog_merge | |
73 | ||
74 | This will make merging ChangeLogs a lot smoother. It merges new | |
75 | entries to the top of the file, rather than trying to fit them in | |
76 | mid-way through. | |
77 | ||
9aafb22b GM |
78 | 1) Get clean, up-to-date copies of the emacs-23 and trunk branches. |
79 | Check for any uncommitted changes with bzr status. | |
80 | ||
81 | 2) M-x cd /path/to/trunk | |
82 | ||
d0ba09dc GM |
83 | The first time only, do this: |
84 | cd .bzr/branch | |
85 | Add the following line to branch.conf: | |
86 | changelog_merge_files = ChangeLog | |
87 | ||
9aafb22b GM |
88 | 3) load admin/bzrmerge.el |
89 | ||
90 | 4) M-x bzrmerge RET /path/to/emacs-23 RET | |
91 | ||
92 | It will prompt about revisions that should be skipped, based on the | |
93 | regexp in bzrmerge-missing. If there are more revisions that you know | |
94 | need skipping, you'll have to do that by hand. | |
95 | ||
b91f171d | 96 | 5) It will stop if there are any conflicts. Resolve them. |
9aafb22b GM |
97 | Using smerge-mode, there are menu items to skip to the next conflict, |
98 | and to take either the trunk, branch, or both copies. | |
99 | ||
565cb736 GM |
100 | 6) After resolving all conflicts, you might need to run the bzmerge |
101 | command again if there are more revisions still to merge. | |
102 | ||
b69258a1 | 103 | Do not commit (or exit Emacs) until you have run bzrmerge to completion. |
9aafb22b | 104 | |
4ebf3ee1 | 105 | Before committing, check bzr status and bzr diff output. |
565cb736 GM |
106 | If you have run bzrmerge enough times, the "pending merge tip" in bzr |
107 | status should be the last revision from the emacs-23 branch, and | |
108 | bzr status -v should show all the revisions you expect to merge. | |
4ebf3ee1 | 109 | |
4474c927 GM |
110 | (Note that it will also show "skipped" revisions. This is expected, |
111 | and is due to a technical limitation of bzr. The log data for those | |
112 | revisions gets merged, the actual changes themselves do not. | |
113 | http://lists.gnu.org/archive/html/emacs-devel/2011-01/msg00609.html ) | |
114 | ||
4ebf3ee1 GM |
115 | Note that ChangeLog entries are automatically merged to the top with |
116 | today's date, but you still might want to check them to see that too | |
565cb736 | 117 | much is not being included, or whitespace between entries is not missing. |
b91f171d GM |
118 | |
119 | Notes: | |
120 | ||
121 | 1) A lot that was in tramp.el in emacs-23 has moved to tramp-sh.el in | |
122 | the trunk. If you end up with a conflict in tramp.el, the changes may | |
123 | need to go to tramp-sh.el instead. Remember to update the file name in | |
124 | the ChangeLog. | |
125 | ||
126 | 2) If a file is modified in emacs-23, and deleted in the trunk, you | |
127 | get a "contents conflict". Assuming the changes don't need to be in | |
128 | the trunk at all, use `bzr resolve path/to/file --take-this' to keep the | |
129 | trunk version. Prior to bzr 2.2.3, this may fail. You can just | |
130 | delete the .OTHER etc files by hand and use bzr resolve path/to/file. | |
565cb736 GM |
131 | |
132 | 3) Conflicts in autoload md5sums in comments. Strictly speaking, the | |
133 | right thing to do is merge everything else, resolve the conflict by | |
b69258a1 | 134 | choosing either the trunk or branch version, then run `make -C lisp |
565cb736 GM |
135 | autoloads' to update the md5sums to the correct trunk value before |
136 | committing. | |
a241b7c0 GM |
137 | |
138 | * Re-adding a file that has been removed from the repository | |
139 | ||
140 | It's easy to get this wrong. Let's suppose you've done: | |
141 | ||
142 | bzr remove file; bzr commit | |
143 | ||
144 | and now, sometime later, you realize this was a mistake and file needs | |
145 | to be brought back. DON'T just do: | |
146 | ||
147 | bzr add file; bzr commit | |
148 | ||
149 | This restores file, but without its history (`bzr log file' will be | |
150 | very short). This is because file gets re-added with a new file-id | |
151 | (use `bzr file-id file' to see the id). | |
152 | ||
153 | Insteading of adding the file, try: | |
154 | ||
155 | bzr revert -rN file; bzr commit | |
156 | ||
157 | where revision N+1 is the one where file was removed. | |
158 | ||
159 | You could also try `bzr add --file-ids-from', if you have a copy of | |
160 | another branch where file still exists. |