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

removed data_files param #672

Closed
wants to merge 1 commit into from
Closed

removed data_files param #672

wants to merge 1 commit into from

Conversation

viniciuscainelli
Copy link

always returning:
error: can't copy 'youtube-dl.bash-completion': doesn't exist or not a regular file

always returning:
error: can't copy 'youtube-dl.bash-completion': doesn't exist or not a regular file
@ghost ghost assigned phihag Feb 6, 2013
@phihag
Copy link
Contributor

phihag commented Feb 6, 2013

I'll have a look what's wrong here. Simply removing is unlikely to be the right choice though.

@jaimeMF
Copy link
Collaborator

jaimeMF commented Feb 6, 2013

I think the problem is that in the compressed archive provided on pypi, youtube-dl.bash-completion is not present (the same happens for the manpage youtube-dl.1).

@phihag
Copy link
Contributor

phihag commented Feb 18, 2013

Should be fixed with 471cf47 and release 2013.02.19.

@phihag phihag closed this Feb 18, 2013
@ru2nuts
Copy link

ru2nuts commented Feb 21, 2013

sorry, I think I am still getting the same error with 2013.02.19.

@phihag phihag reopened this Feb 21, 2013
@phihag
Copy link
Contributor

phihag commented Feb 21, 2013

@ru2nuts On what operating system are you on? And can you give us the error output you get? It works fine on my debian sid x64:

# pip install youtube-dl
Downloading/unpacking youtube-dl
  Downloading youtube_dl-2013.02.19.tar.gz (77Kb): 77Kb downloaded
  Running setup.py egg_info for package youtube-dl

Installing collected packages: youtube-dl
  Running setup.py install for youtube-dl
    changing mode of build/scripts-2.7/youtube-dl from 644 to 755

    changing mode of /usr/local/bin/youtube-dl to 755
Successfully installed youtube-dl
Cleaning up...

@ru2nuts
Copy link

ru2nuts commented Feb 22, 2013

Not using "pip", running "setup.py install" on CentOS 5.8:

$ cat /etc/*-release
CentOS release 5.8 (Final)


$sudo python setup.py install
running install
running bdist_egg
running egg_info
...
Processing dependencies for zzz==0.1
Searching for youtube-dl-dev
Best match: youtube-dl-dev [unknown version]
Downloading http://github.com/rg3/youtube-dl/tarball/2013.02.19#egg=youtube_dl-dev
Processing 2013.02.19
Writing /tmp/easy_install-b4D17R/rg3-youtube-dl-540b9af/setup.cfg
Running rg3-youtube-dl-540b9af/setup.py -q bdist_egg --dist-dir /tmp/easy_install-b4D17R/rg3-youtube-dl-540b9af/egg-dist-tmp-qHZtG3
warning: no files found matching 'youtube-dl.bash-completion'
warning: no files found matching 'youtube-dl.1'
error: Setup script exited with error: can't copy 'youtube-dl.bash-completion': doesn't exist or not a regular file

@ru2nuts
Copy link

ru2nuts commented Feb 22, 2013

@phihag the issue only occurred when I used direct

dependency_links=['http://github.com/rg3/youtube-dl/tarball/2013.02.19#egg=youtube_dl-dev']

in my setup.py
I switched to direct "install_requires" and it works.

@phihag
Copy link
Contributor

phihag commented Feb 22, 2013

@ru2nuts Thanks for the report. Can you post a link (or the contents of) the problematic setup.py? I'll track down the issue then.

@ru2nuts
Copy link

ru2nuts commented Feb 25, 2013

@phihag Not critical for me, since I am now using:

    install_requires=['youtube_dl']

but below is my setup.py with "dependency_links" that failed before:

import os
from setuptools import setup, find_packages

setup(
    name="test",
    version=0.1,
    packages=find_packages(),
    dependency_links=['http://github.com/rg3/youtube-dl/tarball/2013.02.19#egg=youtube_dl-dev'],
    install_requires=[
        'setuptools',
        'youtube_dl-dev'
    ]
)

@phihag
Copy link
Contributor

phihag commented Jan 10, 2015

This should be fixed by now. I apologize for the delay.

@phihag phihag closed this Jan 10, 2015
@ru2nuts
Copy link

ru2nuts commented Jan 13, 2015

Thanks!

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

Successfully merging this pull request may close these issues.

None yet

4 participants