Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [mylyn-dev] stop requiring fast-forwards

I've opened 455362: change Gerrit submit type for Mylyn projects
https://bugs.eclipse.org/bugs/show_bug.cgi?id=455362


--
Sam Davis
Software Engineer, Tasktop Dev
Committer, Eclipse Mylyn
http://tasktop.com

On Tue, Dec 16, 2014 at 1:09 AM, Steffen Pingel <steffen.pingel@xxxxxxxxxxx> wrote:
I don't have permission to make that change. I think filing a bug is necessary.

Steffen 

On Tue, Dec 16, 2014 at 12:53 AM, Sam Davis <sam.davis@xxxxxxxxxxx> wrote:
It seems like everyone likes rebase if necessary. Steffen, are you able to make this change or should I open a bug in Community/Gerrit?


--
Sam Davis
Software Engineer, Tasktop Dev
Committer, Eclipse Mylyn
http://tasktop.com

On Wed, Dec 10, 2014 at 4:55 PM, Sam Davis <sam.davis@xxxxxxxxxxx> wrote:
On Wed, Dec 10, 2014 at 1:36 PM, Matthias Sohn <matthias.sohn@xxxxxxxxx> wrote:

the downside of the cherry-pick strategy is that it breaks patch series into individual commits
so if you have a series of commits depending on each other in review cherry-pick will not
respect the commit dependencies which is likely to break the build if the changes really
depend on each other.

Thanks for pointing that out. While it is the documented behaviour, my experience using the cherry-pick strategy with Gerrit 2.8 is that it doesn't actually work that way. Instead, it seems to have the same behaviour as the Fast Forward Only strategy: commits go into a "submitted" state and it periodically tries to merge them. But I don't like that either, because it's easy not to notice that the change wasn't merged.

+1 for "Rebase if Necessary"

_______________________________________________
mylyn-dev mailing list
mylyn-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/mylyn-dev



--
Steffen Pingel
Principal Software Engineer, Eclipse Mylyn
Mylyn Tasks Lead
http://tasktop.com

_______________________________________________
mylyn-dev mailing list
mylyn-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/mylyn-dev

Back to the top