Skip to content

fix(gh-actions/build-debian): Set -euo pipefail in all steps#100

Merged
adombeck merged 2 commits intomainfrom
UDENG-8986-fix-lintian
Feb 9, 2026
Merged

fix(gh-actions/build-debian): Set -euo pipefail in all steps#100
adombeck merged 2 commits intomainfrom
UDENG-8986-fix-lintian

Conversation

@adombeck
Copy link
Contributor

@adombeck adombeck commented Feb 9, 2026

The "Run lintian on sources" and "Run lintian on binaries" steps succeeded even if lintian exits with a non-zero exit code.

Using set -euo pipefail fixes that. Let's use it consistently in all steps to avoid similar issues in the future.

UDENG-8986

The "Run lintian on sources" and "Run lintian on binaries" steps
succeeded even if lintian exits with a non-zero exit code.

Using `set -euo pipefail` fixes that. Let's use it consistently in all
steps to avoid similar issues in the future.
@adombeck adombeck changed the title gh-actions/build-debian: Set -euo pipefail in all steps fix(gh-actions/build-debian): Set -euo pipefail in all steps Feb 9, 2026
@adombeck adombeck marked this pull request as ready for review February 9, 2026 21:36
@adombeck adombeck requested a review from didrocks as a code owner February 9, 2026 21:36
@adombeck
Copy link
Contributor Author

adombeck commented Feb 9, 2026

@adombeck adombeck merged commit 42ec85a into main Feb 9, 2026
18 of 27 checks passed
@adombeck adombeck deleted the UDENG-8986-fix-lintian branch February 9, 2026 21:51
adombeck added a commit that referenced this pull request Feb 9, 2026
Turns out that lintian already checks the licenses and it's really the
issues found by lintian which we care about (at least that is what
reviewers of uploads to the archive look at).

We just have to make the build-debian action fail when lintian fails
(done in #100) and pass `--fail-on error,warning` to lintian (possible
via the `lintian` input).
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