macfusion-ng / macfusion2

Macfusion2
225 stars 28 forks source link

sshfs-static 64-bit? #39

Closed wcgall closed 6 years ago

wcgall commented 6 years ago

macOS Mojave doesn't like the version of sshfs-static in this version of MacFusion. It gives a warning message to update, & lists sshfs-static as the problem in

About This Mac > System Report > Legacy Software

Should sshfs-static be replaced? Is there a 64-bit version? Does it just need to be recompiled? We have until the next version of macOS to fix it.

tessus commented 6 years ago

I created new static libs a while back #28 and they work on my system. For some reason @ElDeveloper can't get them to work, so it was never merged.

wcgall commented 6 years ago

Thanks. I tried b3, but keep having to grant permissions:

for each "disk".

On Oct 26, 2018, at 1:45 PM, Helmut K. C. Tessarek notifications@github.com wrote:

I created new static libs a while back #28 https://github.com/macfusion-ng/macfusion2/pull/28 and they work on my system. For some reason @ElDeveloper https://github.com/ElDeveloper can't get them to work, so it was never merged.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/macfusion-ng/macfusion2/issues/39#issuecomment-433488990, or mute the thread https://github.com/notifications/unsubscribe-auth/AV1bWKqrPdjsxmVMpx8x3-bj3ulpi0mzks5uo0oqgaJpZM4X8qzj.

wcgall commented 6 years ago

I seem to have fixed that by editing things in Keychain Access.

On Oct 26, 2018, at 2:08 PM, Warren Siegel wsiegel@icloud.com wrote:

Thanks. I tried b3, but keep having to grant permissions:

<Screen Shot 2018-10-26 at Oct 26, 2018 - 2.05.59 PM.png>

for each "disk".

On Oct 26, 2018, at 1:45 PM, Helmut K. C. Tessarek <notifications@github.com mailto:notifications@github.com> wrote:

I created new static libs a while back #28 https://github.com/macfusion-ng/macfusion2/pull/28 and they work on my system. For some reason @ElDeveloper https://github.com/ElDeveloper can't get them to work, so it was never merged.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/macfusion-ng/macfusion2/issues/39#issuecomment-433488990, or mute the thread https://github.com/notifications/unsubscribe-auth/AV1bWKqrPdjsxmVMpx8x3-bj3ulpi0mzks5uo0oqgaJpZM4X8qzj.

tessus commented 6 years ago

Ok, just to clarify, https://evermeet.cx/pub/macosx/macfusion/Macfusion.app.b3-sta.tar.gz works for you?

Well, it also worked for others. So I am not sure what is going on with @ElDeveloper's env. Maybe we shoud merge it anyway and go from there when something is off.

wcgall commented 6 years ago

So far, haven’t done much testing. Fork it?

Sent from my iPad

On Oct 26, 2018, at 7:52 PM, Helmut K. C. Tessarek notifications@github.com wrote:

Ok, just to clarify, https://evermeet.cx/pub/macosx/macfusion/Macfusion.app.b3-sta.tar.gz works for you?

Well, it also worked for others. So I am not sure what is going on with @ElDeveloper's env. Maybe we shoud merge it anyway and go from there when something is off.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.

tessus commented 6 years ago

Fork it?

Nope, but @ElDeveloper also mentioned in #28 that we could merge it, if he's the only one wirh a problem. So, if the new static libs work for you as they do for some others and me, we should finally merge it.

ElDeveloper commented 6 years ago

@wcgall, just wanted to check if you've done more testing. If you can confirm things work for you, then let's merge #28. 👍

wcgall commented 6 years ago

Yes, I've used it as much as I normally do, with no more problems. So I vote for merge.

On Oct 31, 2018, at 11:59 PM, Yoshiki Vázquez Baeza notifications@github.com wrote:

@wcgall https://github.com/wcgall, just wanted to check if you've done more testing. If you can confirm things work for you, then let's merge #28 https://github.com/macfusion-ng/macfusion2/pull/28. 👍

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/macfusion-ng/macfusion2/issues/39#issuecomment-434923406, or mute the thread https://github.com/notifications/unsubscribe-auth/AV1bWJw4owPSH_8y2N0XiprZbnFSAO6Lks5uqnGYgaJpZM4X8qzj.

ElDeveloper commented 6 years ago

Excellent, thanks! Merged 👍