Skip to content

Instantly share code, notes, and snippets.

@outro56
Created September 4, 2015 01:31
Show Gist options
  • Save outro56/4a2403ae8fefdeb832a5 to your computer and use it in GitHub Desktop.
Save outro56/4a2403ae8fefdeb832a5 to your computer and use it in GitHub Desktop.
Bash scripting best practices (copied from https://github.com/progrium/bashstyle/blob/master/README.md)

progrium/bashstyle

Bash is the JavaScript of systems programming. Although in some cases it's better to use a systems language like C or Go, Bash is an ideal systems language for smaller POSIX-oriented or command line tasks. Here's three quick reasons why:

  • It's everywhere. Like JavaScript for the web, Bash is already there ready for systems programming.
  • It's neutral. Unlike Ruby, Python, JavaScript, or PHP, Bash offends equally across all communities. ;)
  • It's made to be glue. Write complex parts in C or Go (or whatever!), and glue them together with Bash.

This document is how I write Bash and how I'd like collaborators to write Bash with me in my open source projects. It's based on a lot of experience and time collecting best practices. Most of them come from these two articles, but here integrated, slightly modified, and focusing on the most bang for buck items. Plus some new stuff!

Keep in mind this is not for general shell scripting, these are rules specifically for Bash and can take advantage of assumptions around Bash as the interpreter.

Big Rules

  • Always double quote variables, including subshells. No naked $ signs
  • All code goes in a function. Even if it's one function, main.
    • Unless a library script, you can do global script settings and call main. That's it.
    • Avoid global variables. Though when defining constants use readonly
  • Always have a main function for runnable scripts, called with main or main "$@"
    • If script is also usable as library, call it using [[ "$0" == "$BASH_SOURCE" ]] && main "$@"
  • Always use local when setting variables, unless there is reason to use declare
    • Exception being rare cases when you are intentionally setting a variable in an outer scope.
  • Variable names should be lowercase unless exported to environment.
  • Always use set -eo pipefail. Fail fast and be aware of exit codes.
    • Use || true on programs that you intentionally let exit non-zero.
  • Never use deprecated style. Most notably:
  • Prefer absolute paths (leverage $PWD), always qualify relative paths with ./.
  • Always use declare and name variable arguments at the top of functions that are more than 2-lines
    • Example: declare arg1="$1" arg2="$2"
    • The exception is when defining variadic functions. See below.
  • Use mktemp for temporary files, always cleanup with a trap.
  • Warnings and errors should go to STDERR, anything parsable should go to STDOUT.
  • Try to localize shopt usage and disable option when finished.

If you know what you're doing, you can bend or break some of these rules, but generally they will be right and be extremely helpful.

Best Practices and Tips

  • Use Bash variable substitution if possible before awk/sed.
  • Generally use double quotes unless it makes more sense to use single quotes.
  • For simple conditionals, try using && and ||.
  • Don't be afraid of printf, it's more powerful than echo.
  • Put then, do, etc on same line, not newline.
  • Skip [[ ... ]] in your if-expression if you can test for exit code instead.
  • Use .sh or .bash extension if file is meant to be included/sourced. Never on executable script.
  • Put complex one-liners of sed, perl, etc in a standalone function with a descriptive name.
  • Good idea to include [[ "$TRACE" ]] && set -x
  • Design for simplicity and obvious usage.
    • Avoid option flags and parsing, try optional environment variables instead.
    • Use subcommands for necessary different "modes".
  • In large systems or for any CLI commands, add a description to functions.
    • Use declare desc="description" at the top of functions, even above argument declaration.
    • This can be queried/extracted with a simple function using reflection.
  • Be conscious of the need for portability. Bash to run in a container can make more assumptions than Bash made to run on multiple platforms.
  • When expecting or exporting environment, consider namespacing variables when subshells may be involved.
  • Use hard tabs. Heredocs ignore leading tabs, allowing better indentation.

Good References and Help

Examples

Regular function with named arguments

Defining functions with arguments

regular_func() {
	declare arg1="$1" arg2="$2" arg3="$3"

	# ...
}

Variadic functions

Defining functions with a final variadic argument

variadic_func() {
	local arg1="$1"; shift
	local arg2="$1"; shift
	local rest="$@"

	# ...
}

Conditionals: Testing for exit code vs output

# Test for exit code (-q mutes output)
if grep -q 'foo' somefile; then
  ...
fi

# Test for output (-m1 limits to one result)
if [[ "$(grep -m1 'foo' somefile)" ]]; then
  ...
fi

More todo

@dagelf
Copy link

dagelf commented Aug 29, 2024

It's so obsolete, the obsoletion notice died during Covid... http://web.archive.org/web/20201205211834/http://wiki.bash-hackers.org/scripting/obsolete

Really, backwards compatibility is one of the greatest inventions ever... never obsolete anything, because, why reinvent the wheel? Do we really not have anything better to do?! Although.....

Maybe "obsolete" isn't the right word, too triggering... maybe it would be better to say, did you know that there are better ways of doing things? Aren't you tired of escaping quotes?! 😄 http://web.archive.org/web/20201112023011/https://mywiki.wooledge.org/BashFAQ/082

@dagelf
Copy link

dagelf commented Aug 29, 2024

Also note that your references (great resources! Thanks!) -- advise against some of your practices! ☮️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment