public inbox for gentoo-dev@lists.gentoo.org
 help / color / mirror / Atom feed
From: Brian Harring <ferringb@gmail.com>
To: gentoo-dev@lists.gentoo.org
Subject: Re: Common sense in [gentoo-dev] (was Council May Summary: Changes to ChangeLog handling)
Date: Mon, 30 May 2011 16:38:34 -0700	[thread overview]
Message-ID: <20110530233834.GA21879@hrair> (raw)
In-Reply-To: <201105310005.09218.dilfridge@gentoo.org>

[-- Attachment #1: Type: text/plain, Size: 1450 bytes --]

On Tue, May 31, 2011 at 12:05:03AM +0200, Andreas K. Huettel wrote:
> Am Montag 30 Mai 2011, 23:55:52 schrieb Brian Harring:
> > If someone has a definition that is commonsense, then propose it- the
> > current "you must log everything" is very, very heavy handed and
> > basically was a forced situation since QA cannot make folks behave
> > when the rules are reliant on common sense.
> 
> Well how about "any change that can influence the behaviour of (portage|your 
> favourite package manager) in any way or present the user with different 
> output"?

Mostly correct, although the problem there is 'influence'; consider:

src_unpack() {
  bunch of code
}

being changed into

new_func() {
  bunch of code
}
src_unpack() {
  invoke new_func
}

That should have no influence, thus not being ChangeLog'd.  The 
problem is when the dev screws up, and it *has* an influence but no 
ChangeLog.

A more real world example is people abusing eval and other things 
(python eclass for example)- folks can/do argue that it has no 
influence, but the complexity means it may have unexpected affect.

That's the crux of the issue, and it comes down to common sense.  
Come up w/ a sane policy for things like that and I'll owe you some 
beer.

Either way, for the rest of it, as Diego said, LGTM.  I'm just 
nitpicking here to make it absolutely clear to people where we start 
running into policy issues.

~brian

[-- Attachment #2: Type: application/pgp-signature, Size: 198 bytes --]

  parent reply	other threads:[~2011-05-30 23:39 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-05-17 10:32 [gentoo-dev] Council May Summary: Changes to ChangeLog handling Petteri Räty
2011-05-20 10:19 ` Mart Raudsepp
2011-05-30 12:03   ` Peter Volkov
2011-05-30 12:23     ` Ulrich Mueller
2011-05-30 18:07       ` William Hubbs
2011-05-30 17:52     ` Michał Górny
2011-05-30 21:55     ` Brian Harring
2011-05-30 22:05       ` Common sense in [gentoo-dev] (was Council May Summary: Changes to ChangeLog handling) Andreas K. Huettel
2011-05-30 22:14         ` [gentoo-dev] Re: Common sense in " Diego Elio Pettenò
2011-05-30 23:38         ` Brian Harring [this message]
2011-05-31  0:01           ` Common sense in [gentoo-dev] " Ângelo Arrifano
2011-06-01 15:08       ` RFC: better policy for ChageLogs (was: Re: [gentoo-dev] " Peter Volkov
2011-06-01 15:15         ` [gentoo-dev] Re: RFC: better policy for ChageLogs Markos Chandras
2011-06-01 15:27           ` Samuli Suominen
2011-06-01 15:34             ` Ciaran McCreesh
2011-06-02 11:21               ` Jorge Manuel B. S. Vicetto
2011-06-01 15:39             ` Andreas K. Huettel
     [not found]               ` <BANLkTimgHYKnfbeQJpq829YBeE-5Yz=aEg@mail.gmail.com>
2011-06-01 16:24                 ` Andreas K. Huettel
2011-06-01 19:50                   ` Nirbheek Chauhan
2011-06-01 23:29                     ` Jorge Manuel B. S. Vicetto
2011-06-01 23:37                       ` Matt Turner
2011-06-02  5:09                         ` Peter Volkov
2011-06-02  7:29                           ` Duncan
2011-06-02  7:40                           ` Eray Aslan
2011-06-02 11:20                             ` Patrick Lauer
2011-06-02 11:05                       ` Nirbheek Chauhan
2011-06-02 11:10                         ` Ciaran McCreesh
2011-06-02 14:43                         ` Rich Freeman
2011-06-01 15:30           ` Nathan Phillip Brink
2011-06-01 16:44             ` Duncan
2011-06-01 22:59               ` Rich Freeman
2011-06-01 23:21                 ` [gentoo-dev] " Diego Elio Pettenò
2011-06-01 23:29                 ` [gentoo-dev] " Dale
2011-06-01 23:40                 ` Jorge Manuel B. S. Vicetto

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=20110530233834.GA21879@hrair \
    --to=ferringb@gmail.com \
    --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