Permalink
Browse files

Fix load_some() installation to handle file paths instead of names.

The ls command in for loop has a pattern now, so full paths are
returned. The user prompt and symbolic link have been updated to handle
this.

Also, not all plugins have '.plugin.' in the middle of their names and
weren't showing up on user prompt. With this commit, everything after
the first period in the file name is stripped.
  • Loading branch information...
1 parent 54680bd commit 76326490ece46a0b78254353855584e31764c9f8 Greg Nofi committed May 15, 2012
Showing with 4 additions and 3 deletions.
  1. +4 −3 install.sh
View
@@ -36,18 +36,19 @@ function load_all() {
function load_some() {
file_type=$1
- for file in `ls $BASH_IT/${file_type}/available/[^_]*`
+ for path in `ls $BASH_IT/${file_type}/available/[^_]*`
do
if [ ! -d "$BASH_IT/$file_type/enabled" ]
then
mkdir "$BASH_IT/$file_type/enabled"
fi
+ file_name=$(basename "$path")
while true
do
- read -p "Would you like to enable the ${file%.*.*} $file_type? [Y/N] " RESP
+ read -p "Would you like to enable the ${file_name%%.*} $file_type? [Y/N] " RESP
case $RESP in
[yY])
- ln -s "$BASH_IT/$file_type/available/$file" "$BASH_IT/$file_type/enabled"
+ ln -s "$path" "$BASH_IT/$file_type/enabled"
break
;;
[nN])

0 comments on commit 7632649

Please sign in to comment.