Ok, done. I'll give a summary of what happened:
- r13232, source:/sandbox/rails-4.1 branch created from source:/trunk@13231
- commits on the rails-4.1 branch are made, along with further changes to the trunk (for 2.6.x) [also some merge and backport commits, but that's not relevant here]
- r13443, rails-4.1 branch is rebased from trunk at r13439
- r13482, rails-4.1 branch is merged into trunk
It went wrong in step 3. This is a rebase from trunk at r13439, but in fact it is a rebase from trunk r13333 upto r13439. Since there are commits made on trunk between r13232 and r13333 which were not yet included on the rails-4.1 branch, these changes are reverted in the final merge of the rails-4.1 branch into trunk with r13482. Basically, the branch was not up-to-date with the trunk during the merge.
I reported this issue (while the branch wasn't merged yet) in #18134 (under point 2.1), and provided a patch which completed the rebase to include all commits. This patch was made against the rails-4.1 branch in order to bring it up-to-date completely with trunk (for easy merging in the end...;). The patch couldn't be integrated before the merge already happened...
In retrospect I should have reported the above mentioned issue separate from the merge tracking info issues, which could have prevented this issue from arising completely...
Nevertheless, back to the issue. The respective commits are:
Commit |
Status: |
r13284 |
changes not reverted (locales) |
r13285 |
changes reverted, but reimplemented already in r13502 |
r13287 |
changes reverted, to-be-fixed |
r13299 |
changes not reverted (locales) |
r13303 |
changes reverted, to-be-fixed |
r13322 |
changes reverted, to-be-fixed |
r13323 |
changes reverted, to-be-fixed |
r13324 |
changes reverted, to-be-fixed |
r13325 |
changes reverted, to-be-fixed |
r13326 |
changes reverted, to-be-fixed |
r13327 |
changes reverted, to-be-fixed |
r13329 |
changes reverted, to-be-fixed |
r13330 |
changes reverted, to-be-fixed |
r13331 |
changes reverted, but reimplemented already in r13502 |
I have fixed this issue (by "re-merging" the respective commits) on a local checkout of source:/trunk@13503 using:
merge from: trunk
revisions range: 13287,13303,13322-13327,13329-13330
merge into: trunk
ignore ancestry: true
merge conflict caused by r13322 in:
* Gemfile
- caused by additional change already in trunk [removal of rake gem requirement] --> solved locally.
I have wrapped up these changes into a single patch file (svn diff > rm18174-reintegrate_commits_on_trunk_reverted_by_r13482_merge-trunk_r13503.patch
) to-be-applied onto source:/trunk. I'll attach it here.
toshio harita: I've removed you as the assignee. It doesn't matter who actually commits this patch...
I'll add some committers as watchers though. This to make sure it gets picked up...
Kind regards,
Mischa.