Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 839237 - sci-physics/geant: automated shellcheck found errors
Summary: sci-physics/geant: 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
Assignee: Guilherme Amadio
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-04-18 11:06 UTC by shellbugbot
Modified: 2022-05-23 15:20 UTC (History)
0 users

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/geant.

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

------------------------------------------------
script: /usr/portage/sci-physics/geant/geant-4.10.7.3.ebuild:70

		-DGEANT4_BUILD_CXXSTD=$((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/geant/geant-4.11.0.1.ebuild:81

		-DCMAKE_CXX_STANDARD=$((usev c++17 || usev c++20) | 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/geant/geant-4.11.0.0.ebuild:83

		-DCMAKE_CXX_STANDARD=$((usev c++17 || usev c++20) | 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:20:59 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=0e8f9e7b0f439e7f85c099b8bbd1e3d3ecbb0e92

commit 0e8f9e7b0f439e7f85c099b8bbd1e3d3ecbb0e92
Author:     Guilherme Amadio <amadio@gentoo.org>
AuthorDate: 2022-05-23 15:11:02 +0000
Commit:     Guilherme Amadio <amadio@gentoo.org>
CommitDate: 2022-05-23 15:17:15 +0000

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

 sci-physics/geant/geant-4.10.7.3.ebuild | 4 ++--
 sci-physics/geant/geant-4.11.0.0.ebuild | 4 ++--
 sci-physics/geant/geant-4.11.0.1.ebuild | 2 +-
 3 files changed, 5 insertions(+), 5 deletions(-)