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

ZSH completion not currently included in PyPI releases #25739

Closed
AndydeCleyre opened this issue Jun 20, 2020 · 2 comments
Closed

ZSH completion not currently included in PyPI releases #25739

AndydeCleyre opened this issue Jun 20, 2020 · 2 comments
Labels

Comments

@AndydeCleyre
Copy link

@AndydeCleyre AndydeCleyre commented Jun 20, 2020

Checklist

  • I'm reporting a broken site support issue
  • I've verified that I'm running youtube-dl version 2020.06.16.1
  • I've checked that all provided URLs are alive and playable in a browser
  • I've checked that all URLs and arguments with special characters are properly quoted or escaped
  • I've searched the bugtracker for similar bug reports including closed ones
  • I've read bugs section in FAQ

Verbose log

This is not relevant, but I'm including it to ensure satisfaction of the general bug reporting guidelines:

$ youtube-dl -v
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] youtube-dl version 2020.06.16.1
[debug] Python version 3.8.3 (CPython) - Linux-5.7.4-zen1-1-zen-x86_64-with-glibc2.2.5
[debug] exe versions: ffmpeg 4.2.3, ffprobe 4.2.3, rtmpdump 2.4
[debug] Proxy map: {}
Usage: youtube-dl [OPTIONS] URL [URL...]

youtube-dl: error: You must provide at least one URL.
Type youtube-dl --help to see a list of all options.

Description

Folks who install from source, or via built packages for their distro, are able to enjoy the ZSH completions you provide.

While those who install via PyPI are able to access the fish and bash completions, so that they can copy/link/add them to their shell manually, they are not provided with the ZSH completions with which to do the same. Neither are they provided with the Makefile+template with which to generate these completions.

I'm requesting that the generated ZSH completions be included in the PyPI build (as the generated completion doesn't seem to be platform independent or anything).

The exact location of the file does not matter to me personally, as I'm installing as a user and within a venv, but FWIW on Arch Linux the default $fpath includes /usr/share/zsh/site-functions.

The change would affect at least three files:

Makefile, which currently includes:

pypi-files: youtube-dl.bash-completion README.txt youtube-dl.1 youtube-dl.fish

setup.py, which currently includes:

    files_spec = [
        ('etc/bash_completion.d', ['youtube-dl.bash-completion']),
        ('etc/fish/completions', ['youtube-dl.fish']),
        ('share/doc/youtube_dl', ['README.txt']),
        ('share/man/man1', ['youtube-dl.1'])
    ]

And MANIFEST.in.

@dstftw dstftw closed this Jun 20, 2020
@dstftw dstftw added the duplicate label Jun 20, 2020
@AndydeCleyre
Copy link
Author

@AndydeCleyre AndydeCleyre commented Jun 20, 2020

I searched for zsh pypi -- which issue is this a duplicate of?

@AndydeCleyre
Copy link
Author

@AndydeCleyre AndydeCleyre commented Jun 20, 2020

Ah, found it. Linking here: #9231

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
2 participants
You can’t perform that action at this time.