Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CI: Fix CI_HELPER #22782

Merged
merged 1 commit into from
Dec 1, 2023
Merged

CI: Fix CI_HELPER #22782

merged 1 commit into from
Dec 1, 2023

Conversation

jefferyto
Copy link
Member

Maintainer: @aparcar
Compile tested: tested in jefferyto#18 (GA run)
Run tested: N/A

Description:
Commit 18d3c52 caused CI_HELPER to be set to '' (two single quotes), leading to an error when test.sh tries to source $CI_HELPER.

CI_HELPER is not set anywhere else and not passed into the container environment, so there is no need to test if it has a value.

This fixes the value of CI_HELPER.

Fixes: 18d3c52 ("github-ci: error on any shell errors")

Commit 18d3c52 caused CI_HELPER to be
set to "''" (two single quotes), leading to an error when test.sh tries
to source $CI_HELPER.

CI_HELPER is not set anywhere else and not passed into the container
environment, so there is no need to test if it has a value.

This fixes the value of CI_HELPER.

Fixes: 18d3c52 ("github-ci: error on any shell errors")

Signed-off-by: Jeffery To <jeffery.to@gmail.com>
@BKPepe BKPepe requested a review from aparcar November 29, 2023 10:54
@neheb neheb merged commit 562f5df into openwrt:master Dec 1, 2023
12 checks passed
@jefferyto jefferyto deleted the ci-fix-helper branch December 3, 2023 14:36
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.

None yet

2 participants