Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Main: don't start settings daemon in installer session #761

Merged
merged 8 commits into from
Oct 22, 2024

Conversation

danirabbit
Copy link
Member

@danirabbit danirabbit commented Sep 30, 2024

In an installer session, settings daemon is started by the session manager. Should also fix gsettings overrides not working (for example for network notifications)

@danirabbit danirabbit requested a review from a team September 30, 2024 22:53
@ryonakano ryonakano self-requested a review September 30, 2024 22:56
@ryonakano
Copy link
Contributor

I tested this branch by creating a hock to install the greeter .deb while building the iso, but unfortunately this does not fix elementary/installer#775 😢

VirtualBox_test_01_10_2024_21_10_38

VirtualBox_test_01_10_2024_21_04_45

@danirabbit
Copy link
Member Author

@ryonakano hm okay. Still probably shouldn't be starting GSD twice here. I also moved in a bit about settings the session name which affects gsettings overrides

@ryonakano

This comment was marked as resolved.

@danirabbit

This comment was marked as resolved.

@danirabbit
Copy link
Member Author

@ryonakano did that fix translations though?? Or is this because you logged in to demo session to install it?

@ryonakano
Copy link
Contributor

@danirabbit Unfortunately the translation issue still presents. I booted from the ISO built using this branch and it looks like this:

VirtualBox_test_08_10_2024_21_11_56

@danirabbit danirabbit marked this pull request as draft October 8, 2024 14:53
@danirabbit
Copy link
Member Author

@ryonakano Wallpaper issue should be fixed by elementary/installer#840

@danirabbit danirabbit marked this pull request as ready for review October 17, 2024 20:26
@ryonakano
Copy link
Contributor

Built a daily ISO with this branch and confirmed the wallpaper issue has been resolved.

@danirabbit This PR looks good to me, but what is the difference with #768?

@danirabbit
Copy link
Member Author

@ryonakano that other PR was just me testing. It can be closed

Copy link
Contributor

@ryonakano ryonakano left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see, let's merge this PR then. Sorry for my late review 🙏

@lenemter lenemter merged commit b8b5761 into master Oct 22, 2024
4 checks passed
@lenemter lenemter deleted the danirabbit/main-settingsdaemon branch October 22, 2024 10:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants