[Bug 227109] Mk/Scripts, cleanup using devel/hs-ShellCheck

bugzilla-noreply at freebsd.org bugzilla-noreply at freebsd.org
Fri Jun 1 16:22:11 UTC 2018


https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=227109

--- Comment #17 from commit-hook at freebsd.org ---
A commit references this bug:

Author: mat
Date: Fri Jun  1 16:20:55 UTC 2018
New revision: 471274
URL: https://svnweb.freebsd.org/changeset/ports/471274

Log:
  SC2006: Use $(..) instead of legacy `..`.

  Backtick command substitution `STATEMENT` is legacy syntax with several
  issues.

   - It has a series of undefined behaviors related to quoting in POSIX.
   - It imposes a custom escaping mode with surprising results.
   - It's exceptionally hard to nest.

  $(STATEMENT) command substitution has none of these problems, and is
  therefore strongly encouraged.

  PR:           227109
  Submitted by: mat
  Sponsored by: Absolight

Changes:
  head/Mk/Scripts/actual-package-depends.sh
  head/Mk/Scripts/create-manifest.sh
  head/Mk/Scripts/find-lib.sh
  head/Mk/Scripts/qa.sh

-- 
You are receiving this mail because:
You are on the CC list for the bug.


More information about the freebsd-ports-bugs mailing list