public inbox for gentoo-dev@lists.gentoo.org
 help / color / mirror / Atom feed
From: Jeremy Olexa <darkside@gentoo.org>
To: <gentoo-dev@lists.gentoo.org>
Subject: Re: [gentoo-dev] Re: [gentoo-commits] gentoo-x86 commit in dev-python/python-openid: python-openid-2.2.4.ebuild
Date: Tue, 29 Jun 2010 02:49:30 +0000	[thread overview]
Message-ID: <4c3504702d1ba64aa4c6fe1a6f265cc2@localhost> (raw)
In-Reply-To: <201006282226.51247.Arfrever@gentoo.org>

On Mon, 28 Jun 2010 22:26:17 +0200, Arfrever Frehtes Taifersar Arahesis
<Arfrever@gentoo.org> wrote:
> 2010-06-29 04:05:54 Jeremy Olexa napisał(a):
>> On Mon, 28 Jun 2010 19:42:27 +0000 (UTC), "Arfrever Frehtes Taifersar
>> Arahesis (arfrever)" <arfrever@gentoo.org> wrote:
>> > arfrever    10/06/28 19:42:27
>> >
>> >   Modified:             python-openid-2.2.4.ebuild
>> >   Log:
>> >   Fix dependencies.
>> >   (Portage version: HEAD/cvs/Linux x86_64)
>>
>> Is there any reason you are so non-verbose here? 'cvs log' or '$EDITOR
>> ChangeLog' equally give us no information about your commit. You are
>> making it hard on other devs in my opinion, I don't think intentionally,
>> but can't you just use the ChangeLog more??
> 
> It was intermediate commit during my work on python-openid-2.2.5.ebuild.
> python-openid-2.2.4.ebuild has been mentioned in ChangeLog in final commit.

You are correct - 2.2.4 IS mentioned in the ChangeLog during the
subsequent commit. So you think it is ok to hide the first commit under
a ChangeLog entry of "Version bump" ? I don't see the logic. My issue
with this is that other devs (or users) still don't know how or why the
dependancies got in the 2.2.5 version or what deps were fixed.

In this case, I would have committed a new 2.2.5 version with the
ChangeLog entry of: "Version bump, fix python dependancies that are
incorrect in the old" or somthing like that. This way commit #1 is not
hidden and placed under a false entry of commit #2.

-Jeremy



      parent reply	other threads:[~2010-06-28 20:43 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20100628194227.C34B42C621@corvid.gentoo.org>
2010-06-29  2:05 ` [gentoo-dev] Re: [gentoo-commits] gentoo-x86 commit in dev-python/python-openid: python-openid-2.2.4.ebuild Jeremy Olexa
2010-06-28 20:26   ` Arfrever Frehtes Taifersar Arahesis
2010-06-28 22:54     ` Sebastian Pipping
2010-06-29  2:49     ` Jeremy Olexa [this message]

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=4c3504702d1ba64aa4c6fe1a6f265cc2@localhost \
    --to=darkside@gentoo.org \
    --cc=gentoo-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