github/shellcheck
1
0
Fork 0
mirror of https://github.com/koalaman/shellcheck synced 2025-08-22 14:23:53 -07:00
Code Issues Projects Releases Packages Wiki Activity Actions

Disable SC2155 for portage files

Browse source 
This idiom is extremely common in Gentoo:
In tensorflow-1.9.0_rc1.ebuild line 400:
                export GCC_HOST_COMPILER_PATH="$(which $(tc-getBUILD_CC))"
                       ^--------------------^ SC2155: Declare and assign separately to avoid masking return values.

Gentoo shell scripts don't run under `set -e`, and no one ever runs code
like `if export ...`, so this warning doesn't help with ebuilds.
This commit is contained in:
Benjamin Gordon 2019-07-02 12:24:11 -06:00
commit a313b48ca6
1 changed files with 5 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

8
src/ShellCheck/Analytics.hs
View file

@ -2745,10 +2745,12 @@ prop_checkMaskedReturns2 = verify checkMaskedReturns "declare a=$(false)"
prop_checkMaskedReturns3 = verify checkMaskedReturns "declare a=\"`false`\""
prop_checkMaskedReturns4 = verifyNot checkMaskedReturns "declare a; a=$(false)"
prop_checkMaskedReturns5 = verifyNot checkMaskedReturns "f() { local -r a=$(false); }"
checkMaskedReturns _ t@(T_SimpleCommand id _ (cmd:rest)) = potentially $ do
prop_checkMaskedReturns6 = verifyNot checkMaskedReturns "#shellcheck shell=portage\ndeclare a=$(false)"
checkMaskedReturns params t@(T_SimpleCommand id _ (cmd:rest)) = potentially $ do
name <- getCommandName t
guard $ name `elem` ["declare", "export"]
|| name == "local" && "r" `notElem` map snd (getAllFlags t)
guard $ (name `elem` ["declare", "export"]
|| name == "local" && "r" `notElem` map snd (getAllFlags t))
&& (not $ isPortageBuild params)
return $ mapM_ checkArgs rest
where
checkArgs (T_Assignment id _ _ _ word) | any hasReturn $ getWordParts word =