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

Stop shelling out to scp #8

Open
geelen opened this issue Nov 8, 2010 · 1 comment
Open

Stop shelling out to scp #8

geelen opened this issue Nov 8, 2010 · 1 comment

Comments

@geelen
Copy link
Owner

geelen commented Nov 8, 2010

At the moment, we write a temporary file then shell out to scp to upload it. It's dumb.

Use Net::Sftp or something instead.

@lparry
Copy link
Collaborator

lparry commented Nov 11, 2010

I've got a patch for this on the branch use_sftp_for_file_writes

the downside of this patch is that shell expansion doesnt work with it. ie. '~' and '$HOME' dont get expanded. if you give an absolute path it works great.

a possible solution might be to use the ssh session to expand the path on the remote system prior to passing it to net::sftp

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

No branches or pull requests

2 participants