I've noticed the devel/git/Makefile doesn't link git-prompt.sh to /usr/local/etc/bash_completion.d. Further, it links git-completion.bash to /usr/local/etc/bash_completion.d/git.sh.
# l /usr/local/etc/bash_completion.d/
drwxr-xr-x 2 root wheel 6 Sep 22 14:13 .
drwxr-xr-x 35 root wheel 57 Sep 28 00:50 ..
-rw-r--r-- 1 root wheel 19809 Sep 12 18:33 _pkg.bash
-r-xr-xr-x 1 root wheel 559 Sep 12 18:38 dbus-bash-completion.sh
lrwxr-xr-x 1 root wheel 59 Sep 19 18:46 git.sh -> ../../share/git-core/contrib/completion/git-completion.bash
-rw-r--r-- 1 root wheel 4603 Sep 12 18:47 tmux
This appears a non-standard approach. Homebrew on OSX links them both using their original filenames, for example. Could you please comment? There's also the csh and zsh completion files, but those are outside my scope.
FreeBSD 11.2, git 2.19.
(In reply to pierogmorski from comment #0)
Both git-prompt.sh and git-completion.bash are available under /usr/local/share/git-core/contrib/completion/ when CONTRIB option is selected.
git-prompt.sh is a script to help users to configure bash's prompt to show git information, it's not a completion related script so IMO it doesn't belong to /usr/local/etc/bash_completion.d
git-completion.bash is installed there as git.sh for a long time, before I adopted this port. I never changed that to avoid breaking users' configuration.
I agree it's non standard so I'll work on a change to install it keeping original name and add a notice in pkg-message.
A commit references this bug:
Date: Fri Sep 28 12:48:38 UTC 2018
New revision: 480858
devel/git: Keep original bash completion filename
Do not rename git-completion.bash to git.sh to make it easier for users
that follow git docs, which point to original filename. Add a note in
pkg-message to let users know about it.
Submitted by: firstname.lastname@example.org
Sponsored by: Rubicon Communications, LLC (Netgate)
(In reply to Renato Botelho from comment #1)
First off, thanks for the blindingly quick turnaround with this.
You make a reasonable argument re: git-prompt.sh. Agreed that it isn't actually part of the bash completion feature.