Browse Source

[setup.py] Only print a warning if documentation files are missing (Fixes #780)

Philipp Hagemeister 11 years ago
parent
commit
6c57e8a063
1 changed files with 19 additions and 5 deletions
  1. 19 5
      setup.py

+ 19 - 5
setup.py

@@ -3,7 +3,9 @@
 
 
 from __future__ import print_function
 from __future__ import print_function
 
 
+import os.path
 import pkg_resources
 import pkg_resources
+import warnings
 import sys
 import sys
 
 
 try:
 try:
@@ -44,12 +46,24 @@ py2exe_params = {
 if len(sys.argv) >= 2 and sys.argv[1] == 'py2exe':
 if len(sys.argv) >= 2 and sys.argv[1] == 'py2exe':
     params = py2exe_params
     params = py2exe_params
 else:
 else:
+    files_spec = [
+        ('etc/bash_completion.d', ['youtube-dl.bash-completion']),
+        ('share/doc/youtube_dl', ['README.txt']),
+        ('share/man/man1', ['youtube-dl.1'])
+    ]
+    root = os.path.dirname(os.path.abspath(__file__))
+    data_files = []
+    for dirname, files in files_spec:
+        resfiles = []
+        for fn in files:
+            if not os.path.exists(fn):
+                warnings.warn('Skipping file %s since it is not present. Type  make  to build all automatically generated files.' % fn)
+            else:
+                resfiles.append(fn)
+        data_files.append((dirname, resfiles))
+
     params = {
     params = {
-        'data_files': [  # Installing system-wide would require sudo...
-            ('etc/bash_completion.d', ['youtube-dl.bash-completion']),
-            ('share/doc/youtube_dl', ['README.txt']),
-            ('share/man/man1', ['youtube-dl.1'])
-        ]
+        'data_files': data_files,
     }
     }
     if setuptools_available:
     if setuptools_available:
         params['entry_points'] = {'console_scripts': ['youtube-dl = youtube_dl:main']}
         params['entry_points'] = {'console_scripts': ['youtube-dl = youtube_dl:main']}