Never mind. This does not work as intended or advertised. I guess the straight merge with duplicate cherry-pick log messages is the only way to go.
On Wed, Sep 26, 2012 at 11:17 AM, Frank Reininghaus <frank78ac@googlemail.com> wrote:
Hi Dawit,
2012/9/26 Dawit A:
> I still do not see how this can work cleanly. People commit changes inI think it has just been overlooked first that this commit is safe
> master and then backport them to the 4.9 branch. Which BTW was by far the
> most common workflow before the switch over to git. As an example if you
> attempt to 'merge' kde-baseapps 4.9 branch into master right now, you will
> get a duplicate log entry for the following commit which has been
> cherry-picked into the 4.9 branch from master:
>
> commit 8c4fa6e03b6b6acedf3a03eef5347f38680818fe
> Author: Emmanuel Pescosta <emmanuelpescosta099@gmail.com>
> Date: Wed Sep 12 19:33:28 2012 +0200
>
> Fixes Bug 305783 - dragging a file over a directory #c4
> does not expand the dir => Bug discovered: When you drag a
> item onto a folder-view-item and then move it away
> instantly before the autoactivation event is triggered
> (After 750ms), the folder will be opened anyway.
>
> BUG: 305783
> REVIEW: 106381
> FIXED-IN: 4.9.2
> (cherry picked from commit 9ab8bcd6aa3ce5d96ee380d5f22d77c2f0a38881)
enough to go into the 4.9 branch. I was the one who encouraged
Emmanuel to commit that fix to 4.9 as well. If you first fix something
in master and later decide that it can also be pushed to 4.9, I see no
other option but to cherry-pick.
I don't know how to avoid a duplicate entry in that case. But given
> How can that be resolved or do we live with the duplicate log entries ?
the big mess of duplicate entries that we had in the past every time
someone decided to merge, I think that a single duplicate entry is
something that we can probably live with.Well, I found an answer to my own question at [1]. Basically, it says doing a rebase of the branch with the master and merging afterwards will prevent the duplicate log message:$ git co -b 4.9 origin/KDE/4.9$ git co -b master origin/master$ git rebase master 4.9$ git co master$ git merge 4.9Perhaps people with more intimate knowledge of git than I want to comment on the validity of above steps ? However, it seems to work just fine when I tested it against kde-baseapps:$ git rebase master 4.9First, rewinding head to replay your work on top of it...Applying: SVN_SILENT made messages (.desktop file)Applying: When user requests to open bookmark urls in a new window or tab, filter theApplying: Fix crash when receiving Nepomuk info for a recently deleted file.Applying: Fixed CTRL+Tab not working when using Dolphin's filemanagement part in Konqueror.Applying: Do not crash when the apply/ok button is pressed after the defaults button when site specific policies presentApplying: Fix a Bug in KFileItemModelRolesUpdate::setEnabledPlugins.$ git br* 4.9masterwork$ git co masterSwitched to branch 'master'$ git merge 4.9Updating d06163e..3ee40a3Fast-forwarddolphin/src/kitemviews/kfileitemmodelrolesupdater.cpp | 9 ++++++++-dolphin/src/kitemviews/kitemlistview.cpp | 2 ++konqueror/settings/kio/kcookiespolicies.cpp | 1 +konqueror/settings/kio/proxy.desktop | 1 +konqueror/src/konqmainwindow.cpp | 9 +++++----5 files changed, 17 insertions(+), 5 deletions(-)$ git log origin/master..commit 3ee40a3c7063e8053cc87b2e9bd8f3a321e67949Author: Emmanuel Pescosta <emmanuelpescosta099@gmail.com>Date: Wed Sep 26 17:36:51 2012 +0200Fix a Bug in KFileItemModelRolesUpdate::setEnabledPlugins.Changes were never overtaken because of a wrong comparison.FIXED-IN: 4.9.2commit 2848b923ddeb65c4dcb5b469c4ac6f0e9a204eb9Author: Dawit Alemayehu <adawit@kde.org>Date: Wed Sep 26 10:28:11 2012 -0400Do not crash when the apply/ok button is pressed after the defaults button when site specific policies presentcommit e95e336767ba9c4d761fdb9f356dd6d78ad01c16Author: Dawit Alemayehu <adawit@kde.org>Date: Tue Sep 25 03:40:50 2012 -0400Fixed CTRL+Tab not working when using Dolphin's filemanagement part in Konqueror.BUG: 302329FIXED-IN: 4.9.2REVIEW: 106569commit 38e3b55b480cb586ccc65c7661775185ddca0c22Author: Frank Reininghaus <frank78ac@googlemail.com>Date: Mon Sep 24 22:59:59 2012 +0200Fix crash when receiving Nepomuk info for a recently deleted file.BUG: 306036FIXED-IN: 4.9.2
commit 4fb0e0a9235b88df409a3718e75f606e56039b6fAuthor: Dawit Alemayehu <adawit@kde.org>Date: Sun Sep 23 02:42:55 2012 -0400When user requests to open bookmark urls in a new window or tab, filter theURLs before opening them.BUG: 145873FIXED-IN: 4.9.2commit 689108b0f37d9ac6f54261f5b94f623e3e63d448Author: Script Kiddy <scripty@kde.org>Date: Sat Sep 22 17:32:41 2012 +0200