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.

Sign up for GitHub

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

Jump to bottom

feature request:windows support #86

Closed
Ecalose opened this issue Feb 25, 2022 · 2 comments · Fixed by #94
Closed

feature request:windows support #86

Ecalose opened this issue Feb 25, 2022 · 2 comments · Fixed by #94
Assignees
@blacknon

Comments

@Ecalose
Copy link

Ecalose commented Feb 25, 2022

could you support windows?i tried to build the binary in the windows but failed.I might build in a wrong way?

CGO_ENABLED=0 GOOS=windows GOARCH=amd64 go build ./cmd/lssh
@blacknon blacknon self-assigned this Feb 27, 2022
@blacknon
Copy link
Owner

Probably because the pkcs11 library you are using is calling C, you will not be able to cross-compile.
If you compile it on Windows, it will work, but it is not recommended because the terminal display is distorted.
Also, since lsftp currently uses some UNIX-like syscalls, I think it will fail to compile.

I would like to support Windows in the future, but I haven't done it yet.

@Ecalose
Copy link
Author

Ecalose commented Mar 3, 2022

Probably because the pkcs11 library you are using is calling C, you will not be able to cross-compile. If you compile it on Windows, it will work, but it is not recommended because the terminal display is distorted. Also, since lsftp currently uses some UNIX-like syscalls, I think it will fail to compile.

I would like to support Windows in the future, but I haven't done it yet.

oh,thx

@blacknon blacknon mentioned this issue Mar 13, 2022
Merged
blacknon added a commit that referenced this issue Mar 13, 2022
@blacknon
Merge pull request #93 from blacknon/topic/issue_#92_listの利用ライブラリを変更する
4dd0041 
Topic/issue #92 listの利用ライブラリを変更する

Close #92.
#86 Fixed.
@blacknon blacknon mentioned this issue Mar 13, 2022
Merged
blacknon added a commit that referenced this issue Mar 13, 2022
@blacknon
Merge pull request #94 from blacknon/develop
234964a 
Version 0.6.7

Close #86 
Close #91 
Close #92 

- Supported Windows.
- Add Reverse Dynamic Forward.
- Add localrc compress option.
- Addition of function to specify host in target PATH with shell command of lsftp. 
    - example: `get host1,host2:/path/to/src /path/to/dest`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@blacknon blacknon

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Version 0.6.7 blacknon/lssh
2 participants
@blacknon @Ecalose