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

Agent for Putty #20

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Agent for Putty #20

wants to merge 4 commits into from

Conversation

ghost
Copy link

@ghost ghost commented May 20, 2018

Works fine under Windows 10, seems to have some problems with Windows 7 sometimes (do not have this old operatign system, but if anybody can spare some time to fix it would be great - wholw coding took one night)

@Falsen
Copy link

Falsen commented Sep 7, 2019

Amazing, I searched for this for a while and I tried these changes which worked flawlessly! Thank you very much, this should be merged

Falsen added a commit to Falsen/ledger-app-ssh-agent that referenced this pull request Sep 8, 2019
@sharppanda
Copy link

sharppanda commented Jul 8, 2020

Is this project still active? Why was this never merged?
Would it be possible to update it for Python3?
@greenknot

@Falsen
Copy link

Falsen commented Jul 8, 2020

@sharppanda You can use my repo for the time being https://github.com/Falsen/ledger-app-ssh-agent.

Since it still works perfectly for me I haven't looked at the code anything

@sharppanda
Copy link

Thank you @Falsen

I wasn't able to upgrade it to python3. Do you plan to do that?
Sunset day for python2 was was January 1st, 2020.

@Falsen
Copy link

Falsen commented Jul 8, 2020

@sharppanda Works good now, possibly if it breaks haha

@sharppanda
Copy link

so you are still using python2? or both version in parallel?

@Falsen
Copy link

Falsen commented Jul 8, 2020

@sharppanda I have both versions installed right now so I don't really see a reason for me to fix it if it doesn't break. I think I tried fixing it a long time but the syntax has changed quite substantially (especially on things critical for it to properly work)

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.

3 participants