* [PATCH 1/2] Documentation/scripts: fix shellcheck issues
@ 2022-11-15 10:56 Alyssa Ross
2022-11-15 10:56 ` [PATCH 2/2] nix/checks.nix: shellcheck more files Alyssa Ross
0 siblings, 1 reply; 4+ messages in thread
From: Alyssa Ross @ 2022-11-15 10:56 UTC (permalink / raw)
To: devel; +Cc: Henri Rosten
There's an open issue about teaching shellcheck to automatically
recognize nix-shell shebangs:
https://github.com/koalaman/shellcheck/issues/1210
Signed-off-by: Alyssa Ross <alyssa.ross@unikie.com>
---
Documentation/scripts/build.sh | 2 +-
Documentation/scripts/update-gems.sh | 3 +++
2 files changed, 4 insertions(+), 1 deletion(-)
diff --git a/Documentation/scripts/build.sh b/Documentation/scripts/build.sh
index 85bd5a5..e6a2cd8 100755
--- a/Documentation/scripts/build.sh
+++ b/Documentation/scripts/build.sh
@@ -4,7 +4,7 @@
cd "$(dirname "$0")/.."
-if [ ! -w . -a ! -w .jekyll-cache ]; then
+if [ ! -w . ] && [ ! -w .jekyll-cache ]; then
JEKYLLFLAGS=--disable-disk-cache
fi
diff --git a/Documentation/scripts/update-gems.sh b/Documentation/scripts/update-gems.sh
index 7ba98d8..d70f55f 100755
--- a/Documentation/scripts/update-gems.sh
+++ b/Documentation/scripts/update-gems.sh
@@ -1,8 +1,11 @@
#! /usr/bin/env nix-shell
#! nix-shell -i bash -p bundler bundix
+
# SPDX-FileCopyrightText: 2022 Unikie
# SPDX-License-Identifier: EUPL-1.2+
+# shellcheck shell=bash
+
set -euo pipefail
bundle lock --update
--
2.35.1
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH 2/2] nix/checks.nix: shellcheck more files
2022-11-15 10:56 [PATCH 1/2] Documentation/scripts: fix shellcheck issues Alyssa Ross
@ 2022-11-15 10:56 ` Alyssa Ross
2022-11-16 8:56 ` Henri Rosten
0 siblings, 1 reply; 4+ messages in thread
From: Alyssa Ross @ 2022-11-15 10:56 UTC (permalink / raw)
To: devel; +Cc: Henri Rosten
I didn't realise that bash's ** feature (which matches any depth of
directories) wasn't enabled by default, because it appeared to work,
just didn't find all scripts.
Signed-off-by: Alyssa Ross <alyssa.ross@unikie.com>
Fixes: 169fdd6 ("release.nix: run shellcheck on build scripts")
---
nix/checks.nix | 1 +
1 file changed, 1 insertion(+)
diff --git a/nix/checks.nix b/nix/checks.nix
index 9eb261f..b2af72a 100644
--- a/nix/checks.nix
+++ b/nix/checks.nix
@@ -17,6 +17,7 @@
nativeBuildInputs = [ shellcheck ];
} ''
+ shopt -s globstar
shellcheck $src/**/*.sh
touch $out
''
--
2.35.1
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 2/2] nix/checks.nix: shellcheck more files
2022-11-15 10:56 ` [PATCH 2/2] nix/checks.nix: shellcheck more files Alyssa Ross
@ 2022-11-16 8:56 ` Henri Rosten
2022-11-16 10:40 ` Alyssa Ross
0 siblings, 1 reply; 4+ messages in thread
From: Henri Rosten @ 2022-11-16 8:56 UTC (permalink / raw)
To: Alyssa Ross; +Cc: devel
On Tue, Nov 15, 2022 at 10:56:37AM +0000, Alyssa Ross wrote:
> I didn't realise that bash's ** feature (which matches any depth of
> directories) wasn't enabled by default, because it appeared to work,
> just didn't find all scripts.
I'm getting shellcheck warnings from
host/rootfs/build/etc/mdev/modalias.sh. Did you intentionally include
scripts under 'build' too, or should those be excluded?
>
> Signed-off-by: Alyssa Ross <alyssa.ross@unikie.com>
> Fixes: 169fdd6 ("release.nix: run shellcheck on build scripts")
> ---
> nix/checks.nix | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/nix/checks.nix b/nix/checks.nix
> index 9eb261f..b2af72a 100644
> --- a/nix/checks.nix
> +++ b/nix/checks.nix
> @@ -17,6 +17,7 @@
>
> nativeBuildInputs = [ shellcheck ];
> } ''
> + shopt -s globstar
> shellcheck $src/**/*.sh
> touch $out
> ''
> --
> 2.35.1
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 2/2] nix/checks.nix: shellcheck more files
2022-11-16 8:56 ` Henri Rosten
@ 2022-11-16 10:40 ` Alyssa Ross
0 siblings, 0 replies; 4+ messages in thread
From: Alyssa Ross @ 2022-11-16 10:40 UTC (permalink / raw)
To: Henri Rosten; +Cc: devel
[-- Attachment #1: Type: text/plain, Size: 530 bytes --]
On Wed, Nov 16, 2022 at 10:56:18AM +0200, Henri Rosten wrote:
> On Tue, Nov 15, 2022 at 10:56:37AM +0000, Alyssa Ross wrote:
> > I didn't realise that bash's ** feature (which matches any depth of
> > directories) wasn't enabled by default, because it appeared to work,
> > just didn't find all scripts.
>
> I'm getting shellcheck warnings from
> host/rootfs/build/etc/mdev/modalias.sh. Did you intentionally include
> scripts under 'build' too, or should those be excluded?
They should indeed be excluded. Thanks for catching!
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2022-11-16 10:40 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-11-15 10:56 [PATCH 1/2] Documentation/scripts: fix shellcheck issues Alyssa Ross
2022-11-15 10:56 ` [PATCH 2/2] nix/checks.nix: shellcheck more files Alyssa Ross
2022-11-16 8:56 ` Henri Rosten
2022-11-16 10:40 ` Alyssa Ross
Code repositories for project(s) associated with this public inbox
https://spectrum-os.org/git/crosvm
https://spectrum-os.org/git/doc
https://spectrum-os.org/git/mktuntap
https://spectrum-os.org/git/nixpkgs
https://spectrum-os.org/git/spectrum
https://spectrum-os.org/git/ucspi-vsock
https://spectrum-os.org/git/www
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).