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

[config] Allow user nemo to connect with connman and dsme #14

Merged
merged 1 commit into from Jun 24, 2019

Conversation

neochapay
Copy link
Contributor

No description provided.

@locusf locusf merged commit b7a073b into nemomobile:master Jun 24, 2019
<allow send_destination="net.connman"/>
<allow send_destination="com.nokia.dsme" send_interface="com.nokia.dsme.request" send_member="req_shutdown"/>
<allow send_destination="com.nokia.dsme" send_interface="com.nokia.dsme.request" send_member="req_reboot"/>
<allow send_destination="com.nokia.dsme" send_interface="com.nokia.dsme.request" send_member="req_powerup"/>

Choose a reason for hiding this comment

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

Out of curiosity: Was there some particular reason for these tweaks to allow nemo user to do reboot/shutdown etc?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

becouse only root or console user can do that

Choose a reason for hiding this comment

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

Well, that is how it more or less is. But is there some reason for making the change? I'd assume it is something along the line "we'd like user to be able to reboot without hassle". But if it is something else, it might be something that needs to be addressed at sailfish side too.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

sfos use some strange magic and all users in all sessions run from console

Choose a reason for hiding this comment

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

ack.

NB there is a task about changing how those reboot/shutdown privilege requirements should go. I'll add a not that it needs to be brought to your (nemomobile/glacier) attention too when it is actually worked on.

Choose a reason for hiding this comment

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants