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 D7611198005 for ; Wed, 27 Feb 2013 01:31:04 +0000 (UTC) Received: from pigeon.gentoo.org (localhost [127.0.0.1]) by pigeon.gentoo.org (Postfix) with SMTP id 6189EE0686; Wed, 27 Feb 2013 01:31:04 +0000 (UTC) Received: from smtp.gentoo.org (smtp.gentoo.org [140.211.166.183]) (using TLSv1 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by pigeon.gentoo.org (Postfix) with ESMTPS id D797FE0686 for ; Wed, 27 Feb 2013 01:31:03 +0000 (UTC) Received: from [192.168.1.210] (unknown [24.86.176.233]) (using TLSv1 with cipher ECDHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) (Authenticated sender: dolsen) by smtp.gentoo.org (Postfix) with ESMTPSA id 968CA33DEDD; Wed, 27 Feb 2013 01:31:02 +0000 (UTC) Message-ID: <1361928640.3997.283.camel@big_daddy.dol-sen.ca> Subject: Re: [gentoo-catalyst] patch, fix broken seed stage update From: Brian Dolbec To: gentoo-catalyst@lists.gentoo.org Cc: zmedico@gentoo.org, fuzzyray@gentoo.org Date: Tue, 26 Feb 2013 17:30:40 -0800 In-Reply-To: <20130226180428.GA22651@odin.tremily.us> References: <20130226180428.GA22651@odin.tremily.us> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.6.3 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-Transfer-Encoding: 8bit X-Archives-Salt: 43dff804-7b74-43cb-8387-9e07323985fa X-Archives-Hash: 9994518d5f0fc760fedcaee25f8dd0e6 On Tue, 2013-02-26 at 13:04 -0500, W. Trevor King wrote: > On Tue, Feb 26, 2013 at 08:47:14AM -0800, Brian Dolbec wrote: > > Also I've rebased everything on current master > > That should make things easier to merge :). > > It looks like some of my earlier comments were addressed by this > reroll, but some are still applicable. Apologies if we'd resolved any > of this earlier and I just missed the reference in my mailbox. > > I map my old comments onto the rebased commits below, but the bulk of > the outstanding suggestions revolve around: > > * ConfigParser-based configuration > * Argparse-based command line parsing > * Logging-based debugging output > * os.path.join(), normpath(), … for path manipulation > > These are mostly “take advantage of Python's standard library” > changes, and I'd be happy to help implement them on top of the current > master if folks feel like that has a chance of getting merged ;). > All that is for a TODO list. What I did was rebase my existing changes, merging fixes to the commits that made the change..., fix the commit messages, etc... Same with the other things you listed below, mostly for a TODO and your wishlist. Pretty much all I agree with. My aim for now is to debug the hell out of it, to stabilize all the existing changes, BEFORE doing a ton more changes. That will also give me more experience in how catalyst is used, possibly ideas how to change it for the better. I'll fix the doc's generation and make a setup.py and new 9999 ebuild. Plus I need to focus more on another project needed for the git tree migration. > Cheers, > Trevor >