Fix a bug in pre-commit-copyright and other shell script fixes.

[KineticTheory]

Background

• @along reported that the pre-commit-copyright script fails when /tmp/$USER doesn't exist.
• Also provide a couple of small changes to environments/shell scripts.

Description of changes

• Pre-commit hooks and check_style.sh expect the path /tmp/$USER to exist. Create this path if it doesn't already exist.
• For the copyright block check, also check shell scripts (extension .sh), but exclude check_style.sh as the script logic gets confused for this file (too many Copyright strings).
• Add bash function rdp (runs xfreerdp). This is useful for running desktop-on-demand Win10 sessions from the Cortez network. Also, cleanup comment formatting in bash_functions.sh.

Status

• Reference:
  • Pre-Merge Code Review
  • CDash Status
• Checks
  • [x] Travis/Appveyor CI checks pass
  • [x] Code coverage does not decrease
  • [x] Valgrind test passes
  • [ ] Toss3 checks pass
  • [ ] Trinitite checks pass
  • [x] Code reviewed/approved, sufficient DbC checks, testing, documentation

[codecov[bot]]

Codecov Report

Merging #1116 (6fa6793) into develop (605e1e1) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##           develop   #1116   +/-   ##
=======================================
  Coverage     89.0%   89.0%           
=======================================
  Files          373     373           
  Lines        19573   19573           
=======================================
  Hits         17420   17420           
  Misses        2153    2153