search

AsteroidOS / starship

SailfishOS synchronization app for AsteroidOS.
GNU General Public License v3.0
22 stars 12 forks source link

Improve SailfishOS build #15

Closed direc85 closed 2 years ago

direc85 commented 2 years ago

Just some version.pri include fixes and systemd path fixes in the form of rpm macros: use %{_userunitdir} instead of %{_libdir} which points to /usr/lib64 which doesn't work at least on my Xperia 10 II, and the RPM build fails, too. Tested with SDK 3.5.7 (~SFOS 4.1) with SFOS 4.2 EA build target installed.

There were two headers missing from harbour-starfish.cpp, so I added them in.

I declared the starfish config and SAILFISHOS_PLATFORM to version.pri and this seems to fix building the project and setting the variant in a centralized manner.

Tested with Sony Xperia XA2 Ultra (armv7hl) and Sony Xperia 10 II (aarch64) which both are now able to connect to the watch (LG G Watch R aka. lenok), so that's a start!

This PR is accompanied with a PR over at asteroidsyncservice.

Kinda sorta closes #14 and #9, and contributes to #4, too.

direc85 commented 2 years ago

If you are super curious and can't wait to get your hands on the packages, here are the packages I tested the build with: harbour-starfish-1.0.zip. They didn't break my devices, but as always, download and use at your own risk.

Please note that these builds use the latest asteroidsyncservice, libasteroid and libwatchfish. As there is an update for asteroidsyncservice, too, I decided not to include those bumps in this PR.

FlorentRevest commented 2 years ago

Looks great, thank you! :D