Skip to content

Conversation

@ninjapanzer
Copy link

Fix for issue #6

These changes include paramiko and pycrypt as dependencies to the plugin and allow for navigation editing and saving of files on osx.

I have only tested OSX. I have not specifically modified the ability for this to work with putty. I have only wrapped commands for the different os's based upon os mode.

This is my first go at python so I am sure my convention has left something to be desired.

I had a little issue with the nature of navigation of directories via paramiko's interface so for OSX I have changed the behavior of entering a folder. If you do the folder list is replaced with only the new folders contents. I also added a .. menu option to allow navigation back up the directory chain.

One thing I am not sure of is I had some issue with pycrypt until I built it locally on my system. I don't know if that is something that needs to be done via os or just once as I am unfamiliar with how python binds things like this.

Let me now if you have any feedback.

Paul

…connect process

Now connects using paramiko added password tracking.

Need to interface overtop the send methods for paramiko

General implementation works with login CD and LS.

Some hooks after initial LS do not function

Allowed for correct closing of transport and sftp threads

Added way to force display of directory on each connection.

SSH transports are no longer lost

Created new cleanls to work from paramiko

Path corrections

Opening file does not update search path

Deleted some ugly logging statementsw

Refactored and corrected folder file identification for paramiko ls results

assured that thread is stopped on disconnect

Almost fully functional demo

First Testing version working on osx

Split logic for posix in download localpath variable

removed a bunch of print statements
@ninjapanzer ninjapanzer mentioned this pull request Jan 24, 2013
@ninjapanzer
Copy link
Author

I have a alpha of symlink support in a feature branch. I will be revising and merging that over the week

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.

1 participant