Skip to content

SQSCANGHA-97 Use /usr/bin/env for shebang#193

Merged
henryju merged 1 commit intoSonarSource:masterfrom
eliandoran:use_env_bash
Jun 30, 2025
Merged

SQSCANGHA-97 Use /usr/bin/env for shebang#193
henryju merged 1 commit intoSonarSource:masterfrom
eliandoran:use_env_bash

Conversation

@eliandoran
Copy link
Copy Markdown
Contributor

Some platforms such as NixOS, Guix, and even some BSD distributions don't have /bin/bash resulting in errors when trying to run via a self-hosted runner:

c00da1f7-23a6-4abd-95c0-9d1c3b591c2a.sh: line 1: /var/ci/runner/_actions/sonarsource/sonarqube-scan-action/v5/scripts/sanity-checks.sh: cannot execute: required file not found

The solution is to use /usr/bin/env which is guaranteed to be available everywhere.

Further reference:

Please be aware that we are not actively looking for feature contributions. The truth is that it's extremely difficult for someone outside SonarSource to comply with our roadmap and expectations. Therefore, we typically only accept minor cosmetic changes and typo fixes. If you would like to see a new feature, please create a new thread in the forum "Suggest new features".

With that in mind, if you would like to submit a code contribution, make sure that you adhere to the following guidelines and all tests are passing:

  • Please explain your motives to contribute this change: what problem you are trying to fix, what improvement you are trying to make
  • Make sure any code you changed is covered by tests
  • If there is a JIRA ticket available, please make your commits and pull request start with the ticket ID (SONAR-XXXX)

We will try to give you feedback on your contribution as quickly as possible.

Thank You!
The SonarSource Team

@henryju henryju changed the title Use /usr/bin/env for shebang SQSCANGHA-97 Use /usr/bin/env for shebang Jun 30, 2025
@henryju henryju merged commit ef211f9 into SonarSource:master Jun 30, 2025
60 of 61 checks passed
@henryju
Copy link
Copy Markdown
Member

henryju commented Jun 30, 2025

Hi @eliandoran thanks for your contribution.

luketainton pushed a commit to luketainton/actions_gha-workflows that referenced this pull request Aug 15, 2025
#27)

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [SonarSource/sonarqube-scan-action](https://github.com/SonarSource/sonarqube-scan-action) | action | minor | `v5.2.0` -> `v5.3.0` |

---

### Release Notes

<details>
<summary>SonarSource/sonarqube-scan-action (SonarSource/sonarqube-scan-action)</summary>

### [`v5.3.0`](https://github.com/SonarSource/sonarqube-scan-action/releases/tag/v5.3.0)

[Compare Source](SonarSource/sonarqube-scan-action@v5.2.0...v5.3.0)

##### What's Changed

- SQSCANGHA-83 Avoid unbound variable error on parameter expansion by [@&#8203;aleksandra-bozhinoska-sonarsource](https://github.com/aleksandra-bozhinoska-sonarsource) in SonarSource/sonarqube-scan-action#192
- SQSCANGHA-97 Use /usr/bin/env for shebang by [@&#8203;eliandoran](https://github.com/eliandoran) in SonarSource/sonarqube-scan-action#193
- SQSCANGHA-98 Update SonarScanner CLI to 7.2.0.5079 by [@&#8203;github-actions](https://github.com/github-actions)\[bot] in SonarSource/sonarqube-scan-action#196

##### New Contributors

- [@&#8203;eliandoran](https://github.com/eliandoran) made their first contribution in SonarSource/sonarqube-scan-action#193

**Full Changelog**: SonarSource/sonarqube-scan-action@v5.2.0...v5.3.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box

---

This PR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiI0MS40Mi4yIiwidXBkYXRlZEluVmVyIjoiNDEuNDIuMiIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOlsidHlwZS9kZXBlbmRlbmNpZXMiXX0=-->

Reviewed-on: https://git.tainton.uk/actions/gha-workflows/pulls/27
Reviewed-by: Luke Tainton <[email protected]>
Co-authored-by: Renovate [BOT] <[email protected]>
Co-committed-by: Renovate [BOT] <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants