Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 839234 - sci-physics/root: automated shellcheck found errors
Summary: sci-physics/root: automated shellcheck found errors
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Guilherme Amadio
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-04-18 11:06 UTC by shellbugbot
Modified: 2022-05-23 15:21 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description shellbugbot 2022-04-18 11:06:08 UTC
Shellckeck found errors in one or more shellscripts provided by sci-physics/root.

Kindly refer to the findings below and the shellcheck wiki for further details:

------------------------------------------------
script: /usr/portage/sci-physics/root/root-9999.ebuild:171

		-DCMAKE_CXX_STANDARD=$((usev c++11 || usev c++14 || usev c++17) | cut -c4-)
                                     
error: Shells disambiguate $(( differently or not at all. For $(command substitution), add space after $( . For $((arithmetics)), fix parsing errors. (https://github.com/koalaman/shellcheck/wiki/SC1102)

------------------------------------------------
script: /usr/portage/sci-physics/root/root-6.26.02.ebuild:160

		-DCMAKE_CXX_STANDARD=$((usev c++14 || usev c++17) | cut -c4-)
                                     
error: Shells disambiguate $(( differently or not at all. For $(command substitution), add space after $( . For $((arithmetics)), fix parsing errors. (https://github.com/koalaman/shellcheck/wiki/SC1102)

------------------------------------------------
script: /usr/portage/sci-physics/root/root-6.24.06-r2.ebuild:160

		-DCMAKE_CXX_STANDARD=$((usev c++11 || usev c++14 || usev c++17) | cut -c4-)
                                     
error: Shells disambiguate $(( differently or not at all. For $(command substitution), add space after $( . For $((arithmetics)), fix parsing errors. (https://github.com/koalaman/shellcheck/wiki/SC1102)

------------------------------------------------
script: /usr/portage/sci-physics/root/root-6.26.9999.ebuild:171

		-DCMAKE_CXX_STANDARD=$((usev c++14 || usev c++17) | cut -c4-)
                                     
error: Shells disambiguate $(( differently or not at all. For $(command substitution), add space after $( . For $((arithmetics)), fix parsing errors. (https://github.com/koalaman/shellcheck/wiki/SC1102)

This is an automated bug report. Conservative settings were used to minimize false-positives but there's still a chance that the above list contains non-issues.
Running shellcheck again with default settings could detect other possible issues.

References:
[1] this script: https://github.com/heeplr/gentoo-shellbug
[2] shellcheck: https://www.shellcheck.net/
Comment 1 Larry the Git Cow gentoo-dev 2022-05-23 15:21:00 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=10ac46020de851a183cfe6eccfa3f09c7fdaf68a

commit 10ac46020de851a183cfe6eccfa3f09c7fdaf68a
Author:     Guilherme Amadio <amadio@gentoo.org>
AuthorDate: 2022-05-23 15:19:27 +0000
Commit:     Guilherme Amadio <amadio@gentoo.org>
CommitDate: 2022-05-23 15:19:35 +0000

    sci-physics/root: fix bug 839234
    
    Closes: https://bugs.gentoo.org/839234
    Package-Manager: Portage-3.0.30, Repoman-3.0.3
    Signed-off-by: Guilherme Amadio <amadio@gentoo.org>

 sci-physics/root/root-6.24.06-r2.ebuild | 2 +-
 sci-physics/root/root-6.26.02.ebuild    | 2 +-
 sci-physics/root/root-6.26.9999.ebuild  | 2 +-
 sci-physics/root/root-9999.ebuild       | 2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)