ajeetdsouza / zoxide

A smarter cd command. Supports all major shells.
MIT License
20.54k stars 523 forks source link

New installer options (v2) #672

Closed aarondill closed 2 months ago

aarondill commented 5 months ago

This is a rewrite of #590 to reduce the diff and make it a more reasonable pr.

Add support for options in install script

This adds support for serveral new command line options (taken from the usage):

aarondill commented 5 months ago

@ajeetdsouza I hope I've accomplished what you expected when we first discussed this (almost a year ago) here. If anything is missing, or you have any changes you'd like made, please let me know :smile:

rseichter commented 2 months ago

I used the official install script for the first time today, because of the outdated Debian 11 package. Right away I was surprised by the hard-coded $HOME/.local/{bin,man} directories. While this was easy enough to fix manually, I would appreciate being able to pass the desired directories (or in my case a directory prefix /usr/local) as either a installer script argument or environment variable. I checked the list of Zoxide pull requests and found this one, so I'd like to vote in favour.

rseichter commented 2 months ago

Thank you for the new configuration options. They make automation easier for me, and I have already updated my jobs accordingly.

ajeetdsouza commented 2 months ago

Thank you @aarondill for the PR.