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.

Sign up for GitHub

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

Jump to bottom

ci: footprint: Fix shopt not found #80166

Merged
merged 1 commit into from
Oct 22, 2024
Merged

ci: footprint: Fix shopt not found #80166

merged 1 commit into from
Oct 22, 2024

Conversation

golowanow
Copy link
Member

@golowanow golowanow commented Oct 21, 2024 •
edited
Loading

@golowanow
ci: footprint: Fix shopt not found
9c3c48c 
Use bash explicitly to have shopt available.

Signed-off-by: Dmitrii Golovanov <dmitrii.golovanov@intel.com>
@golowanow golowanow added the bug The issue is a bug, or the PR is fixing a bug label Oct 21, 2024
@golowanow golowanow added Hotfix Fix for issues blocking development, i.e. CI issues, tests failing in CI, etc. and removed bug The issue is a bug, or the PR is fixing a bug area: Continuous Integration labels Oct 21, 2024
@zephyrbot zephyrbot added the size: XS A PR changing only a single line of code label Oct 21, 2024
@nashif nashif added Trivial Changes that can be reviewed by anyone, i.e. doc changes, minor build system tweaks, etc. and removed Hotfix Fix for issues blocking development, i.e. CI issues, tests failing in CI, etc. labels Oct 21, 2024
@aescolar aescolar merged commit b758e20 into zephyrproject-rtos:main Oct 22, 2024
33 checks passed
@aescolar
Copy link
Member

@golowanow @nashif note https://github.com/zephyrproject-rtos/zephyr/actions/runs/11459954282/job/31885588961

@golowanow
Copy link
Member Author

golowanow commented Oct 22, 2024 •
edited
Loading

@golowanow @nashif note https://github.com/zephyrproject-rtos/zephyr/actions/runs/11459954282/job/31885588961

Looks like globstar option is not available by default there, my bad.
#80239

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aescolar aescolar aescolar approved these changes

@stephanosio stephanosio stephanosio approved these changes

@fabiobaltieri fabiobaltieri fabiobaltieri approved these changes

@kartben kartben Awaiting requested review from kartben

@nashif nashif Awaiting requested review from nashif

Assignees

@nashif nashif

@stephanosio stephanosio

Labels
area: Continuous Integration size: XS A PR changing only a single line of code Trivial Changes that can be reviewed by anyone, i.e. doc changes, minor build system tweaks, etc.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@golowanow @aescolar @fabiobaltieri @stephanosio @nashif @zephyrbot