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

`setup.cfg` not included in release tarball #14857

Closed
ghost opened this issue Nov 26, 2017 · 1 comment
Closed

`setup.cfg` not included in release tarball #14857

ghost opened this issue Nov 26, 2017 · 1 comment

Comments

@ghost
Copy link

@ghost ghost commented Nov 26, 2017

Please follow the guide below

  • You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
  • Put an x into all the boxes [ ] relevant to your issue (like this: [x])
  • Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2017.11.26. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

  • I've verified and I assure that I'm running youtube-dl 2017.11.26

Before submitting an issue make sure you have:

  • At least skimmed through the README, most notably the FAQ and BUGS sections
  • Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

  • Bug report (encountered problems with youtube-dl)
  • Site support request (request for adding support for a new site)
  • Feature request (request for a new functionality)
  • Question
  • Other

setup.cfg is missing in the release tarball youtube-dl-2017-11-26.tar.gz. This lets the make target codetest fail with lots of errors (all of which should be ignored according to setup.cfg).

Suggested fix: Add setup.cfg to the tarball.

@yan12125
Copy link
Collaborator

@yan12125 yan12125 commented Nov 27, 2017

Done.

@yan12125 yan12125 closed this in dafb4c6 Nov 27, 2017
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
1 participant
You can’t perform that action at this time.