Skip to content

Generalize dependencies.sh - #7

Open
gkielian wants to merge 1 commit into
alibillalhammoud:mainfrom
gkielian:generalize_dependencies_script
Open

Generalize dependencies.sh#7
gkielian wants to merge 1 commit into
alibillalhammoud:mainfrom
gkielian:generalize_dependencies_script

Conversation

@gkielian

@gkielian gkielian commented Nov 10, 2023

Copy link
Copy Markdown

Utilizing ${HOME} in place of /home/$(logname), for home directory. This will make the script more general, working on computers where the home directory path is not /home/$(logname).

Formatted spaces to 4 spaces, using spaces instead of tabs.

Removed trailing forwardslash from paths.

Added double quotes "" for bash to guard against word splitting and globbing in the case that any variables have wildcard or space characters.

Utilizing ${HOME} in place of /home/$(logname), for home directory.
This will make the script more general.

Formatted spaces to 4 spaces, using spaces instead of tabs.

Removed trailing forwardslash from paths.

Added double quotes "" for bash to guard against word splitting and
globbing in the case that any variables have wildcard or space
characters.
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.

1 participant