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

Use sftp's mkdir instead of exec #134

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

Conversation

PeterRincker
Copy link

SftpConnection.prototype.createRemoteDirectory now purely uses the SFTP
subsystem instead of relying on exec and mkdir -p. This ensures that
sftp-deployment works in chroot’ed environments.

SftpConnection.prototype.createRemoteDirectory now purely uses the SFTP
subsystem instead of relying on exec and mkdir -p. This ensures that
sftp-deployment works in chroot’ed environments.
@christophgockel
Copy link

I know this is and old pull request, but as this repo seems stale, I've picked it up as a fork and already merged your PR into it @PeterRincker. This helped me immediately to get through the main issues I had with this plugin. So thank you very much!

I tried several SFTP syncing plugins and they all seemed outdated/non-maintained. With your changes and some minor small updates by myself the plugin is ready for atom 1.20 now with the majority of feature working again - at least the parts I need at the moment.

In case you're still interested, you can check it out here: https://github.com/christophgockel/atom-sftp-sync

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.

2 participants