Skip to content

Conversation

@larsewi
Copy link
Contributor

@larsewi larsewi commented Sep 2, 2025

Example output from check_scripts.sh:

$ ./user-scripts/check-scripts.sh
--- ./user-scripts/../build-scripts/produce-debug-symbols.orig
+++ ./user-scripts/../build-scripts/produce-debug-symbols
@@ -4,8 +4,7 @@
 . detect-environment
 . compile-options

-if [ $BUILD_TYPE != "RELEASE" ]
-then
+if [ $BUILD_TYPE != "RELEASE" ]; then
     echo "Debug symbols not requested"
     exit 0
 else

File 'produce-debug-symbols' requires formatting.
Do you wish to format 'produce-debug-symbols'? [y/N] y
Formatting file 'produce-debug-symbols'...

In ./user-scripts/../build-scripts/produce-debug-symbols line 7:
if [ $BUILD_TYPE != "RELEASE" ]; then
     ^---------^ SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean:
if [ "$BUILD_TYPE" != "RELEASE" ]; then

For more information:
  https://www.shellcheck.net/wiki/SC2086 -- Double quote to prevent globbing ...

File 'produce-debug-symbols' requires manual intervention.
Press any key to continue...

@larsewi
Copy link
Contributor Author

larsewi commented Sep 3, 2025

@cf-bottom Jenkins with exotics please :)

@cf-bottom
Copy link

@larsewi larsewi force-pushed the linting branch 2 times, most recently from 6e2ea0f to 75968eb Compare September 3, 2025 10:54
@larsewi larsewi marked this pull request as ready for review September 3, 2025 11:02
@larsewi larsewi force-pushed the linting branch 4 times, most recently from 50d388f to 7159f45 Compare September 3, 2025 12:47
Copy link
Contributor

@craigcomstock craigcomstock left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lets move the find files and lint that is in yaml to a script/makefile so mortals can run it.

larsewi and others added 6 commits September 4, 2025 11:10
Ticket: ENT-12601
Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>
Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>
Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>
Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>
Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>
@larsewi larsewi force-pushed the linting branch 2 times, most recently from 601e7ad to da09115 Compare September 4, 2025 14:07
@cfengine cfengine deleted a comment from cf-bottom Sep 4, 2025
@cfengine cfengine deleted a comment from cf-bottom Sep 5, 2025
Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>
Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>
Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>
Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>
Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>
Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>
Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>
Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>
Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>
Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>
Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>
Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>
Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>
Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>
Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>
Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>
Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>
Ticket: ENT-12601
Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>
Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>
@larsewi
Copy link
Contributor Author

larsewi commented Sep 5, 2025

Try again
Build Status

@craigcomstock
Copy link
Contributor

cancelled the job because two exotics failed testing-pr because of a buildscripts error I am fixing in #1894 and jenkins needs to be restarted.

@craigcomstock
Copy link
Contributor

@cf-bottom jenkins exotics

@cfengine cfengine deleted a comment from cf-bottom Sep 6, 2025
@cf-bottom
Copy link

@larsewi
Copy link
Contributor Author

larsewi commented Sep 8, 2025

@cf-bottom Jenkins with exotics please :)

@cf-bottom
Copy link

@larsewi larsewi merged commit aa86704 into cfengine:master Sep 8, 2025
30 of 37 checks passed
@larsewi larsewi deleted the linting branch October 2, 2025 14:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants