diff options
| author | Philipp Hagemeister <phihag@phihag.de> | 2013-11-25 22:15:20 +0100 | 
|---|---|---|
| committer | Philipp Hagemeister <phihag@phihag.de> | 2013-11-25 22:15:33 +0100 | 
| commit | 0c75c3fa7a24c05a74891ec49e5a18de4f2792f1 (patch) | |
| tree | 6cc1ead57d6a3d23de717e0ec2be6289068b88b4 | |
| parent | 2a15e7063bceed326bcbc0a01ba77324f0373a0c (diff) | |
Do not warn about fixed output template if --max-downloads is 1
Fixes #1828
| -rw-r--r-- | youtube_dl/YoutubeDL.py | 8 | ||||
| -rw-r--r-- | youtube_dl/__init__.py | 4 | 
2 files changed, 6 insertions, 6 deletions
| diff --git a/youtube_dl/YoutubeDL.py b/youtube_dl/YoutubeDL.py index 0cef1daf3..50f750593 100644 --- a/youtube_dl/YoutubeDL.py +++ b/youtube_dl/YoutubeDL.py @@ -258,10 +258,6 @@ class YoutubeDL(object):          if self.params.get('cookiefile') is not None:              self.cookiejar.save() -    def fixed_template(self): -        """Checks if the output template is fixed.""" -        return (re.search(u'(?u)%\\(.+?\\)s', self.params['outtmpl']) is None) -      def trouble(self, message=None, tb=None):          """Determine action to take when a download problem appears. @@ -798,7 +794,9 @@ class YoutubeDL(object):      def download(self, url_list):          """Download a given list of URLs.""" -        if len(url_list) > 1 and self.fixed_template(): +        if (len(url_list) > 1 and +                '%' not in self.params['outtmpl'] +                and self.params.get('max_downloads') != 1):              raise SameFileError(self.params['outtmpl'])          for url in url_list: diff --git a/youtube_dl/__init__.py b/youtube_dl/__init__.py index 1f15c7eaa..102508cf9 100644 --- a/youtube_dl/__init__.py +++ b/youtube_dl/__init__.py @@ -206,7 +206,9 @@ def parseOpts(overrideArguments=None):              dest='playlistend', metavar='NUMBER', help='playlist video to end at (default is last)', default=-1)      selection.add_option('--match-title', dest='matchtitle', metavar='REGEX',help='download only matching titles (regex or caseless sub-string)')      selection.add_option('--reject-title', dest='rejecttitle', metavar='REGEX',help='skip download for matching titles (regex or caseless sub-string)') -    selection.add_option('--max-downloads', metavar='NUMBER', dest='max_downloads', help='Abort after downloading NUMBER files', default=None) +    selection.add_option('--max-downloads', metavar='NUMBER', +                         dest='max_downloads', type=int, default=None, +                         help='Abort after downloading NUMBER files')      selection.add_option('--min-filesize', metavar='SIZE', dest='min_filesize', help="Do not download any videos smaller than SIZE (e.g. 50k or 44.6m)", default=None)      selection.add_option('--max-filesize', metavar='SIZE', dest='max_filesize', help="Do not download any videos larger than SIZE (e.g. 50k or 44.6m)", default=None)      selection.add_option('--date', metavar='DATE', dest='date', help='download only videos uploaded in this date', default=None) | 
