-
Notifications
You must be signed in to change notification settings - Fork 378
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
current directory not changed by -R #66
Comments
Hello @fluxer, By default, PRoot changes the [guest] current working directory
Regarding the -R option, it is similar to the -r option but it
As a consequence, the "chdir($PWD)" performed by PRoot into the Now if you move to -- let's say -- "/etc" before running PRoot,
Another example, if you move to a directory that exists into the
If you don't want your $HOME bound/mounted into the guest rootfs,
or use the -R option, then (order matters) bound/mount an empty
Please, let me know if you think we can make this default Regards, |
I understand your point but I must say I didn't expected that. Usually, chroot changes working directory to the root (/) right after changing root to the guest filesystem I'm familiar with the -b option, I've used it in certain cases. It certainly has it's purpose. Just now I noticed that there is a -w (and certian alternative) options that may do what I'm looking for (e.g. proot -w / -R new_root) but again, I find it confusing compared to the chroot, provided by coreutils or busybox, behavior and would like to see this behavior corrected. If not, then at least there is a way to make proot behave as I expect so it's better than nothing. And by the way, -b empty:/home/smil3y/ seems neat :). Cheers, |
Oddly, when proot is invoked with -R the current directory is not actually changed to the
cpecified new root directory leaving the shell in the current directory with access outside
of the new root. This does not happend with invoked with -r. Here is an exmaple:
That looks like a critical issue and I expect it not to happen, or is there a reasonfor doing so?
The text was updated successfully, but these errors were encountered: