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

bastille cmd exit code not respected #272

Closed
mcgaw opened this issue Nov 10, 2020 · 2 comments
Closed

bastille cmd exit code not respected #272

mcgaw opened this issue Nov 10, 2020 · 2 comments
Labels
bug Something isn't working

Comments

@mcgaw
Copy link

mcgaw commented Nov 10, 2020 •
edited
Loading

Describe the bug
The exit code from the command run using bastille cmd is not respected.

Bastille and FreeBSD version
0.7.20200714
12.2-STABLE
12.2-STABLE
12.2-STABLE

How did you install bastille?
github

Steps to reproduce
bastille cmd my_container more xxxx
echo $?

Expected behavior
Output of 1 on console. 0 is output.
@mcgaw mcgaw added the bug Something isn't working label Nov 10, 2020
@mcgaw mcgaw changed the title [BUG] bastille cmd exit code not respected Nov 10, 2020
@cedwards cedwards mentioned this issue Dec 13, 2020
Closed
yerrysherry pushed a commit to yerrysherry/bastille that referenced this issue Jan 4, 2021
@serege
bastille cmd exit code not respected BastilleBSD#272
36acd5b
yerrysherry pushed a commit to yerrysherry/bastille that referenced this issue Jan 4, 2021
@serege
bastille cmd exit code not respected BastilleBSD#272
b213414
cedwards added a commit that referenced this issue Jul 31, 2021
@cedwards
Merge pull request #303 from yerrysherry/cmdbug
7f1dbc1 
bastille cmd exit code not respected #272
@zilti
Copy link
Contributor

zilti commented Jun 25, 2022

Did pull request #303 fix this for you?

@mcgaw
Copy link
Author

mcgaw commented Jun 25, 2022

Hi, I've quickly tested it and the issue now appears resolved. Sorry for not keeping an eye on it but I switched to iocage not long after raising this issue since it appeared more mature and satisfied broadly the same use cases. Not sure how things have evolved since then.

@mcgaw mcgaw closed this as completed Jun 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@zilti @mcgaw