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
next prev parent 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.