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

rcp: fix -q option handling #683

Closed
wants to merge 1 commit into from
Closed

Conversation

robertek
Copy link

Using the -q option for rcp does not work.

Testing done after the fix:

❯ bastille rcp adguard /etc/hosts .

[adguard]:
/usr/local/bastille/jails/adguard/root//etc/hosts -> ./hosts


❯ bastille rcp adguard -q /etc/hosts .

[adguard]:

@yaazkal yaazkal added the bug Something isn't working label Jul 8, 2024
@yaazkal
Copy link
Collaborator

yaazkal commented Jul 14, 2024

Hi @robertek after testing your patch I got:

[-q]: Not found.

That was the original error? Beacause I get it before and after the patch.
If that was, the thing is more related to the command not picking the jail correctly. Let me know if you want to try again.

I'v tested on FreeBSD 14.0-RELEASE with Bastille HEAD at cb5697a

@yaazkal
Copy link
Collaborator

yaazkal commented Jul 14, 2024

Thanks @robertek for the report, This was also present in the cp command so I've made a fix where the root cause was. Closing this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants