janware/jw-pkg
1
Code Pull requests Activity Actions

rules.mk: Add targets to check shell syntax #32

Merged
Jan Lindemann merged 2 commits from jan/feature/20260624-rules-mk-add-targets-to-check-shell-syntax into master 2026-06-24 22:03:26 +02:00 AGit
Conversation 0 Commits 2 Files changed 2 +29 -5
Jan Lindemann commented 2026-06-24 21:52:37 +02:00
Owner
Copy link

defs.mk: Use EXE_SH for shell scripts only

To make EXE_SH useful for automated shell syntax checking, remove all non POSIX shell / bash scripts from that variable, and place the non-shell scripts into EXE_SCRIPTS.

rules.mk: Add targets to check shell syntax

Add the target sh-syntax-check, which triggers bash syntax linting with shellcheck This commit stops short of making target all depend on it, because the fallout is impressive.

#### defs.mk: Use EXE_SH for shell scripts only To make EXE_SH useful for automated shell syntax checking, remove all non POSIX shell / bash scripts from that variable, and place the non-shell scripts into EXE_SCRIPTS. #### rules.mk: Add targets to check shell syntax Add the target sh-syntax-check, which triggers bash syntax linting with shellcheck This commit stops short of making target all depend on it, because the fallout is impressive. 
To make EXE_SH useful for automated shell syntax checking, remove all non POSIX shell / bash scripts from that variable, and place the non-shell scripts into EXE_SCRIPTS.
Signed-off-by: Jan Lindemann <jan@janware.com>
rules.mk: Add targets to check shell syntax
All checks were successful
CI / Packaging - Kali Linux (pull_request) Successful in 3m3s
Details
CI / Packaging - OpenSUSE Tumbleweed (pull_request) Successful in 3m33s
Details
CI / Packaging test (pull_request) Successful in 0s
Details
CI / Packaging - Kali Linux (push) Successful in 3m9s
Details
CI / Packaging - OpenSUSE Tumbleweed (push) Successful in 3m23s
Details
CI / Packaging test (push) Successful in 0s
Details
3aecb6c685
 
Add the target sh-syntax-check, which triggers bash syntax linting with shellcheck This commit stops short of making target all depend on it, because the fallout is impressive.
Signed-off-by: Jan Lindemann <jan@janware.com>
Jan Lindemann scheduled this pull request to auto merge when all checks succeed 2026-06-24 21:52:39 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
janware/jw-pkg!32
No description provided.