From: | Brian Harring <ferringb@×××××.com> | ||
---|---|---|---|
To: | gentoo-portage-dev@l.g.o | ||
Cc: | rafael@×××××××××××××.com | ||
Subject: | Re: [gentoo-portage-dev] Re: [PATCH] Add support to Mercurial SCM in bin/repoman | ||
Date: | Mon, 18 Jan 2010 04:52:56 | ||
Message-Id: | 20100118045157.GC5008@hrair.c.nixle.com | ||
In Reply to: | [gentoo-portage-dev] Re: [PATCH] Add support to Mercurial SCM in bin/repoman by Rafael Martins |
1 | On Sun, Jan 17, 2010 at 08:46:24PM -0200, Rafael Martins wrote: |
2 | > I'm re-sending the patch, with a small fix. Please disregard the previous patch. |
3 | |
4 | Might I suggest breaking classes out for each syncer rather then |
5 | continuing the huge and nasty giant if/elif ? |
6 | |
7 | Certainly would make it easier to maintain and extend. |
8 | ~harring |