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

List more kinds of possible packages/documents/aliases in zsh completion #123

Merged
merged 1 commit into from
Feb 1, 2025

Conversation

mgkurtz
Copy link
Contributor

@mgkurtz mgkurtz commented Jan 31, 2025

Thanks a lot for providing a zsh completion ❤️ I love shell completions and so I improved it (at least from my point of view) a little further by enlarging the list of package and document names, the completion script suggests.

The current completion uses the package names from $TEXMFROOT/texlive.tlpdb, this is usually nice, but with TeX-Live installed as Debian package, this texlive.tlpdb does not even exist 😕 Hence, I added five more sources of which at least some should exist and provide helpful completions:

  • Package names listed in a local $TEXMFHOME/texlive.tlpdb.
  • If texlive.tlpdb exists neither locally, nor system wide: fallback to texdoc’s own Data.tlpdb.lua.
  • PDF files listed in the ls-R file.
  • Aliases registered in texdoc-dist.cnf and texdoc.cnf.
  • Any files found in $TEXMFHOME/doc.

I hope you like it and feel free to criticize 😀

@wtsnjp
Copy link
Member

wtsnjp commented Feb 1, 2025

Thanks for the suggestion! Looks good to me.

@Freed-Wu if you have time, could you please look at the changes in the zsh completion function and share your thoughts with us?

@Freed-Wu
Copy link
Contributor

Freed-Wu commented Feb 1, 2025

Screenshot_20250201_121839

  • above: PR
  • below: original

LGTM

@wtsnjp wtsnjp merged commit 9535c1f into TeX-Live:master Feb 1, 2025
3 checks passed
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.

3 participants