From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from lists.gentoo.org (pigeon.gentoo.org [208.92.234.80]) by finch.gentoo.org (Postfix) with ESMTP id 1EDDE138247 for ; Sat, 14 Dec 2013 22:25:01 +0000 (UTC) Received: from pigeon.gentoo.org (localhost [127.0.0.1]) by pigeon.gentoo.org (Postfix) with SMTP id 40C41E0B25; Sat, 14 Dec 2013 22:24:47 +0000 (UTC) Received: from qmta04.westchester.pa.mail.comcast.net (qmta04.westchester.pa.mail.comcast.net [76.96.62.40]) by pigeon.gentoo.org (Postfix) with ESMTP id 7D08BE0AEB for ; Sat, 14 Dec 2013 22:24:45 +0000 (UTC) Received: from omta24.westchester.pa.mail.comcast.net ([76.96.62.76]) by qmta04.westchester.pa.mail.comcast.net with comcast id 1aMP1n0041ei1Bg54aQl4A; Sat, 14 Dec 2013 22:24:45 +0000 Received: from odin.tremily.us ([24.18.63.50]) by omta24.westchester.pa.mail.comcast.net with comcast id 1aQj1n008152l3L3kaQj9J; Sat, 14 Dec 2013 22:24:44 +0000 Received: by odin.tremily.us (Postfix, from userid 1000) id AEF4DD68B16; Sat, 14 Dec 2013 14:24:42 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=tremily.us; s=odin; t=1387059882; bh=ysPBfeiIgE1rrEKkXhUR93uVNw/G7S+GBwuRkicsmwE=; h=From:To:Cc:Subject:Date:In-Reply-To:References; b=C1l6E1vNddsK3Adlzm14g0PohAaG0S4YRQApmslT9NkMH6LH++V2S4hKB3rskdhAX MescijVjGDpHIExD2QHHR/8H2FEh7qbzm8iyWrKMfQVIEwpEw69yqabt7jFpx/6tRt c2RNI9yJ7vnafUpuAzyzMZ661o6dywxSyFQGMbFQ= From: "W. Trevor King" To: Catalyst Cc: "W. Trevor King" Subject: [gentoo-catalyst] [PATCH v2 00/21] rewrite-on-master patches (part 1) Date: Sat, 14 Dec 2013 14:24:08 -0800 Message-Id: X-Mailer: git-send-email 1.8.3.2 In-Reply-To: References: Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: List-Id: Gentoo Linux mail X-BeenThere: gentoo-catalyst@lists.gentoo.org Reply-to: gentoo-catalyst@lists.gentoo.org MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=comcast.net; s=q20121106; t=1387059885; bh=j579ME8AzBUuW7JMPuafAhwRla69GZR2MWksE9o3pm0=; h=Received:Received:Received:From:To:Subject:Date:Message-Id: MIME-Version:Content-Type; b=rJuUBjbltzZYJRocvFERGcv86umveShqBnlhKdBqqY5ckY2rT4j9Cmr+16M8epzig sXfP65nu1r0Usq1MKAVBjnjjbshDjTgHmJfqQ5fqfhMy7zFMc14ZJiQDTTIF+zHPgt TEi0g+sASZ1ye0bg+QNDSch8d3ufLWythNSN+IDOdxl91Z7GCSJepNVJ/dIGMSTZso MT3fiQVgVJl3OIk8FGwPALhAQcPE0sTILtdXNRUKcx5bB4efzxc7GDY15z+eNnac2U 1rx8cc4OtmHcuf+wJPqtqv79gPAGKBwwGhSfQS8bfDTqzWRGZ1MfkPZxFZKYJtZKJX d1Xjjj1J4kgLg== X-Archives-Salt: 1f0b7b34-f557-48ec-9419-26ba905161e3 X-Archives-Hash: b7633ce1ccea998be102df161c3c82ff From: "W. Trevor King" On Sat, Dec 14, 2013 at 10:44:13AM -0800, Matt Turner wrote: > I feel they're basically ready to go, but I want everyone to be in > the habit of sending patches to the list, so please do. Here you go :). > Preemptive review: I saw just a few minor things, almost entirely > dealing with lack of whitespace around operators. git log -p --reverse > origin/master..wking/dolsen-rewrite-part-1 and then search for ^\+.*[^ > ]\+[^ ] (less regex) finds a couple. I added spaces around = and +. Beyond that, I agree with Dustin: On Sat, Dec 14, 2013 at 03:31:50PM -0600, Dustin C. Hatch wrote: > On 12/14/2013 10:42, W. Trevor King wrote: > > The current code base is so far from PEP 8 that changing touched > > sections to match PEP 8 looks really out of place. In this > > instance it's probably fine, but I'd ok with putting off all/most > > PEP 8 cleanups to their own pure-reformatting commits. > > I strongly agree with this idea. If we start mixing PEP8 and non > PEP8 compliant code, it will get really hard to read really fast. I > think we should wait until most if not all of this rewrite get onto > master before we start changing stuff for the sake of reformatting. On Sat, Dec 14, 2013 at 10:44:13AM -0800, Matt Turner wrote: > I'd opt to just put parenthesis around prints where we're going to be > line wrapping its argument. It accomplishes the same thing as the > trailing slash, but moves us slightly in the direction of python3. So, > fix up lines you think it possible that match ^\+.*\\$ Whitespace inconsistency is still readable, but having ‘print(…)’ where we touch lines, surrounded by ‘print …’ where we didn't touch lines is a bit too far for me ;). Can we just move to: from __future__ import print_function and ‘print(…)’ as a mass-replace once we finish the rest of Brian's series? > The other thing: I think we should drop most of the last patch for > now. Wrapping long lines before we switch from 8-space tabs to PEP8 > 4-space style seems premature. The other half of that commit that > replaces a series of prints with a single one looks good though. I've split the commit and placed the long-line wrapping at the end of this part of the series. There aren't too many lines in there, so rebasing the remainder of the series onto master shouldn't be to bad if you don't merge the last commit from this part. > You may want to put your Signed-off-by on the patches to note that > you refactored them heavily. Done. Brian, I'll wait until you've acked this form of your branch, and then add sign-offs from you too, so you can ok (or not) my Git refactoring. Cheers, Trevor Brian Dolbec (21): modules/tinderbox_target.py: Use 'portdir' instead of hard-coding '/usr/portage' modules/generic_stage_target.py: Use 'portdir' instead of hard-coding '/usr/portage' modules/generic_stage_target.py: Use 'portdir' instead of hard-coding '/usr/portage' modules/generic_stage_target.py: Use 'distdir' instead of hard-coding '${PORTAGE}/distfiles' modules/generic_stage_target.py: Use a 'local_overlay' setting instead of hard-coding '/usr/local/portage' catalyst: Split confdefaults into line-per-entry catalyst: Add 'repo_name' default catalyst: Add 'snapshot_name' default catalyst: Add 'packagedir' default instead of hard-coding '/usr/portage/packages' catalyst: Add 'port_tmpdir' default instead of hard-coding '/var/tmp/portage' modules/generic_stage_target.py: Don't use paths as mountmap keys modules/generic_stage_target.py: Use 'proc' instead of '/proc' as the mountmap key modules/generic_stage_target.py: Use 'dev' instead of '/dev' as the mountmap key modules/generic_stage_target.py: Use 'distdir' instead of '/usr/portage/distfiles' as the mountmap key modules/generic_stage_target.py: Use 'port_tmpdir' instead of '/var/tmp/portage' as the mountmap key modules/generic_stage_target.py: Use 'devpts' instead of '/dev/pts' as the mountmap key modules/generic_stage_target.py: Use 'packagedir' instead of '/usr/portage/packages' as the mountmap key modules/generic_stage_target.py: Use 'kerncache' instead of '/tmp/kerncache' as the mountmap key modules/generic_stage_target.py: Use 'ccache' instead of '/var/tmp/ccache' as the mountmap key catalst: improve usage() output formatting slightly catalyst: cleanup long lines catalyst | 101 ++++++++++++++++++++++++---------------- modules/generic_stage_target.py | 92 ++++++++++++++++++------------------ modules/snapshot_target.py | 14 ++++-- modules/tinderbox_target.py | 8 +++- 4 files changed, 123 insertions(+), 92 deletions(-) -- 1.8.3.2