lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Message-ID: <YvReL0HkZt639BoO@Laptop-X1> Date: Thu, 11 Aug 2022 09:41:03 +0800 From: Hangbin Liu <liuhangbin@...il.com> To: Siddh Raman Pant <code@...dh.me> Cc: Piyush Thange <pthange19@...il.com>, davem <davem@...emloft.net>, edumazet <edumazet@...gle.com>, kuba <kuba@...nel.org>, pabeni <pabeni@...hat.com>, shuah <shuah@...nel.org>, "vladimir.oltean" <vladimir.oltean@....com>, idosch <idosch@...dia.com>, petrm <petrm@...dia.com>, troglobit <troglobit@...il.com>, amcohen <amcohen@...dia.com>, tobias <tobias@...dekranz.com>, "po-hsu.lin" <po-hsu.lin@...onical.com>, netdev <netdev@...r.kernel.org>, linux-kselftest <linux-kselftest@...r.kernel.org>, linux-kernel <linux-kernel@...r.kernel.org>, linux-kernel-mentees <linux-kernel-mentees@...ts.linuxfoundation.org> Subject: Re: [PATCH] selftests:net:forwarding: Included install command On Wed, Aug 10, 2022 at 03:23:15PM +0530, Siddh Raman Pant wrote: > On Wed, 10 Aug 2022 15:05:08 +0530 Piyush Thange <pthange19@...il.com> wrote: > > If the execution is skipped due to "jq not installed" message then > > the installation methods on different OS's have been provided with > > this message. > > > > Signed-off-by: Piyush Thange <pthange19@...il.com> > > --- > > tools/testing/selftests/net/forwarding/lib.sh | 8 ++++++++ > > 1 file changed, 8 insertions(+) > > > > diff --git a/tools/testing/selftests/net/forwarding/lib.sh b/tools/testing/selftests/net/forwarding/lib.sh > > index 37ae49d47853..c4121856fe06 100755 > > --- a/tools/testing/selftests/net/forwarding/lib.sh > > +++ b/tools/testing/selftests/net/forwarding/lib.sh > > @@ -152,6 +152,14 @@ require_command() > > > > if [[ ! -x "$(command -v "$cmd")" ]]; then > > echo "SKIP: $cmd not installed" > > + if [[ $cmd == "jq" ]]; then > > + echo " Install on Debian based systems" > > + echo " sudo apt -y install jq" > > + echo " Install on RHEL based systems" > > + echo " sudo yum -y install jq" > > + echo " Install on Fedora based systems" > > + echo " sudo dnf -y install jq" > > + fi > > exit $ksft_skip > > fi > > } > > -- > > 2.37.1 > > This is very specific to `jq` command. What's special with `jq` and not > others? If methods have to be shown, they should be shown for all the > programs which are not installed. Agree. The user could decide if jq should be install via REQUIRE_JQ. There are also other cmds that vendor may not build by default. I didn't see any selftests need to handle the installation. The users should takes care of it. require_command() has takes care most of the needed cmds. If we want to improve the user's experience for the needed cmds. I think add the needed cmds to README file is better. Thanks Hangbin
Powered by blists - more mailing lists