pcolby / bipolar

Free your data from Polar FlowSync
GNU General Public License v3.0
115 stars 16 forks source link

FlowSync crash after installing Bipolar (mac) #92

Closed madshatlevik closed 5 years ago

madshatlevik commented 5 years ago

After installing BiPolar and the hook, FlowSync crashes. Im using the lates version of FlowSync 3.0.0.1337 on mac.

Error: dyld: Library not loaded: @rpath/QtNetwork.framework/Versions/5/QtNetwork Referenced from: /Applications/FlowSync.app/Contents/MacOS/FlowSync Reason: no suitable image found. Did find: /Applications/FlowSync.app/Contents/MacOS/../Frameworks/QtNetwork.framework/Versions/5/QtNetwork: mach-o, but wrong architecture /Applications/FlowSync.app/Contents/MacOS/../Frameworks/QtNetwork.framework/Versions/5/QtNetwork: stat() failed with errno=1 Abort trap: 6

pcolby commented 5 years ago

Thanks @madshatlevik, I don't have access to a (modern) OSX host currently, so this might take a little to diagnose.

What version of OSX are you running? What version of Bipolar? (I assume 0.5.9 (the latest), but just checking)

Would you be able to run the following commands?

cat /Applications/FlowSync.app/Contents/Frameworks/QtNetwork.framework/Versions/5/Resources/Info.plist 
file /Applications/FlowSync.app/Contents/Frameworks/QtNetwork.framework/Versions/5/QtNetwork

Both before (ie on a fresh FlowSync install), and after installing the hook?

Thanks!

madshatlevik commented 5 years ago

Hi!

Here are the output on fresh FlowSync install:

$ cat /Applications/FlowSync.app/Contents/Frameworks/QtNetwork.framework/Versions/5/Resources/Info.plist

<?xml version="1.0" encoding="UTF-8"?> <!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">

CFBundlePackageType FMWK CFBundleShortVersionString 5.5 CFBundleVersion 5.5.1 CFBundleGetInfoString Created by Qt/QMake CFBundleSignature ???? CFBundleExecutable QtNetwork CFBundleIdentifier org.qt-project.QtNetwork NOTE Please, do NOT change this file -- It was generated by Qt/QMake.

$ file /Applications/FlowSync.app/Contents/Frameworks/QtNetwork.framework/Versions/5/QtNetwork /Applications/FlowSync.app/Contents/Frameworks/QtNetwork.framework/Versions/5/QtNetwork: Mach-O 64-bit dynamically linked shared library x86_64

After Hook installed:

$ cat /Applications/FlowSync.app/Contents/Frameworks/QtNetwork.framework/Versions/5/Resources/Info.plist <?xml version="1.0" encoding="UTF-8"?> <!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">

CFBundlePackageType FMWK CFBundleShortVersionString 5.5 CFBundleVersion 5.5.1 CFBundleGetInfoString Created by Qt/QMake CFBundleSignature ???? CFBundleExecutable QtNetwork CFBundleIdentifier org.qt-project.QtNetwork NOTE Please, do NOT change this file -- It was generated by Qt/QMake.

$ file /Applications/FlowSync.app/Contents/Frameworks/QtNetwork.framework/Versions/5/QtNetwork /Applications/FlowSync.app/Contents/Frameworks/QtNetwork.framework/Versions/5/QtNetwork: Mach-O dynamically linked shared library i386

madshatlevik commented 5 years ago

Forgot to answer the two questions: Latest OSX Mojave 10.14.3 and latest Bipolar 0.5.9.

pcolby commented 5 years ago

Thanks @madshatlevik,

That show's the issue... :)

The original FlowSync for OSX targeted a 32-bit architecture, so the hook needed to do the same. However, it seems the latest FlowSync is now built for 64-bit architectures instead. I'm not sure exactly when the switch occurred (will be interesting to trace it), but regardless, the solution is to update Bipolar to supply a 64-bit OSX hook now. I'll hopefully get some time to do that in the next few days.

Thanks.

madshatlevik commented 5 years ago

Thanks! Looking forward to the update :)

pcolby commented 5 years ago

Ok, that (part) was easy. Travis CI is now building the new x86_64 builds. Note, I've also created a temporary hook-tmp-issue-92 tag, so Travis CI has somewhere to put the 'release artefacts', so we'll need to wait for both to complete. Then we'll have a new standalone hook to test.

I probably won't get time to test it tomorrow, but of course you're welcome to test it yourself once baked.

Once we've tested, I'll update the Bipolar pkg to include the new hook.

pcolby commented 5 years ago
pcolby commented 5 years ago

The hook finally built (Travis was having a lot of timeout issues downloading the Qt sources). I don't have time to test it just now, but you can try the latest hook from https://github.com/pcolby/bipolar/releases/tag/hook-tmp-issue-92 if you're keen (just download the QtNetwork-macosx10.10 one, and rename it to QtNetwork before overwriting the one included with the Bipolar package).

Let me know how it does (if you do decide to try it), otherwise I'll probably won't have access to a test OSX machine till next week.

Cheers.

madshatlevik commented 5 years ago

I just downloaded and tested it and it works great. Thanks a lot!

pcolby commented 5 years ago

Thanks @madshatlevik, that's excellent news! :)

I'll get the Bipolar installer updated to include the new hook, and release once I've added the session name fallback for #91.

Thanks again.

pcolby commented 5 years ago

The updated hook has now been released on 0.5.10 :)

https://github.com/pcolby/bipolar/releases/tag/v0.5.10