ajalkane / ringingrestorer_sailfish

RingingRestorer for SailfishOS
GNU General Public License v3.0
3 stars 2 forks source link

aarch64 build? #2

Closed poetaster closed 1 year ago

poetaster commented 2 years ago

I can also help getting the app into chum (build.sailfishfishos.org) for automagical builds?

Thanks!

ajalkane commented 2 years ago

Hey... I will try and find time to make an aarch64 build, hopefully with chum.

poetaster commented 2 years ago

cool! thanks and let me know if you need any help with build/chum ... I'm maintaining a bunch of apps. otherwise piggz and rinigus are really responsive on #sailfishos (-porters) at OFTC irc.

Orologiaio commented 2 years ago

I second this request, I just switched to an Xperia 10 iii and I'm constantly being reminded how useful this app is by not having it available whenever I put my phone in silent mode :)

I have no coding skills at all, but if I can help in any way, let me know.

ajalkane commented 2 years ago

Hey guys... I said I tried to find time, but as it's already a year and I still haven't done it then I guess I must admit my failure. Any of you can do this? I'm ok with forking the codebase or whatever needs to be done.

poetaster commented 2 years ago

Hey guys... I said I tried to find time, but as it's already a year and I still haven't done it then I guess I must admit my failure. Any of you can do this? I'm ok with forking the codebase or whatever needs to be done.

I'll try to find some time next week. I'll fork first and let you know how it goes.

poetaster commented 2 years ago

I second this request, I just switched to an Xperia 10 iii and I'm constantly being reminded how useful this app is by not having it available whenever I put my phone in silent mode :)

I have no coding skills at all, but if I can help in any way, let me know.

I just made some minor changes to the spec and did an aarch64 build. You could try installing that: https://github.com/poetaster/ringingrestorer_sailfish/releases/tag/1.5

I haven't had time to test it yet. Thanks!

ajalkane commented 2 years ago

Thank you. I did a quick test that it seems to work 👍.

The application itself fails to detect the daemon is running though, so it gives an error message regarding that.

Orologiaio commented 2 years ago

Thank you. I did a quick test that it seems to work 👍.

The application itself fails to detect the daemon is running though, so it gives an error message regarding that.

I can confirm both things. App seemingly works but can't detect that the daemon is running. Thanks!

poetaster commented 1 year ago

Thank you. I did a quick test that it seems to work +1.

The application itself fails to detect the daemon is running though, so it gives an error message regarding that.

Could you post the error message? I'm guessing it has to do with dbus permissions?

Orologiaio commented 1 year ago

Thank you. I did a quick test that it seems to work +1. The application itself fails to detect the daemon is running though, so it gives an error message regarding that.

Could you post the error message? I'm guessing it has to do with dbus permissions?

This is what appears in the app itself if launched from the app drawer:

"RingingRestorer backend is not running. If you have just started your phone, close this application, wait a couple of minutes, and then try starting the application again."

If I launch it via terminal, instead, a couple of errors about missing libraries appear, but the app UI also starts and it properly shows the button saying "Stop RingingRestorer" and the message "RingingRestorer is active. When a profile changed from ringing to silent, will ask how long until ringing is resumed."

Here's a screenshot of the terminal output:

RingingRestorer terminal launch
Orologiaio commented 1 year ago

I also just took a look on my old Xperia X which also is on SFOS 4.4 like my Xperia 10 iii, and I can confirm what Cmdr-Zod is saying in the other thread: https://github.com/ajalkane/ringingrestorer_sailfish/issues/3

I probably never noticed this error message while I was still using the X before I switched to the 10 iii a couple weeks ago because RingingRestorer never stopped working, and I probably never launched the app itself recently (as you usually don't need to).

Cmdr-Zod commented 1 year ago

I just tried your build on my Xperia 10 iii, I still see the error message in the UI regarding the backend not running, but so far it is working as expected. Thank you very much. Please put it on Openrepos or Chum once it has received enough testing, I'm in the process of switching from an Xperia X to a 10 iii, and getting all the apps in 64bit is a small adventure on its own.

poetaster commented 1 year ago

Ok, I've added a Sailjail disable block which might solve the issue with dbus calls to the service/systemd: https://github.com/poetaster/ringingrestorer_sailfish/releases/tag/1.6

Still having issues building on obs, but it's in the works.

Orologiaio commented 1 year ago

Ok, I've added a Sailjail disable block which might solve the issue with dbus calls to the service/systemd: https://github.com/poetaster/ringingrestorer_sailfish/releases/tag/1.6

Still having issues building on obs, but it's in the works.

It works!

poetaster commented 1 year ago

It works! Great! I'll get it in rotation or maybe @ajalkane pushes the one build to openrepos. Here we go....

poetaster commented 1 year ago

Ok, found the reason it won't compile on obs. From the pro:

system(qdbusxml2cpp -p src/daemon/generated/mceiface/mceiface.h:src/daemon/generated/mceiface/mceiface.cpp src/daemon/nemo-keepalive/lib/mceiface.xml)

I don't think is actually being used in the build and so I've removed it. I also had to do some odd rpm install magic since the systemd unit didn't want to install as it would with Creator.

If someone has the time, please test a build from: https://build.merproject.org/project/show/home:poetaster:ringtonerestorer

If it works, I'll push it to chum.

Orologiaio commented 1 year ago

Ok, found the reason it won't compile on obs. From the pro:

system(qdbusxml2cpp -p src/daemon/generated/mceiface/mceiface.h:src/daemon/generated/mceiface/mceiface.cpp src/daemon/nemo-keepalive/lib/mceiface.xml)

I don't think is actually being used in the build and so I've removed it. I also had to do some odd rpm install magic since the systemd unit didn't want to install as it would with Creator.

If someone has the time, please test a build from: https://build.merproject.org/project/show/home:poetaster:ringtonerestorer

If it works, I'll push it to chum.

Works for me.

ajalkane commented 1 year ago

Ok, found the reason it won't compile on obs. From the pro:

system(qdbusxml2cpp -p src/daemon/generated/mceiface/mceiface.h:src/daemon/generated/mceiface/mceiface.cpp src/daemon/nemo-keepalive/lib/mceiface.xml)

I don't think is actually being used in the build and so I've removed it. I also had to do some odd rpm install magic since the systemd unit didn't want to install as it would with Creator.

If someone has the time, please test a build from: https://build.merproject.org/project/show/home:poetaster:ringtonerestorer

If it works, I'll push it to chum.

Thank you,

now there is no error message. But for some reason "Stop RingingRestorer" button does not have effect. It shows in UI that it's stopped, but still when putting phone to silent the restore dialog appears. On the other hand, the daemon notices this change because when setting the restore timeout in dialog, it has no effect.

How it should work is that if RingingRestorer is not active, then that timeout dialog does not appear. I'm not quite sure what's going on there. I tested with build 4.3.0.15_aarch64

If you have idea for a fix that's nice, but if not I can push this version already to OpenRepos.

poetaster commented 1 year ago

Ok, found the reason it won't compile on obs. From the pro:

system(qdbusxml2cpp -p src/daemon/generated/mceiface/mceiface.h:src/daemon/generated/mceiface/mceiface.cpp src/daemon/nemo-keepalive/lib/mceiface.xml)

I don't think is actually being used in the build and so I've removed it. I also had to do some odd rpm install magic since the systemd unit didn't want to install as it would with Creator. If someone has the time, please test a build from: https://build.merproject.org/project/show/home:poetaster:ringtonerestorer If it works, I'll push it to chum.

Thank you,

now there is no error message. But for some reason "Stop RingingRestorer" button does not have effect. It shows in UI that it's stopped, but still when putting phone to silent the restore dialog appears. On the other hand, the daemon notices this change because when setting the restore timeout in dialog, it has no effect.

How it should work is that if RingingRestorer is not active, then that timeout dialog does not appear. I'm not quite sure what's going on there. I tested with build 4.3.0.15_aarch64

If you have idea for a fix that's nice, but if not I can push this version already to OpenRepos.

Ok, perhaps the generated interface is of significance. The error message was probably related to Sailjail. I just re-enabled that line and uploaded a rebuilt (QT creator build) file to: https://github.com/poetaster/ringingrestorer_sailfish/releases/tag/1.7

Another idea I had is that it could be, given how the unit files are being installed that it maybe there more than once?

ajalkane commented 1 year ago

I went through the code, and maybe what I remembered was wrong because at quick glance I couldn't see the code that would prevent the dialog from opening.

Of course I can't for the life of me understand why it should open if RingingRestorer is not active, but that's not here nor there. I must have some reason :P. So to me it seems this build works correctly as the code is.

poetaster commented 1 year ago

I went through the code, and maybe what I remembered was wrong because at quick glance I couldn't see the code that would prevent the dialog from opening.

Of course I can't for the life of me understand why it should open if RingingRestorer is not active, but that's not here nor there. I must have some reason :P. So to me it seems this build works correctly as the code is.

Ok, if the app/service do what they are intended to, it's a moot point :) I think pushing the build from github is the better idea (I believe the obs builds might not even work on openrepos). If they work, I'll also post builds for x86 and armv7.

ajalkane commented 1 year ago

Thanks! When there's builds usable on OpenRepos I can try uploading them there.

Orologiaio commented 1 year ago

Ok, found the reason it won't compile on obs. From the pro:

system(qdbusxml2cpp -p src/daemon/generated/mceiface/mceiface.h:src/daemon/generated/mceiface/mceiface.cpp src/daemon/nemo-keepalive/lib/mceiface.xml)

I don't think is actually being used in the build and so I've removed it. I also had to do some odd rpm install magic since the systemd unit didn't want to install as it would with Creator. If someone has the time, please test a build from: https://build.merproject.org/project/show/home:poetaster:ringtonerestorer If it works, I'll push it to chum.

Thank you,

now there is no error message. But for some reason "Stop RingingRestorer" button does not have effect. It shows in UI that it's stopped, but still when putting phone to silent the restore dialog appears. On the other hand, the daemon notices this change because when setting the restore timeout in dialog, it has no effect.

How it should work is that if RingingRestorer is not active, then that timeout dialog does not appear. I'm not quite sure what's going on there. I tested with build 4.3.0.15_aarch64

If you have idea for a fix that's nice, but if not I can push this version already to OpenRepos.

I confirm, the "stop daemon" button does nothing... sorry, when I tested the build I didn't think to also try disabling the app, only to check if it worked as intended when active :/

poetaster commented 1 year ago

https://github.com/poetaster/ringingrestorer_sailfish/releases/tag/1.7 now has all the architectures.

The question about whether to publish depends on whether the stop daemon button is fundamental. We could make that a ticket if the functions are otherwise as they should be.

Orologiaio commented 1 year ago

https://github.com/poetaster/ringingrestorer_sailfish/releases/tag/1.7 now has all the architectures.

The question about whether to publish depends on whether the stop daemon button is fundamental. We could make that a ticket if the functions are otherwise as they should be.

I never use it, personally.

poetaster commented 1 year ago

Thanks! When there's builds usable on OpenRepos I can try uploading them there. ... I never use it, personally.

The files I uploaded under the release on github should be fine for openrepos....

ajalkane commented 1 year ago

Thank you,

I've uploaded them to OpenRepos