All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mikko Rantalainen <mikko.rantalainen@peda.net>
To: grub-devel@gnu.org
Subject: Re: GIT workflow
Date: Thu, 14 Nov 2013 15:20:10 +0200	[thread overview]
Message-ID: <5284CE0A.6090703@peda.net> (raw)
In-Reply-To: <527FBC01.5010509@gmail.com>

Vladimir 'φ-coder/phcoder' Serbinenko, 2013-11-10 19:01 (Europe/Helsinki):
> Hello, all. We've switched to git some time ago, now we should have some
> kind of workflow documents. In particular I think of following points:
> - Developpers with commit access can create branches as they see fit as
> long as it's prefixed by their name and they don't do sth nasty like
> storing binary or unrelated files.
> - When committing bigger work should we merge or squash? I think that
> squash should be possible if developper desires. Is there any reason to
> use merges?

Squashed merge is identical to rebase && merge --no-ff except for the 
detail that squashing loses any meaningful history for the patch series. 
I'd seriously suggest rebase followed by merge --no-ff over squashed 
merges. The only exception is the case where commits in the original 
work are not logical patches but instead random snapshots of the 
directory tree during development of the patch. In that case, squashing 
the patch series loses no valuable information.

The reason to keep patch series: git bisect

> - Which commits should we sign? All? Some? Official releases?

Depends on what you mean by "sign". If you mean

   Signed-off-by: A U Thor <a.u.thor@example.com>

that's the "Developer Certificate Of Origin":
http://elinux.org/Developer_Certificate_Of_Origin

Other projects (e.g Grub) can decide their own policy for such metadata. 
Additional info is available at 
http://stackoverflow.com/questions/1962094/what-is-the-sign-off-feature-in-git-for

If you mean digitally signed, the correct method is to use signed tags 
for all the releases meant for non-developers. See "git help tag" and 
look for "--sign".

-- 
Mikko



  reply	other threads:[~2013-11-14 13:20 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-11-10 17:01 GIT workflow Vladimir 'φ-coder/phcoder' Serbinenko
2013-11-14 13:20 ` Mikko Rantalainen [this message]
2013-11-25 18:26   ` Andrey Borzenkov
2013-11-28  6:45     ` Vladimir 'phcoder' Serbinenko
2013-11-28 17:26       ` Andrey Borzenkov
2013-12-03 12:24         ` Colin Watson
  -- strict thread matches above, loose matches on Subject: below --
2008-03-10 12:37 Git workflow Peter Gordon
2008-03-11  1:58 ` Thomas Harning
2008-03-11  5:15 ` Junio C Hamano

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5284CE0A.6090703@peda.net \
    --to=mikko.rantalainen@peda.net \
    --cc=grub-devel@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.