Skip to content
This repository has been archived by the owner on Dec 8, 2023. It is now read-only.

Not rendering after service restart #26

Closed
arpio23 opened this issue May 13, 2023 · 5 comments
Closed

Not rendering after service restart #26

arpio23 opened this issue May 13, 2023 · 5 comments
Labels
question Further information is requested

Comments

@arpio23
Copy link
Contributor

arpio23 commented May 13, 2023

This might be a device specific issue?

After upgrading the shell, the service restarts and you see actually the last rendered frame from the last session and nothing works. Seems like its running on a dummy display.

Can also be reproduced by just restarting cutie-ui-io.service.

I encountered same issue when testing wayfire, its not really the cutie code. Its even possible to start apps by blindly swipe and touch but its not rendering on the screen.

Is it possible to prevent the service to restart after upgrade? Maybe just informing the user that a restart is needed?

Maybe it is the service file itself? Cause I just adapted the cutie one for wayfire.

@arpio23 arpio23 added bug Something isn't working question Further information is requested and removed bug Something isn't working labels May 13, 2023
@erikinkinen
Copy link
Member

This is a device specific issue. I have a similar problem on sargo. Restarting the Android-side hwcomposer service does fix it on sargo. AFAIK, it is an issue in the proprietary hwcomposer vendor implementation. It also affects phosh.

You can try running systemctl restart android-service@hwcomposer.

@arpio23
Copy link
Contributor Author

arpio23 commented May 14, 2023

Thats what I was thinking also. If we going to target not only a couple of devices, is it possible to make other services restart if cutie restarts?

erikinkinen added a commit that referenced this issue May 14, 2023
As discussed in #26 , restarting shell on upgrade causes problems. Also, it is not just generally a good idea.
Users should reboot phones after updates anyways.
@erikinkinen
Copy link
Member

Thats what I was thinking also. If we going to target not only a couple of devices, is it possible to make other services restart if cutie restarts?

I did not mean to make the service restart on upgrade, but it was the default setting. The phosh packaging explicitly disables this. So, I disabled in now on cutie-shell.

After all, the whole phone should preferable be rebooted after updating. When we have a graphical front-end for updates. It should reboot as well. Actually, I would prefer offline updates then (in the style of gnome-software or kde's discover).

@arpio23
Copy link
Contributor Author

arpio23 commented May 14, 2023

I'm happy with this solution. I would close this issue.

@erikinkinen
Copy link
Member

I'm happy with this solution. I would close this issue.

Ok, then. It is resolved.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
question Further information is requested
Projects
Status: Done
Development

No branches or pull requests

2 participants