public inbox for gentoo-catalyst@lists.gentoo.org
 help / color / mirror / Atom feed
From: Brian Dolbec <dolsen@gentoo.org>
To: gentoo-catalyst@lists.gentoo.org
Cc: Brian Dolbec <dolsen@gentoo.org>
Subject: [gentoo-catalyst] [PATCH 1/3] Remove unused urllib import.
Date: Fri, 13 Dec 2013 19:28:26 -0800	[thread overview]
Message-ID: <1386991708-9385-2-git-send-email-dolsen@gentoo.org> (raw)
In-Reply-To: <1386991708-9385-1-git-send-email-dolsen@gentoo.org>

This urllib import was added in commit 64c16cae70da13de3c55d8555a2e4c5dcdf2fcad
to fix an issue, but it is not used.  So must have covered up the real bug.  Removing it now
that I've completed some testing and haven't come across anything I can attribute to it.
---
 catalyst/support.py | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/catalyst/support.py b/catalyst/support.py
index 316dfa3..0ddf98d 100644
--- a/catalyst/support.py
+++ b/catalyst/support.py
@@ -18,10 +18,6 @@ except:
 # pids this process knows of.
 spawned_pids = []
 
-try:
-        import urllib
-except SystemExit, e:
-        raise
 
 def cleanup(pids,block_exceptions=True):
         """function to go through and reap the list of pids passed to it"""
-- 
1.8.3.2



  reply	other threads:[~2013-12-14  3:28 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-12-14  3:28 [gentoo-catalyst] rewite-on-master patches round-3 Brian Dolbec
2013-12-14  3:28 ` Brian Dolbec [this message]
2013-12-14  3:28 ` [gentoo-catalyst] [PATCH 2/3] Remove unused variable new and an undefined variable s Brian Dolbec
2013-12-14  3:28 ` [gentoo-catalyst] [PATCH 3/3] Fix undefined variable: RLIMIT_NOFILE Brian Dolbec
2013-12-14  5:28 ` [gentoo-catalyst] rewite-on-master patches round-3 Matt Turner
2013-12-14 23:53   ` W. Trevor King

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=1386991708-9385-2-git-send-email-dolsen@gentoo.org \
    --to=dolsen@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