Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

sudo python setup.py install is broken on Linux #2242

Closed
lyda opened this issue Jan 26, 2014 · 2 comments
Closed

sudo python setup.py install is broken on Linux #2242

lyda opened this issue Jan 26, 2014 · 2 comments

Comments

@lyda
Copy link

@lyda lyda commented Jan 26, 2014

This bit of setup.py:

    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'])
        ]
    }

should probably be:

    params = {
        'data_files': [  # Installing system-wide would require sudo...
            ('share/doc/youtube_dl', ['README.md']),
        ]
    }
@jaimeMF
Copy link
Collaborator

@jaimeMF jaimeMF commented Jan 26, 2014

Without any output of running the command I guess you have the same problem as in #780, therefore I close it as a duplicate.
Downloading the source form pypi or from https://yt-dl.org and running the setup.py will work. The reason is that youtube-dl.bash-completion, README.txt and youtube-dl.1 are not included in the git repo, they are created when releasing new version and added to the source that it's distributed.
You can create them yourself, by running make youtube-dl.bash-completion youtube-dl.1 README.txt from the source directory, but you will need to have pandoc installed. Then running python setup.py will work.

@jaimeMF jaimeMF closed this Jan 26, 2014
@phihag
Copy link
Contributor

@phihag phihag commented Jan 27, 2014

This should now be fixed in the current git version, where we only emit a warning if the files are missing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.