From: Rogério Brito <rbrito@ime.usp.br>
Date: Fri, 10 Apr 2020 12:46:46 +0000 (-0300)
Subject: Merge pull request #3 from unit193/master
X-Git-Url: https://git.rapsys.eu/youtubedl/commitdiff_plain/8b2307d66b10597b66d9491668d75d0e5ae5ab0c?hp=e22397a1c5329e12f3719961c9456e378f94dbb1

Merge pull request #3 from unit193/master

* Tweak the auto-update features of youtube-dl.
* Add infrastructure to enable tests after build time.

Courtesy of @unit193.
---

diff --git a/debian/control b/debian/control
index 731e1c0..eae9bf9 100644
--- a/debian/control
+++ b/debian/control
@@ -7,7 +7,9 @@ Build-Depends:
  debhelper-compat (= 12),
  dh-exec,
  dh-python,
+ flake8 <!nocheck>,
  pandoc,
+ python3-nose <!nocheck>,
  python3-pkg-resources,
  python3-setuptools,
  zip
diff --git a/debian/patches/disable-autoupdate-mechanism.patch b/debian/patches/disable-autoupdate-mechanism.patch
index 2b906bd..45b68d1 100644
--- a/debian/patches/disable-autoupdate-mechanism.patch
+++ b/debian/patches/disable-autoupdate-mechanism.patch
@@ -15,9 +15,29 @@ Last-Update: 2018-03-16
  youtube_dl/options.py  | 2 +-
  2 files changed, 7 insertions(+), 2 deletions(-)
 
+--- a/README.md
++++ a/README.md
+@@ -53,9 +53,6 @@
+ # OPTIONS
+     -h, --help                       Print this help text and exit
+     --version                        Print program version and exit
+-    -U, --update                     Update this program to latest version. Make
+-                                     sure that you have sufficient permissions
+-                                     (run with sudo if needed)
+     -i, --ignore-errors              Continue on download errors, for example to
+                                      skip unavailable videos in a playlist
+     --abort-on-error                 Abort downloading of further videos (in the
 --- a/youtube_dl/__init__.py
 +++ b/youtube_dl/__init__.py
-@@ -441,7 +441,12 @@
+@@ -36,7 +36,6 @@
+     write_string,
+     render_table,
+ )
+-from .update import update_self
+ from .downloader import (
+     FileDownloader,
+ )
+@@ -441,7 +440,12 @@
      with YoutubeDL(ydl_opts) as ydl:
          # Update version
          if opts.update_self:
diff --git a/debian/patches/disable_more_online_tests.patch b/debian/patches/disable_more_online_tests.patch
new file mode 100644
index 0000000..d7fcc47
--- /dev/null
+++ b/debian/patches/disable_more_online_tests.patch
@@ -0,0 +1,21 @@
+Description: Disable remaining tests that require internet.
+Author: Unit 193 <unit193@ubuntu.com>
+
+---
+ Makefile |    5 ++++-
+ 1 file changed, 4 insertions(+), 1 deletion(-)
+
+--- a/Makefile
++++ b/Makefile
+@@ -49,7 +49,10 @@
+ 		--exclude test_subtitles.py \
+ 		--exclude test_write_annotations.py \
+ 		--exclude test_youtube_lists.py \
+-		--exclude test_youtube_signature.py
++		--exclude test_youtube_signature.py \
++		--exclude test_downloader_http.py \
++		--exclude test_InfoExtractor.py \
++		--exclude test_http.py
+ 
+ tar: youtube-dl.tar.gz
+ 
diff --git a/debian/patches/series b/debian/patches/series
index 611c3c9..119b552 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -1,2 +1,3 @@
 skip_support_file_installation.patch
 disable-autoupdate-mechanism.patch
+disable_more_online_tests.patch
diff --git a/debian/rules b/debian/rules
index 82d26cb..e1e9a31 100755
--- a/debian/rules
+++ b/debian/rules
@@ -9,12 +9,13 @@ export PATH := $(CURDIR)/debian/utils:$(PATH)
 
 override_dh_auto_configure:
 	PREFIX=/usr SYSCONFDIR=/etc make
-	mv Makefile Makefile.inactive
 	rm youtube-dl
 
 override_dh_clean:
-	test -f Makefile || mv Makefile.inactive Makefile
 	make clean
 	dh_clean
 
 override_dh_auto_test:
+ifeq (,$(filter nocheck,$(DEB_BUILD_OPTIONS)))
+	make offlinetest PYTHON=python3
+endif