From: Sebastian Luther <SebastianLuther@gmx.de>
To: gentoo-portage-dev@lists.gentoo.org
Subject: Re: [gentoo-portage-dev] [PATCH] Document bugzilla workflow
Date: Wed, 15 Jan 2014 21:43:58 +0100 [thread overview]
Message-ID: <52D6F30E.9070402@gmx.de> (raw)
In-Reply-To: <20140115194103.3ed610fc@TOMWIJ-GENTOO>
Am 15.01.2014 19:41, schrieb Tom Wijsman:
>>> Yes, I see some commit messages not refer to bugs which is
>>> something we will want to avoid; think this might need to go
>>> into the commit policy.
>>>
>> There's nothing wrong with fixing/implementing something that
>> nobody filed a bug about.
>
> Sorry, consider the common case where a bug was filed but the
> commit message does not refer to that bug. Also note that I am
> trying to refer to the ChangeLog of Portage itself, not that of the
> ebuild; thus I mean the commit messages for the Portage repo, not
> to the Portage tree.
>
Not sure if we're talking about the same things.
1) If you fix something that has a bug, you should refer to that in
the git commit message.
2) There's nothing wrong with a git commit message that does not refer
to a bug, if there is no bug filed.
> The whole point of documenting it in a workflow is to make it
> converge;
Not the "converge" I meant.
What I meant was to allow people to test different styles and hope
that the one that works best will be adopted by everyone at some
point. Once that happens you can document that style.
> if you instead leave things unclear or undocumented, you have no
> guaranteed convergence and might even see a disconvergence.
Yes, maybe. One then needs to see if that is a problem and if it is
then force everyone to use one style.
>
> It's already making people unhappy right now; because as it is
> documented now, it is turned from the meaningful experience that
> the previous Portage team had before to something that is
> meaningless. It is a regression in checking the list of bugs that
> block the tracker, as the states of the bugs no longer have a value
> as it is documented now.
>
Previously the bug state was not used at all. There is no regression.
next prev parent reply other threads:[~2014-01-15 20:44 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-01-08 18:25 [gentoo-portage-dev] [PATCH] Document bugzilla workflow SebastianLuther
2014-01-15 3:11 ` Tom Wijsman
2014-01-15 6:29 ` Sebastian Luther
2014-01-15 16:20 ` Tom Wijsman
2014-01-15 17:29 ` Sebastian Luther
2014-01-15 18:41 ` Tom Wijsman
2014-01-15 20:43 ` Sebastian Luther [this message]
2014-01-15 22:49 ` Tom Wijsman
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=52D6F30E.9070402@gmx.de \
--to=sebastianluther@gmx.de \
--cc=gentoo-portage-dev@lists.gentoo.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox