Skip to content
New issue

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

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

Already on GitHub? # to your account

elogind: update to 246.10. #38347

Merged
merged 1 commit into from
Sep 13, 2022
Merged

Conversation

motorto
Copy link
Contributor

@motorto motorto commented Jul 28, 2022

Testing the changes

  • I tested the changes in this PR: YES

Local build testing

  • I built this PR locally for my native architecture, x86_64-glibc
  • I built this PR locally for these architectures (if supported. mark crossbuilds):
    • armv6l

Going to test on a vm using seatd to see if the errors still occurs

@Gottox in case you didn't see the message IRC could you check if the problem with libseat still occurs ? Ok my quick test it doesn't, but to be honest I don't have a lot of experience it with

@paper42 paper42 added the needs-testing Testing a PR or reproducing an issue needed label Jul 29, 2022
@motorto
Copy link
Contributor Author

motorto commented Aug 2, 2022

Been using this for 3 days on my main system without a problem.(x86_64-glibc)
Hum found an issue, cant seem to shutdown the system with loginctl poweroff && loginctl reboot
Get the following error: 2022-08-03T21:44:13.00727 daemon.err: Aug 3 22:44:13 loginctl[15860]: Failed to power off system via elogind: Access denied
I am able to loginctl lock-session and screen locks and to suspend with loginctl suspend. Weird

Talked in the IRC and started building with the polkit option to true since without even polkit installed the loginctl poweroff/reboot wont work

  • I will create a pr documenting this change to void handbooks

@motorto
Copy link
Contributor Author

motorto commented Aug 11, 2022

Using this for more than a week even with the new dbus update and it works fine. (I know I am biased, but I don't see someone else testing it)

@oreo639
Copy link
Member

oreo639 commented Sep 11, 2022

Tested it with sway and gnome and haven't noticed any issues.

@paper42
Copy link
Member

paper42 commented Sep 13, 2022

I have been testing this for about a month with GNOME

@paper42 paper42 removed the needs-testing Testing a PR or reproducing an issue needed label Sep 13, 2022
@paper42 paper42 merged commit aacf5ad into void-linux:master Sep 13, 2022
@motorto motorto deleted the elogind-246.10 branch September 22, 2022 12:59
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants