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

Reduce the number of binaries needed to be added to the path #96

Open
johanhelsing opened this issue Dec 30, 2015 · 3 comments
Open

Reduce the number of binaries needed to be added to the path #96

johanhelsing opened this issue Dec 30, 2015 · 3 comments

Comments

@johanhelsing
Copy link

Adding the whole repo to the path (as per the installation instructions) litters the path with many binaries that have names that are seeminlgy unrelated to git-radar.

  • fetch.sh
  • radar-base.sh
  • test-branches.sh
  • test-colors.sh
  • test-commits.sh
  • test-directories.sh
  • test-files.sh
  • test-format-config.sh
  • test-performance.sh
  • test-radar-base.sh
  • test-stash.sh
  • test-status.sh

It would be nice if this could be changed so git-radar only adds a single binary to the path, or at least if the scripts are renamed so it's obvious that they belong to git-radar. i.e. git-radar-fetch.sh

Possibly related to #51 #10

@srstevenson
Copy link
Contributor

Following UNIX convention (e.g. hier(7)), binaries executed by the user (only git-radar here) should be installed to $prefix/bin, whilst the helpers executed by that binary go in $prefix/libexec.

@michaeldfallen
Copy link
Owner

Putting only git-radar in /bin and the source scripts in /libexec is exactly what the homebrew release does. Since I was on OSX it was the only release process I really cared about / was able to properly dogfood.

I'm now on Ubuntu as my daily driver and in need of finding a better release approach for linux. FPM seems the way to go.

So the outcome of all this is:
Yeah, I agree. I'm looking in to it and would like to fix it soon.

Sorry that's not more helpful.

@djmattyg007
Copy link

It's worth mentioning that /usr/libexec is now merged with /usr/lib on most distros.

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

No branches or pull requests

4 participants