From: "W. Trevor King" <wking@tremily.us>
To: gentoo-catalyst@lists.gentoo.org
Cc: catalyst@gentoo.org
Subject: Re: [gentoo-catalyst] [PATCH 1/7] Remove unused urllib import.
Date: Mon, 1 Sep 2014 22:22:37 -0700 [thread overview]
Message-ID: <20140902052237.GJ8263@odin.tremily.us> (raw)
In-Reply-To: <1409625101-27112-2-git-send-email-dolsen@gentoo.org>
[-- Attachment #1: Type: text/plain, Size: 1908 bytes --]
Previous submissions for this patch: [1,2]. It's already been acked
by Matt [3] and me [4]. I acked it again on IRC on 2014-06-15:
08:02 < wking> dol-sen: I don't see how c718315 (Remove unused
urllib import, 2013-02-11) could possibly break anything ;).
08:02 < wking> But I'm going through stages one through three anyway
just to be sure ;)
08:03 < wking> Are you going to mail these to the list, or are you
planning on polishing them first, and then submitting them to the
list?
08:03 <@dol-sen> they need some work
…
18:16 < wking> dol-sen: c718315 is now 135 of 263 in my stage 3, so
I think it's good to merge to master (after floating on the list,
if that's still Catalyst policy).
21:50 <@jmbsvicetto> dol-sen / wking: please keep reviewing those
patches with any help in the catalyst ml. I'll likely be tied for
a while and won't have time to go over them, but I trust your
judgement
21:50 <@jmbsvicetto> dol-sen: feel free to commit them whenever
you're happy.
21:51 <@jmbsvicetto> Zero_Chaos: ^^ Please try to review them when
you can to ensure you don't get any surprises
21:52 <@jmbsvicetto> dol-sen: I can try to setup automated builds
with the rewrite branch on my server so we can get automated
feedback about it
So after three submissions and three acks (one on this exact c718315
commit), I think we can safely land this one ;).
Cheers,
Trevor
[1]: http://article.gmane.org/gmane.linux.gentoo.catalyst/2158
[2]: http://article.gmane.org/gmane.linux.gentoo.catalyst/2364
[3]: http://thread.gmane.org/gmane.linux.gentoo.catalyst/2361/focus=2378
[4]: http://thread.gmane.org/gmane.linux.gentoo.catalyst/2361/focus=2425
--
This email may be signed or encrypted with GnuPG (http://www.gnupg.org).
For more information, see http://en.wikipedia.org/wiki/Pretty_Good_Privacy
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
next prev parent reply other threads:[~2014-09-02 5:22 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-09-02 2:31 [gentoo-catalyst] [PATCH 0/7] Pending branch patches (various) Brian Dolbec
2014-09-02 2:31 ` [gentoo-catalyst] [PATCH 1/7] Remove unused urllib import Brian Dolbec
2014-09-02 5:22 ` W. Trevor King [this message]
2014-09-02 2:31 ` [gentoo-catalyst] [PATCH 2/7] Remove unused variable new and an undefined variable s Brian Dolbec
2014-09-02 5:43 ` W. Trevor King
2014-09-05 18:16 ` W. Trevor King
2014-09-02 2:31 ` [gentoo-catalyst] [PATCH 3/7] stage1-controller.sh: Fix portage bin path hard coding Brian Dolbec
2014-09-02 3:35 ` Rick "Zero_Chaos" Farina
2014-09-02 5:05 ` [gentoo-catalyst] [PATCH v2] stage1-controller.sh: Remove some old poor cleaning code Brian Dolbec
2014-09-02 5:19 ` [gentoo-catalyst] [PATCH v2 3/7] " Brian Dolbec
2014-09-02 5:31 ` [gentoo-catalyst] [PATCH v2] " W. Trevor King
2014-09-09 18:38 ` Brian Dolbec
2014-09-10 17:16 ` Rick "Zero_Chaos" Farina
2014-09-10 21:00 ` Anthony G. Basile
2014-09-02 2:31 ` [gentoo-catalyst] [PATCH 4/7] chroot-functions.sh: Remove --nodeps option from portage update Brian Dolbec
2014-09-02 2:31 ` [gentoo-catalyst] [PATCH 5/7] chroot-functions.sh: Fix a mis-worded comment Brian Dolbec
2014-09-02 2:31 ` [gentoo-catalyst] [PATCH 6/7] setup_pkgmgr(): WIP Make the 'build' use flag optional Brian Dolbec
2014-09-09 18:27 ` Brian Dolbec
2014-09-02 2:31 ` [gentoo-catalyst] [PATCH 7/7] Fix a relative path bug Brian Dolbec
2014-09-02 5:06 ` W. Trevor King
2014-09-02 5:26 ` Brian Dolbec
2014-09-09 17:20 ` Brian Dolbec
2014-09-11 3:30 ` [gentoo-catalyst] [PATCH 0/7] Pending branch patches (various) Brian Dolbec
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=20140902052237.GJ8263@odin.tremily.us \
--to=wking@tremily.us \
--cc=catalyst@gentoo.org \
--cc=gentoo-catalyst@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