r18562 - in /packages/experimental/epiphany-extensions-more: debian/README.Debian debian/changelog misc/video-downloader.py

[joss at users.alioth.debian.org]

Author: joss
Date: Mon Mar  2 12:56:54 2009
New Revision: 18562

URL: http://svn.debian.org/wsvn/pkg-gnome/?sc=1&rev=18562
Log:
video-downloader.py: fix proxy support.

Modified:
    packages/experimental/epiphany-extensions-more/debian/README.Debian
    packages/experimental/epiphany-extensions-more/debian/changelog
    packages/experimental/epiphany-extensions-more/misc/video-downloader.py

Modified: packages/experimental/epiphany-extensions-more/debian/README.Debian
URL: http://svn.debian.org/wsvn/pkg-gnome/packages/experimental/epiphany-extensions-more/debian/README.Debian?rev=18562&op=diff
==============================================================================
--- packages/experimental/epiphany-extensions-more/debian/README.Debian (original)
+++ packages/experimental/epiphany-extensions-more/debian/README.Debian Mon Mar  2 12:56:54 2009
@@ -1,5 +1,5 @@
-epiphany-extension-more
-=======================
+epiphany-extensions-more
+========================
 
 This is a collection of extensions from various sources. The quality and
 stability of the extensions are just as various.
@@ -44,4 +44,3 @@
  -----------
 The specific menu entry was removed, instead we override the existing
 menu entry with an identical-looking one.
-

Modified: packages/experimental/epiphany-extensions-more/debian/changelog
URL: http://svn.debian.org/wsvn/pkg-gnome/packages/experimental/epiphany-extensions-more/debian/changelog?rev=18562&op=diff
==============================================================================
--- packages/experimental/epiphany-extensions-more/debian/changelog (original)
+++ packages/experimental/epiphany-extensions-more/debian/changelog Mon Mar  2 12:56:54 2009
@@ -1,6 +1,7 @@
 epiphany-extensions-more (2.24.1) UNRELEASED; urgency=low
 
   * Add universal-edit-button extension.
+  * video-downloader.py: fix proxy support.
 
  -- Josselin Mouette <joss at debian.org>  Mon, 16 Feb 2009 23:49:23 +0100
 

Modified: packages/experimental/epiphany-extensions-more/misc/video-downloader.py
URL: http://svn.debian.org/wsvn/pkg-gnome/packages/experimental/epiphany-extensions-more/misc/video-downloader.py?rev=18562&op=diff
==============================================================================
--- packages/experimental/epiphany-extensions-more/misc/video-downloader.py (original)
+++ packages/experimental/epiphany-extensions-more/misc/video-downloader.py Mon Mar  2 12:56:54 2009
@@ -188,13 +188,14 @@
 
 def download_videos(window, embed, video_urls, open_inside):
     clive_cmd_base = [ 'clive', '--quiet', '--emit-csv' ]
+    clive_env = {}
     if proxy != None:
-        clive_cmd_base.append('--proxy=%s' % proxy)
+        clive_env["http_proxy"]=proxy
 
     for video_url in video_urls:
         clive_cmd = clive_cmd_base + [ video_url ] 
         try:
-            new_download_ps = Popen(clive_cmd, stdout=PIPE, stderr=PIPE)
+            new_download_ps = Popen(clive_cmd, stdout=PIPE, stderr=PIPE, close_fds=True, env=clive_env)
         except OSError, e:
             print 'OSError executing clive', e
             if e.args[0] == 2: