X-Git-Url: http://gitweb.michael.orlitzky.com/?p=dead%2Fwhatever-dl.git;a=blobdiff_plain;f=bin%2Fwhatever-dl;h=787e9dbb93c748ed5f949bce8e50052a15968081;hp=970e0e09eb809097d83059ee615c3dfadcbfa039;hb=0662d93e5088ecfd2ce351910ab9a1d3568f8359;hpb=af614c64b3d5998471af5e54b3d8f36d3e00cc63

diff --git a/bin/whatever-dl b/bin/whatever-dl
index 970e0e0..787e9db 100755
--- a/bin/whatever-dl
+++ b/bin/whatever-dl
@@ -1,4 +1,4 @@
-#!/usr/bin/ruby -w
+#!/usr/bin/ruby -wKU
 #
 # whatever-dl, a script to download online (web-based) videos.
 #
@@ -19,14 +19,35 @@
 # http://www.fsf.org/licensing/licenses/gpl.html
 #
 
-# We require the UriUtilities class to handle
-# the download of the video URL.
-require 'src/uri_utilities'
+# We need Pathname to get the real filesystem path
+# of this script (and not, for example, the path of
+# a symlink which points to it.
+require 'pathname'
 
+# And getoptlong to check for our one option, --continue.
+require 'getoptlong'
+
+# This bit of magic adds the parent directory (the
+# project root) to the list of ruby load paths.
+# Thus, our require statements will work regardless of
+# how or from where the script was run.
+executable = Pathname.new(__FILE__).realpath.to_s
+$: << File.dirname(executable) + '/../'
+
+# Load our config file.
+require 'bin/configuration'
+
+# And the downloaders...
+require 'src/downloader'
+
+# The Dir.glob that's coming up doesn't use the
+# Ruby library path so we need to tell it where to
+# look explicitly.
+websites_pattern = File.dirname(executable) + '/../src/websites/*.rb'
 
 # All of the website classes are located in one
 # directory, so we can 'require' them automatically.
-Dir.glob('src/websites/*.rb').each do |r|
+Dir.glob(websites_pattern).each do |r|
   require r
 end
 
@@ -35,74 +56,87 @@ EXIT_SUCCESS = 0
 EXIT_NO_URL = 1
 EXIT_INVALID_URL = 2
 EXIT_COULDNT_GET_VIDEO_URL = 3
-EXIT_OUTPUT_FILE_ALREADY_EXISTS = 4
+EXIT_IO_ERROR = 4
 EXIT_ERROR_READING_FROM_VIDEO_URL = 5
 EXIT_CONNECTION_REFUSED = 6
 EXIT_HTTP_ERROR = 7
 EXIT_ACCESS_DENIED = 8
 
+def usage()
+  puts <<EOF
+
+Usage: whatever-dl [options] <url>
+
+Options:
+  -c, --continue        Continue downloading a previously-attempted file.
+
+EOF
+
+end
+
 # Only actually do something if this script was called
 # directly (i.e. not from the tests).
 if (__FILE__ == $0) then
+  # Default options.
+  options = { :continue => false }
+
+  # Parse the command-line options into the options hash.
+  opts = GetoptLong.new(["--continue", "-c", GetoptLong::NO_ARGUMENT],
+                        ["--help", "-h", GetoptLong::NO_ARGUMENT])
+
+  opts.each do |opt, arg|
+    case opt
+    when '--help'
+      usage()
+      Kernel.exit(EXIT_SUCCESS)
+    when '--continue'
+      options[:continue] = true
+    end
+  end
+
+  # Warn about nonsensical options.
+  if options[:continue] and not (Configuration::DOWNLOAD_METHOD == :wget)
+    puts 'WARNING: The --continue flag does nothing unless DOWNLOAD_METHOD is :wget.'
+  end
+
+  # Note that GetoptLong steals its arguments from ARGV, so we don't need
+  # to take optional arguments into account when figuring out whether or not
+  # we were passed a URL.
   if (ARGV.length < 1) then
     # If the user didn't give us a URL, yell
     # at him or her.
-    puts 'Usage: whatever-dl <url>'
+    usage()
     Kernel.exit(EXIT_NO_URL)
   end
 
-  # Check the URL against each website's class.
-  # The class will know whether or not the URL
-  # "belongs" to its website.
-
-  site = nil
-  
-  Website.subclasses.each do |w|
-    if w.owns_url?(ARGV[0])
-      site = w.new()
-      break
-    end
-  end
+  # Factory method.
+  site = Website.create(ARGV[0])
 
   if site.nil?
     puts 'Invalid URL.'
     exit(EXIT_INVALID_URL)
   end
   
-  video_url = site.get_video_url(ARGV[0])
+  video_url = site.get_video_url()
 
   if video_url.nil?
-    puts 'Error retrieving video URL.'
-    exit(EXIT_COULDNT_GET_VIDEO_URL)
-  end
-
-  video_uri = URI.parse(video_url)
-  uu = UriUtilities.new()
-  
+    puts 'Error retrieving video URL:'
+    puts "Site not supported, and the generic parser couldn't find any videos."
+    exit(EXIT_COULDNT_GET_VIDEO_URL) end
 
-  # Here, we start out with a default file name and
-  # extension. If UriUtilities can parse a sane filename
-  # out of the URL, we'll use that. Otherwise, we fall
-  # back to the default.
-  outfile_name = 'default.ext'
+  # The Downloader class is a factory; it should decide
+  # which subclass we get.
+  downloader = Downloader.create(Configuration::DOWNLOAD_METHOD)
   
-  if not uu.get_filename(video_uri).nil?
-    outfile_name = uu.get_filename(video_uri)
-  else
-    puts "We couldn't determine the video's filename. Falling back to the default, #{outfile_name}."
-  end
-
-  
-  if File.exists?(outfile_name)
-    puts "Error: output file already exists. Please remove #{outfile_name}, and try again."
-    Kernel.exit(EXIT_OUTPUT_FILE_ALREADY_EXISTS)
-  end
-
-
   # Attempt to download the file, and rescue and report
-  # any (predictable) exceptions.
+  # any (predictable) exceptions. The wget downloader will
+  # naturally not report any of these, since it will die in
+  # its own process.
   begin
-    uu.download_with_progress_bar(video_uri, outfile_name)
+    downloader.download(video_url,
+                        site.get_video_filename(),
+                        site.headers(),
+                        continue=options[:continue])
   rescue Errno::ECONNREFUSED => e
     puts 'The connection to the server (to download the video file) was refused. Check your connection, and try again later.'
     Kernel.exit(EXIT_CONNECTION_REFUSED)
@@ -111,7 +145,13 @@ if (__FILE__ == $0) then
   rescue OpenURI::HTTPError => e
     puts "An HTTP error occurred while downloading the video file: #{e.message}."
     Kernel.exit(EXIT_HTTP_ERROR)
+  rescue IOError => e
+    puts "Input/Output Error: #{e.message}"
+    Kernel.exit(EXIT_IO_ERROR)
   end
 
+  # Write an empty line at the end for aesthetic reasons.
+  puts ''
+  
   Kernel.exit(EXIT_SUCCESS)
 end