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

Run shell before the start of the build #508

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

koko-ng
Copy link

@koko-ng koko-ng commented Jan 2, 2023

Hi :)
For some build systems there were calls to build before shell, which made it impossible to open a shell in the build environment if the build failed. I don't think that moving shell before has an implication on the environment as the code in between only seems to do checks on the project.

Fixes #479

@TingPing
Copy link
Member

TingPing commented Jan 3, 2023

It does change the behavior since previously it would configure the project, then open a shell, now it just opens a shell.

I'm not sure this distinction matters or not but another possibility is if run_shell is TRUE to not consider configuration failures as fatal?

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.

[Bug]: --build-shell parameter stops after configuration not download/unpackaging
2 participants