Date: Fri, 01 Jun 2018 16:22:01 +0000 From: bugzilla-noreply@freebsd.org To: ports-bugs@FreeBSD.org Subject: [Bug 227109] Mk/Scripts, cleanup using devel/hs-ShellCheck Message-ID: <bug-227109-7788-v8vdqEDpp6@https.bugs.freebsd.org/bugzilla/> In-Reply-To: <bug-227109-7788@https.bugs.freebsd.org/bugzilla/> References: <bug-227109-7788@https.bugs.freebsd.org/bugzilla/>
next in thread | previous in thread | raw e-mail | index | archive | help
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D227109 --- Comment #13 from commit-hook@freebsd.org --- A commit references this bug: Author: mat Date: Fri Jun 1 16:20:42 UTC 2018 New revision: 471270 URL: https://svnweb.freebsd.org/changeset/ports/471270 Log: SC2091: Remove surrounding $() to avoid executing output. ShellCheck has detected that you have a command that just consists of a command substitution. This is typically done in order to try to get the shell to execute a command, because $(..) does indeed execute commands. However, it's also replaced by the output of that command. PR: 227109 Submitted by: mat Sponsored by: Absolight Changes: head/Mk/Scripts/qa.sh --=20 You are receiving this mail because: You are on the CC list for the bug.=
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?bug-227109-7788-v8vdqEDpp6>