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

Sultan does not throw an error on mv if the destination directory does not have right permissions #23

Open
davydany opened this issue Jan 31, 2017 · 1 comment
Labels

Comments

@davydany
Copy link
Owner

davydany commented Jan 31, 2017

It appears that Sultan isn't throwing any errors if the user doesn't have permissions to do a mv. It simply swallows it, and the error doesn't get thrown.

with Sultan.load() as s:
    s.mv(source_file, deployed).run()

Also, why is it swallowing errors and not throwing them if it encounters it?

@davydany davydany added the bug label Jan 31, 2017
@LX5321
Copy link

LX5321 commented Oct 2, 2019

In my opinion when the mv command is executed, the control is passed to the operating system from the python environment. I don't think python scripts detect errors thrown by the operating system stream and terminates. Since the output is not captured by python, it goes undetected in the OS stream.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants