From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gentoo-dev+bounces-59767-garchives=archives.gentoo.org@lists.gentoo.org>
Received: from lists.gentoo.org (pigeon.gentoo.org [208.92.234.80])
	by finch.gentoo.org (Postfix) with ESMTP id 5F9981381F3
	for <garchives@archives.gentoo.org>; Mon, 22 Apr 2013 12:00:46 +0000 (UTC)
Received: from pigeon.gentoo.org (localhost [127.0.0.1])
	by pigeon.gentoo.org (Postfix) with SMTP id 8C920E0A8E;
	Mon, 22 Apr 2013 12:00:40 +0000 (UTC)
Received: from mail-ia0-f175.google.com (mail-ia0-f175.google.com [209.85.210.175])
	(using TLSv1 with cipher ECDHE-RSA-RC4-SHA (128/128 bits))
	(No client certificate requested)
	by pigeon.gentoo.org (Postfix) with ESMTPS id A0B3DE0A03
	for <gentoo-dev@lists.gentoo.org>; Mon, 22 Apr 2013 12:00:39 +0000 (UTC)
Received: by mail-ia0-f175.google.com with SMTP id i38so1114467iae.34
        for <gentoo-dev@lists.gentoo.org>; Mon, 22 Apr 2013 05:00:38 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
        d=gmail.com; s=20120113;
        h=mime-version:x-received:reply-to:sender:in-reply-to:references:date
         :x-google-sender-auth:message-id:subject:from:to:cc:content-type;
        bh=3IRW/VlTkBLkn7uFumsMpxvLfdlTnPZ5M073H2TkeWE=;
        b=V2zrW+T1ZeSnrkx4i7EwNEaXn58imHucqwYIgUxEHzC+0To+z4cIbuVCxvmQAS7bx/
         LtFOd2D7dKTQwsXpNdE2orjHEWgbQw1Xlkb6VZ3FILW1hahco2nsXuL8WrbohywW2kPb
         VFUik0WtqhJAviZe9BXeoyv5IGxkBvJnZHynNmszs1X9BQdfxyAMeStDYwP6LUA1TeRW
         v1G197vbdxbYbsMgkTNB+C4Eek1Q1uVAO/Jz4wDG7vdmmfXvUMsGrDy2vg3Q9evc9pGG
         WfadwFMAn7DeJMn7UXHzlxLpfge24d5lBOH1Jf3QSWPDym3KsGy7v6yoFEWtkXzmschA
         Nz/g==
Precedence: bulk
List-Post: <mailto:gentoo-dev@lists.gentoo.org>
List-Help: <mailto:gentoo-dev+help@lists.gentoo.org>
List-Unsubscribe: <mailto:gentoo-dev+unsubscribe@lists.gentoo.org>
List-Subscribe: <mailto:gentoo-dev+subscribe@lists.gentoo.org>
List-Id: Gentoo Linux mail <gentoo-dev.gentoo.org>
X-BeenThere: gentoo-dev@lists.gentoo.org
Reply-to: gentoo-dev@lists.gentoo.org
MIME-Version: 1.0
X-Received: by 10.43.146.3 with SMTP id jw3mr11492291icc.39.1366632038910;
 Mon, 22 Apr 2013 05:00:38 -0700 (PDT)
Sender: yngwin@gmail.com
Received: by 10.64.30.234 with HTTP; Mon, 22 Apr 2013 05:00:38 -0700 (PDT)
In-Reply-To: <20130421170549.41cfea49@portable>
References: <20130419091632.D01152171D@flycatcher.gentoo.org>
	<20130419153043.30ffc50c@portable>
	<CAB9SyzRq-RuMUU0cXnWYC_+-yv3FSJ=fDw335QmUrqQinsbzQw@mail.gmail.com>
	<20130421170549.41cfea49@portable>
Date: Mon, 22 Apr 2013 20:00:38 +0800
X-Google-Sender-Auth: FZf8a9biqBPTPhPOTwClPLgAFfU
Message-ID: <CAB9SyzQ7f6145adfAcZgMVhdKWBjCHPsVSV-nTOCQbx7rt1PSA@mail.gmail.com>
Subject: Re: [OT/NIT] Re: [gentoo-dev] Re: [gentoo-commits] gentoo-x86 commit
 in profiles: ChangeLog package.mask
From: Ben de Groot <yngwin@gentoo.org>
To: Alexis Ballier <aballier@gentoo.org>
Cc: gentoo-dev <gentoo-dev@lists.gentoo.org>
Content-Type: multipart/alternative; boundary=001a11c2e9aa381e1f04daf1d178
X-Archives-Salt: e070c7c8-2f26-4b76-be33-11d2557f9c4a
X-Archives-Hash: 2cd401a4ab54a36f71930f26762109a5

--001a11c2e9aa381e1f04daf1d178
Content-Type: text/plain; charset=UTF-8

On 21 April 2013 23:05, Alexis Ballier <aballier@gentoo.org> wrote:

> On Sun, 21 Apr 2013 20:53:28 +0800
> Ben de Groot <yngwin@gentoo.org> wrote:
> > >
> > > PS: btw, some hunks are weird in your commit, a locale issue ?
> > >
> >
> > No, just a line in my vimrc that removes trailing whitespace.
> >
>
> You should probably disable it or remove trailing whitespaces in a
> separate commit though. Having functional changes mixed with
> whitespace/cosmetics in a single commit makes it hard to read
> and understand.
>
> [This is really a nitpick, no need to have a debate, it's only a
> suggestion ;)]
>
> Alexis.
>

I don't see the problem. Also, in this case I see only one extra hunk.
But once we have proper tools (like git) we can revisit this.
I don't think there is currently any guideline in devmanual that
recommends not mixing functional changes with cosmetics.

-- 
Cheers,

Ben | yngwin
Gentoo developer
Gentoo Qt project lead, Gentoo Wiki admin

--001a11c2e9aa381e1f04daf1d178
Content-Type: text/html; charset=UTF-8

<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On 21 April 2013 23:05, Alexis Ballier <span dir="ltr">&lt;<a href="mailto:aballier@gentoo.org" target="_blank">aballier@gentoo.org</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Sun, 21 Apr 2013 20:53:28 +0800<br>
Ben de Groot &lt;<a href="mailto:yngwin@gentoo.org">yngwin@gentoo.org</a>&gt; wrote:<br>
&gt; &gt;<br>
</div><div class="im">&gt; &gt; PS: btw, some hunks are weird in your commit, a locale issue ?<br>
&gt; &gt;<br>
&gt;<br>
&gt; No, just a line in my vimrc that removes trailing whitespace.<br>
&gt;<br>
<br>
</div>You should probably disable it or remove trailing whitespaces in a<br>
separate commit though. Having functional changes mixed with<br>
whitespace/cosmetics in a single commit makes it hard to read<br>
and understand.<br>
<br>
[This is really a nitpick, no need to have a debate, it&#39;s only a<br>
suggestion ;)]<br>
<span class="HOEnZb"><font color="#888888"><br>
Alexis.<br>
</font></span></blockquote></div><br></div><div class="gmail_extra">I don&#39;t see the problem. Also, in this case I see only one extra hunk.<br></div><div class="gmail_extra">But once we have proper tools (like git) we can revisit this.<br>
</div><div class="gmail_extra">I don&#39;t think there is currently any guideline in devmanual that<br>recommends not mixing functional changes with cosmetics.<br clear="all"></div><div class="gmail_extra"><br>-- <br>Cheers,<br>
<br>Ben | yngwin<br>Gentoo developer<br>Gentoo Qt project lead, Gentoo Wiki admin
</div></div>

--001a11c2e9aa381e1f04daf1d178--