tuzig / capacitor-ssh-plugin

An ssh plugin for hybrid apps, supports both iOS and Android.
MIT License
1 stars 2 forks source link

Support iOS simulator #1

Open daonb opened 2 years ago

daonb commented 2 years ago

When trying to build the NMSSH library for an iPad simulator we get the error message:


ld: in /Users/daonb/src/NMSSH/NMSSH-iOS/Libraries/lib/libssh2.a(agent.o), building for iOS Simulator, but linking in object file built for iOS, for architecture arm64
Akaame commented 1 year ago

Hello, I was going to take a jab at #3 because its issuehunt bounty seemed attractive. Having a look at that I have seen this issue and tried my hand at this and come up with how this would work with the involvement of iSSH2/NMSSHT7 and this repository.

Ionic, Capacitor et al. are kind of dead. I do not know if you are still maintaining this and if the bounty for #3 still holds.

daonb commented 1 year ago

Thanks for reaching out. The bounty still hold as I need this plugin to work on the simulator. You're right it involves NMSSHT7 which is also controlled by me and I'd welcome your PR.

I wonder what made you thing capacitor is dead? If anything it changes too rapidly and occasionally break their API.

Akaame commented 1 year ago

I meant "dead" in the sense that it does not have the buzz it used to have. But let's not derail from the topic.

So here is the gist of the problem:

Have a look at this proposal and if you like it I can start working on this. If iSSH2 is not receptive to the change we want to make we can get the prebuilt xcframeworks from somewhere else or build them ourselves forking the library. How does it sound?

Cheers.

daonb commented 1 year ago

Sounds good!

I don't like the podfile and I'd be happy to replace it. Another thing you'll need to add is a basic test that automated the simulator and connects to a local SSH server.

daonb commented 1 year ago

I've did some research and I'm not sure we can replace the podfile as capacitor needs it.

Akaame commented 1 year ago

I think you have misunderstood me. podfile is necessary and okay. What is wrong is the original NMSSH library was vendoring binary files along with the library itself which is a horrible way to distribute software. I proposed to change that. Was busy with work. Will try to create an initial fix for the NMSSHT7 today.