Re: [E-devel] Git, merges, and better work-flows

2013-10-06 Thread Bertrand Jacquin
On 2013-10-06 23:47, Tom Hacohen wrote: > Yeah, you can delete the remote branch after merging to master. > > The trick here is that we don't send emails about commits in dev > branches > and we don't send emails about commits that were merged into master > from a > dev branch (that is, commits

Re: [E-devel] Git, merges, and better work-flows

2013-10-06 Thread Tom Hacohen
Yeah, you can delete the remote branch after merging to master. The trick here is that we don't send emails about commits in dev branches and we don't send emails about commits that were merged into master from a dev branch (that is, commits that have already existed in the repo at the time of the

Re: [E-devel] Git, merges, and better work-flows

2013-10-06 Thread Daniel Juyung Seo
But that will create a remote branch which is not needed for others. Or I can delete the remote branch after merging to master. Is that ok? Daniel Juyung Seo (SeoZ) On Sun, Oct 6, 2013 at 12:36 AM, Tom Hacohen wrote: > Well. There's a way to avoid that: If you push to a remote branch before >

Re: [E-devel] Git, merges, and better work-flows

2013-10-05 Thread Tom Hacohen
Well. There's a way to avoid that: If you push to a remote branch before merging, there will be no tsunami. A mail will only be sent to the merge commit. On Sat, Oct 5, 2013 at 4:05 PM, Daniel Juyung Seo wrote: > I thought the same as David Seikel :) > That's why the commit tsunami happened. > >

Re: [E-devel] Git, merges, and better work-flows

2013-10-05 Thread Tom Hacohen
*about the merge commit. On Sat, Oct 5, 2013 at 4:36 PM, Tom Hacohen wrote: > Well. There's a way to avoid that: If you push to a remote branch before > merging, there will be no tsunami. A mail will only be sent to the merge > commit. > > > On Sat, Oct 5, 2013 at 4:05 PM, Daniel Juyung Seo wro

Re: [E-devel] Git, merges, and better work-flows

2013-10-05 Thread Daniel Juyung Seo
I thought the same as David Seikel :) That's why the commit tsunami happened. Anyhow, it's same as before except we now allow merge commit for some cases. Daniel Juyung Seo (SeoZ) On Sat, Oct 5, 2013 at 8:09 PM, Tom Hacohen wrote: > No. The point is not to squash them, but have one cover-lette

Re: [E-devel] Git, merges, and better work-flows

2013-10-05 Thread Tom Hacohen
Exactly. Awesome. :) On Sat, Oct 5, 2013 at 9:58 AM, Daniel Juyung Seo wrote: > On Fri, Oct 4, 2013 at 11:48 PM, Tom Hacohen >wrote: > > > On 04/10/13 15:40, Michael Blumenkrantz wrote: > > > On Fri, 04 Oct 2013 15:18:46 +0100 > > > Tom Hacohen wrote: > > > > > >> On 02/10/13 16:17, Tom Hacohe

Re: [E-devel] Git, merges, and better work-flows

2013-10-05 Thread Tom Hacohen
No. The point is not to squash them, but have one cover-letter commit that holds them all. If you "git log --first-parent" you won't see all the commits, just the merge commit that describes the whole changeset. On Sat, Oct 5, 2013 at 10:36 AM, David Seikel wrote: > On Sat, 5 Oct 2013 17:58:17

Re: [E-devel] Git, merges, and better work-flows

2013-10-05 Thread David Seikel
On Sat, 5 Oct 2013 17:58:17 +0900 Daniel Juyung Seo wrote: > On Fri, Oct 4, 2013 at 11:48 PM, Tom Hacohen > wrote: > > > On 04/10/13 15:40, Michael Blumenkrantz wrote: > > > On Fri, 04 Oct 2013 15:18:46 +0100 > > > Tom Hacohen wrote: > > > > > >> On 02/10/13 16:17, Tom Hacohen wrote: > > >>> He

Re: [E-devel] Git, merges, and better work-flows

2013-10-05 Thread Daniel Juyung Seo
On Fri, Oct 4, 2013 at 11:48 PM, Tom Hacohen wrote: > On 04/10/13 15:40, Michael Blumenkrantz wrote: > > On Fri, 04 Oct 2013 15:18:46 +0100 > > Tom Hacohen wrote: > > > >> On 02/10/13 16:17, Tom Hacohen wrote: > >>> Hey guys, > >>> > >>> I would like to suggest a new work-flow. This work-flow wil

Re: [E-devel] Git, merges, and better work-flows

2013-10-04 Thread David Seikel
On Sat, 5 Oct 2013 11:54:46 +0900 Carsten Haitzler (The Rasterman) wrote: > On Fri, 4 Oct 2013 15:57:52 -0300 Lucas De Marchi > said: > > > On Fri, Oct 4, 2013 at 7:05 AM, Carsten Haitzler > > wrote: > > > On Fri, 04 Oct 2013 11:42:35 +0200 Bertrand Jacquin > > > said: > > > > > >> > steps 3,

Re: [E-devel] Git, merges, and better work-flows

2013-10-04 Thread The Rasterman
On Fri, 4 Oct 2013 15:57:52 -0300 Lucas De Marchi said: > On Fri, Oct 4, 2013 at 7:05 AM, Carsten Haitzler wrote: > > On Fri, 04 Oct 2013 11:42:35 +0200 Bertrand Jacquin > > said: > > > >> > steps 3, 4, 5, 6, 8... are an incantation. to be done in that order. > >> > there is > >> > --no-ff... a

Re: [E-devel] Git, merges, and better work-flows

2013-10-04 Thread Lucas De Marchi
On Fri, Oct 4, 2013 at 7:05 AM, Carsten Haitzler wrote: > On Fri, 04 Oct 2013 11:42:35 +0200 Bertrand Jacquin said: > >> > steps 3, 4, 5, 6, 8... are an incantation. to be done in that order. >> > there is >> > --no-ff... and he series.. its a DEFINITE incantation. read up on git >> > as a >> > l

Re: [E-devel] Git, merges, and better work-flows

2013-10-04 Thread Jérémy Zurcher
On Friday 04 October 2013 15:59, Tom Hacohen wrote : > On 04/10/13 16:05, Jérémy Zurcher wrote: > > On Friday 04 October 2013 15:18, Tom Hacohen wrote : > >>> > >> > >> So, is this a "go"? May I write up some documentation about it and start > >> doing it? > >> > >> -- > >> Tom. > > > > for me it

Re: [E-devel] Git, merges, and better work-flows

2013-10-04 Thread Tom Hacohen
On 04/10/13 16:05, Jérémy Zurcher wrote: > On Friday 04 October 2013 15:18, Tom Hacohen wrote : >>> >> >> So, is this a "go"? May I write up some documentation about it and start >> doing it? >> >> -- >> Tom. > > for me it's a +1 go > > in case that's part of your plan, > I've added an 'eo2: ' pre

Re: [E-devel] Git, merges, and better work-flows

2013-10-04 Thread Jérémy Zurcher
On Friday 04 October 2013 15:18, Tom Hacohen wrote : > > > > So, is this a "go"? May I write up some documentation about it and start > doing it? > > -- > Tom. for me it's a +1 go in case that's part of your plan, I've added an 'eo2: ' prefix to the commits in devs/jeyzu/eo2-old. I can still

Re: [E-devel] Git, merges, and better work-flows

2013-10-04 Thread Tom Hacohen
On 04/10/13 15:40, Michael Blumenkrantz wrote: > On Fri, 04 Oct 2013 15:18:46 +0100 > Tom Hacohen wrote: > >> On 02/10/13 16:17, Tom Hacohen wrote: >>> Hey guys, >>> >>> I would like to suggest a new work-flow. This work-flow will not be >>> mandatory, but just an allowed alternative to the curren

Re: [E-devel] Git, merges, and better work-flows

2013-10-04 Thread Michael Blumenkrantz
On Fri, 04 Oct 2013 15:18:46 +0100 Tom Hacohen wrote: > On 02/10/13 16:17, Tom Hacohen wrote: > > Hey guys, > > > > I would like to suggest a new work-flow. This work-flow will not be > > mandatory, but just an allowed alternative to the current "commit to > > master" approach. > > > > At the mom

Re: [E-devel] Git, merges, and better work-flows

2013-10-04 Thread Tom Hacohen
On 02/10/13 16:17, Tom Hacohen wrote: > Hey guys, > > I would like to suggest a new work-flow. This work-flow will not be > mandatory, but just an allowed alternative to the current "commit to > master" approach. > > At the moment we do not allow merges, at all. This was to prevent people > from li

Re: [E-devel] Git, merges, and better work-flows

2013-10-04 Thread The Rasterman
On Fri, 04 Oct 2013 11:42:35 +0200 Bertrand Jacquin said: > > steps 3, 4, 5, 6, 8... are an incantation. to be done in that order. > > there is > > --no-ff... and he series.. its a DEFINITE incantation. read up on git > > as a > > leaky abstraction. it is very much one. it forces peole to learn

Re: [E-devel] Git, merges, and better work-flows

2013-10-04 Thread Bertrand Jacquin
> steps 3, 4, 5, 6, 8... are an incantation. to be done in that order. > there is > --no-ff... and he series.. its a DEFINITE incantation. read up on git > as a > leaky abstraction. it is very much one. it forces peole to learn a > series of > incantations/steps all the time as opposed to just h

Re: [E-devel] Git, merges, and better work-flows

2013-10-03 Thread Tom Hacohen
On 03/10/13 03:55, Carsten Haitzler (The Rasterman) wrote: > On Wed, 02 Oct 2013 16:20:38 +0100 Tom Hacohen said: > >> On 02/10/13 16:17, Tom Hacohen wrote: >>> Hey guys, >>> >>> I would like to suggest a new work-flow. This work-flow will not be >>> mandatory, but just an allowed alternative to t

Re: [E-devel] Git, merges, and better work-flows

2013-10-03 Thread Tom Hacohen
On 03/10/13 08:48, Peter Kjellerstedt wrote: >> -Original Message- >> From: Daniel Juyung Seo [mailto:seojuyu...@gmail.com] >> Sent: den 3 oktober 2013 06:55 >> To: Enlightenment developer list >> Subject: Re: [E-devel] Git, merges, and better work-flows >

Re: [E-devel] Git, merges, and better work-flows

2013-10-03 Thread Peter Kjellerstedt
> -Original Message- > From: Daniel Juyung Seo [mailto:seojuyu...@gmail.com] > Sent: den 3 oktober 2013 06:55 > To: Enlightenment developer list > Subject: Re: [E-devel] Git, merges, and better work-flows > > On Thu, Oct 3, 2013 at 12:20 AM, Tom Hacohen > wrote:

Re: [E-devel] Git, merges, and better work-flows

2013-10-02 Thread Daniel Juyung Seo
Thanks Tasn for the suggestion. I like the idea. It's been a while since we started "Rebase instead of Merge" policy and it worked well so far. Now it's time to enhance it more. Thanks. Daniel Juyung Seo (SeoZ) On Thu, Oct 3, 2013 at 12:17 AM, Tom Hacohen wrote: > Hey guys, > > I would like t

Re: [E-devel] Git, merges, and better work-flows

2013-10-02 Thread Daniel Juyung Seo
On Thu, Oct 3, 2013 at 12:20 AM, Tom Hacohen wrote: > On 02/10/13 16:17, Tom Hacohen wrote: > > Hey guys, > > > > I would like to suggest a new work-flow. This work-flow will not be > > mandatory, but just an allowed alternative to the current "commit to > > master" approach. > > > > At the moment

Re: [E-devel] Git, merges, and better work-flows

2013-10-02 Thread Lucas De Marchi
On Thu, Oct 3, 2013 at 12:25 AM, Carsten Haitzler wrote: > On Thu, 3 Oct 2013 00:10:17 -0300 Lucas De Marchi > said: > >> On Wed, Oct 2, 2013 at 11:55 PM, Carsten Haitzler >> wrote: >> > On Wed, 02 Oct 2013 16:20:38 +0100 Tom Hacohen >> > said: >> > >> >> On 02/10/13 16:17, Tom Hacohen wrote: >

Re: [E-devel] Git, merges, and better work-flows

2013-10-02 Thread The Rasterman
On Thu, 3 Oct 2013 00:10:17 -0300 Lucas De Marchi said: > On Wed, Oct 2, 2013 at 11:55 PM, Carsten Haitzler > wrote: > > On Wed, 02 Oct 2013 16:20:38 +0100 Tom Hacohen > > said: > > > >> On 02/10/13 16:17, Tom Hacohen wrote: > >> > Hey guys, > >> > > >> > I would like to suggest a new work-flow

Re: [E-devel] Git, merges, and better work-flows

2013-10-02 Thread Lucas De Marchi
On Wed, Oct 2, 2013 at 11:55 PM, Carsten Haitzler wrote: > On Wed, 02 Oct 2013 16:20:38 +0100 Tom Hacohen said: > >> On 02/10/13 16:17, Tom Hacohen wrote: >> > Hey guys, >> > >> > I would like to suggest a new work-flow. This work-flow will not be >> > mandatory, but just an allowed alternative t

Re: [E-devel] Git, merges, and better work-flows

2013-10-02 Thread The Rasterman
On Wed, 02 Oct 2013 16:20:38 +0100 Tom Hacohen said: > On 02/10/13 16:17, Tom Hacohen wrote: > > Hey guys, > > > > I would like to suggest a new work-flow. This work-flow will not be > > mandatory, but just an allowed alternative to the current "commit to > > master" approach. > > > > At the mome

Re: [E-devel] Git, merges, and better work-flows

2013-10-02 Thread Gustavo Lima Chaves
* Tom Hacohen [2013-10-02 17:58:45 +0100]: > On 02/10/13 17:51, Gustavo Sverzut Barbieri wrote: > > -0 I'm not convinced but I don't have any evidence that it's bad, just > > a personal feeling. > > > > However I'm not an active committer, so I don't count much these days > > and I'll follow what

Re: [E-devel] Git, merges, and better work-flows

2013-10-02 Thread Tom Hacohen
On 02/10/13 18:12, Adrien Nader wrote: > Hi, > > On Wed, Oct 02, 2013, Gustavo Sverzut Barbieri wrote: >> -0 I'm not convinced but I don't have any evidence that it's bad, just >> a personal feeling. > > All it does is add a commit that says "branch X was merged in branch Y". > There are two possib

Re: [E-devel] Git, merges, and better work-flows

2013-10-02 Thread Adrien Nader
Hi, On Wed, Oct 02, 2013, Gustavo Sverzut Barbieri wrote: > -0 I'm not convinced but I don't have any evidence that it's bad, just > a personal feeling. All it does is add a commit that says "branch X was merged in branch Y". There are two possible drawbacks: - commit message should be checked to

Re: [E-devel] Git, merges, and better work-flows

2013-10-02 Thread Tom Hacohen
On 02/10/13 17:51, Gustavo Sverzut Barbieri wrote: > -0 I'm not convinced but I don't have any evidence that it's bad, just > a personal feeling. > > However I'm not an active committer, so I don't count much these days > and I'll follow whatever you choose. All I'm asking is the permission to do

Re: [E-devel] Git, merges, and better work-flows

2013-10-02 Thread Gustavo Sverzut Barbieri
-0 I'm not convinced but I don't have any evidence that it's bad, just a personal feeling. However I'm not an active committer, so I don't count much these days and I'll follow whatever you choose. On Wed, Oct 2, 2013 at 12:20 PM, Tom Hacohen wrote: > On 02/10/13 16:17, Tom Hacohen wrote: >> Hey

Re: [E-devel] Git, merges, and better work-flows

2013-10-02 Thread Tom Hacohen
On 02/10/13 16:17, Tom Hacohen wrote: > Hey guys, > > I would like to suggest a new work-flow. This work-flow will not be > mandatory, but just an allowed alternative to the current "commit to > master" approach. > > At the moment we do not allow merges, at all. This was to prevent people > from li

[E-devel] Git, merges, and better work-flows

2013-10-02 Thread Tom Hacohen
Hey guys, I would like to suggest a new work-flow. This work-flow will not be mandatory, but just an allowed alternative to the current "commit to master" approach. At the moment we do not allow merges, at all. This was to prevent people from littering the log with their inability to rebase (g