[Powerdevil] [Bug 389991] Powerdevil global shortcuts not working after 5.12 config migration

[LtWorf]

https://bugs.kde.org/show_bug.cgi?id=389991

--- Comment #64 from Salvo "LtWorf" Tomaselli  ---
Ah my fault ignore that, it's confirmed (so it had a reply) but still open

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 389991] Powerdevil global shortcuts not working after 5.12 config migration

[LtWorf]

https://bugs.kde.org/show_bug.cgi?id=389991

--- Comment #63 from Salvo "LtWorf" Tomaselli  ---
that is closed too

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 389991] Powerdevil global shortcuts not working after 5.12 config migration

[bugzilla_noreply]

https://bugs.kde.org/show_bug.cgi?id=389991

--- Comment #62 from kdeb...@d3801741e.33mail.com ---
Alright, I filed https://bugs.kde.org/show_bug.cgi?id=391633 previously but got
no response.

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 389991] Powerdevil global shortcuts not working after 5.12 config migration

[Nate Graham]

https://bugs.kde.org/show_bug.cgi?id=389991

--- Comment #61 from Nate Graham  ---
kdeb...@d3801741e.33mail.com, though it looks like *this* bug is fixed, since
you're nevertheless still experiencing an issue, that means it must be caused
by something else. Please file a new bug to track that.

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 389991] Powerdevil global shortcuts not working after 5.12 config migration

[David Edmundson]

https://bugs.kde.org/show_bug.cgi?id=389991

David Edmundson  changed:

   What|Removed |Added

 Resolution|WAITINGFORINFO  |FIXED
 Status|NEEDSINFO   |RESOLVED

--- Comment #60 from David Edmundson  ---
Your settings have migrated fine.  This bug remains fixed.

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 389991] Powerdevil global shortcuts not working after 5.12 config migration

[David Edmundson]

https://bugs.kde.org/show_bug.cgi?id=389991

David Edmundson  changed:

   What|Removed |Added

 Status|RESOLVED|CLOSED

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 389991] Powerdevil global shortcuts not working after 5.12 config migration

[bugzilla_noreply]

https://bugs.kde.org/show_bug.cgi?id=389991

--- Comment #59 from kdeb...@d3801741e.33mail.com ---
Created attachment 112046
  --> https://bugs.kde.org/attachment.cgi?id=112046=edit
kglobalshortcutsrc

The kglobalshortcutsrc file is attached but it wasn't in ~/.local/share, it was
in ~/.config/

I'm running Neon 16.04.

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 389991] Powerdevil global shortcuts not working after 5.12 config migration

[LtWorf]

https://bugs.kde.org/show_bug.cgi?id=389991

--- Comment #58 from Salvo "LtWorf" Tomaselli  ---
This seems like a different issue to me.

I have the same on my work laptop, I can't control the brightness with the app,
so i had to write a shell script to do it. However it is not a problem with the
shortcuts since the slider in the app is missing too.

I tried asking in IRC but nobody had a clue on what could be the issue, since
the files in /sys to control it exist, and the slider to control the keyboard
brightness is present.

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 389991] Powerdevil global shortcuts not working after 5.12 config migration

[David Edmundson]

https://bugs.kde.org/show_bug.cgi?id=389991

David Edmundson  changed:

   What|Removed |Added

 Resolution|--- |WAITINGFORINFO
 Status|REOPENED|NEEDSINFO

--- Comment #57 from David Edmundson  ---
please provide your ~/.local/share/kglobalshortcutsrc

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 389991] Powerdevil global shortcuts not working after 5.12 config migration

[bugzilla_noreply]

https://bugs.kde.org/show_bug.cgi?id=389991

kdeb...@d3801741e.33mail.com changed:

   What|Removed |Added

 Status|RESOLVED|REOPENED
 Resolution|FIXED   |---
 CC||kdeb...@d3801741e.33mail.co
   ||m

--- Comment #56 from kdeb...@d3801741e.33mail.com ---
Hi folks,

Still experiencing this on a Dell laptop. The brightness can't be controlled
from either the keyboard or the Battery and Brightness widget. What information
can I provide to start troubleshooting?

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 389991] Powerdevil global shortcuts not working after 5.12 config migration

[bugzilla_noreply]

https://bugs.kde.org/show_bug.cgi?id=389991

--- Comment #55 from stanisl...@null.net ---
@David, Today I was able to solve my issue after reading comment from bug
390930. I have reverted all global shortcuts to their default values and now it
works. But I am not aware that I would change anything in shortcuts settings
before.

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 389991] Powerdevil global shortcuts not working after 5.12 config migration

[David Edmundson]

https://bugs.kde.org/show_bug.cgi?id=389991

--- Comment #54 from David Edmundson  ---
@stan, please provide your ~/.local/share/kglobalshortcutsrc

Did you touch the settings manually between 5.12.0 and 5.12.32?

also ideally the output of running dbus-monitor then pressing the shortcut

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 389991] Powerdevil global shortcuts not working after 5.12 config migration

[Kai Uwe Broulik]

https://bugs.kde.org/show_bug.cgi?id=389991

Kai Uwe Broulik  changed:

   What|Removed |Added

 CC||tipos...@tiscali.it

--- Comment #53 from Kai Uwe Broulik  ---
*** Bug 390930 has been marked as a duplicate of this bug. ***

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 389991] Powerdevil global shortcuts not working after 5.12 config migration

[bugzilla_noreply]

https://bugs.kde.org/show_bug.cgi?id=389991

stanisl...@null.net changed:

   What|Removed |Added

 CC||stanisl...@null.net

--- Comment #52 from stanisl...@null.net ---
Hello,
after update to the latest version I can lower the brightness of the screen but
can not increase it with use of keyboar shortcuts Fn+F11 and Fn+F12 (before the
update nothing worked). I use Dell Latitude E5550.  I think it wasn't fully
fixed. Thank you.

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 389991] Powerdevil global shortcuts not working after 5.12 config migration

[gbyte . co]

https://bugs.kde.org/show_bug.cgi?id=389991

gbyte.co  changed:

   What|Removed |Added

 CC|cont...@gbyte.co|

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 389991] Powerdevil global shortcuts not working after 5.12 config migration

[Kai Uwe Broulik]

https://bugs.kde.org/show_bug.cgi?id=389991

Kai Uwe Broulik  changed:

   What|Removed |Added

 CC||rockonthemoo...@gmail.com

--- Comment #51 from Kai Uwe Broulik  ---
*** Bug 390850 has been marked as a duplicate of this bug. ***

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 389991] Powerdevil global shortcuts not working after 5.12 config migration

[Pete Theisen]

https://bugs.kde.org/show_bug.cgi?id=389991

--- Comment #50 from Pete Theisen  ---
After the update last night update, and a reboot, it is working. Thanks.

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 389991] Powerdevil global shortcuts not working after 5.12 config migration

[Nate Graham]

https://bugs.kde.org/show_bug.cgi?id=389991

Nate Graham  changed:

   What|Removed |Added

 CC||petethei...@verizon.net

--- Comment #49 from Nate Graham  ---
*** Bug 390795 has been marked as a duplicate of this bug. ***

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 389991] Powerdevil global shortcuts not working after 5.12 config migration

[ulusu]

https://bugs.kde.org/show_bug.cgi?id=389991

--- Comment #48 from ulusu  ---
Manjaro linux just received "powerdevil 5.12.1-1.1" update. Problem solved.

Regards,
ulusu

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 389991] Powerdevil global shortcuts not working after 5.12 config migration

[francisco_t]

https://bugs.kde.org/show_bug.cgi?id=389991

--- Comment #47 from francisco_t  ---
(In reply to David Edmundson from comment #46)
> Git commit 4ae36aee91a23dcb0736418295269da14152 by David Edmundson.
> Committed on 20/02/2018 at 12:26.
> Pushed by davidedmundson into branch 'Plasma/5.12'.
> 
> Fix PowerDevil shortcuts migration
> 
> Summary:
> 5.11 -> 5.12 migrated shortcuts from kded5 to the correct component name
> org_kde_powerdevil for good reasons
> However, despite a kconfupdate script working correctly and moving the
> old keys, kglobalaccel is running whilst we update and it syncs the old
> values later. On future loads it detects the duplication and removes the
> powerdevil entries.
> 
> This patch removes the old powermanagent entries in the kded5 component
> at powerdevil startup which is at runtime where we can talk to
> kglobalaccel, but before powerdevil has registered the new actions.
> 
> This should handle both users that haven't upgraded to 5.12 yet, as well
> as users who hit the previous config script.
> 
> Ideally kconf_update needs to be patched to do migration at appropriate
> times.
> FIXED-IN: 5.12.2
> 
> Test Plan:
> "broke" my config by putting in values in the old place with a custom
> shortcut for poweroff
> Restarted kglobalaccel
> Ran powerdevil
> Shortucts worked
> Cleanly quit kglobalaccel
> Config stayed synced
> 
> Reviewers: #plasma, broulik
> 
> Reviewed By: #plasma, broulik
> 
> Subscribers: broulik, fvogt, aacid, cfeck, plasma-devel
> 
> Tags: #plasma
> 
> Differential Revision: https://phabricator.kde.org/D10668
> 
> M  +0-2CMakeLists.txt
> M  +55   -0daemon/powerdevilapp.cpp
> M  +1-0daemon/powerdevilapp.h
> D  +0-1kconf_update/CMakeLists.txt
> D  +0-12   kconf_update/powerdevil_move_shortcuts.upd
> 
> https://commits.kde.org/powerdevil/4ae36aee91a23dcb0736418295269da14152

Will the fix work for multiple users with different languages?
The last update updated my spanish (root) user, but not in the english users.

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 389991] Powerdevil global shortcuts not working after 5.12 config migration

[David Edmundson]

https://bugs.kde.org/show_bug.cgi?id=389991

David Edmundson  changed:

   What|Removed |Added

   Version Fixed In||5.12.2
 Status|CONFIRMED   |RESOLVED
  Latest Commit||https://commits.kde.org/pow
   ||erdevil/4ae36aee91a23dc
   ||b0736418295269da14152
 Resolution|--- |FIXED

--- Comment #46 from David Edmundson  ---
Git commit 4ae36aee91a23dcb0736418295269da14152 by David Edmundson.
Committed on 20/02/2018 at 12:26.
Pushed by davidedmundson into branch 'Plasma/5.12'.

Fix PowerDevil shortcuts migration

Summary:
5.11 -> 5.12 migrated shortcuts from kded5 to the correct component name
org_kde_powerdevil for good reasons
However, despite a kconfupdate script working correctly and moving the
old keys, kglobalaccel is running whilst we update and it syncs the old
values later. On future loads it detects the duplication and removes the
powerdevil entries.

This patch removes the old powermanagent entries in the kded5 component
at powerdevil startup which is at runtime where we can talk to
kglobalaccel, but before powerdevil has registered the new actions.

This should handle both users that haven't upgraded to 5.12 yet, as well
as users who hit the previous config script.

Ideally kconf_update needs to be patched to do migration at appropriate
times.
FIXED-IN: 5.12.2

Test Plan:
"broke" my config by putting in values in the old place with a custom shortcut
for poweroff
Restarted kglobalaccel
Ran powerdevil
Shortucts worked
Cleanly quit kglobalaccel
Config stayed synced

Reviewers: #plasma, broulik

Reviewed By: #plasma, broulik

Subscribers: broulik, fvogt, aacid, cfeck, plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D10668

M  +0-2CMakeLists.txt
M  +55   -0daemon/powerdevilapp.cpp
M  +1-0daemon/powerdevilapp.h
D  +0-1kconf_update/CMakeLists.txt
D  +0-12   kconf_update/powerdevil_move_shortcuts.upd

https://commits.kde.org/powerdevil/4ae36aee91a23dcb0736418295269da14152

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 389991] Powerdevil global shortcuts not working after 5.12 config migration

[Christoph Feck]

https://bugs.kde.org/show_bug.cgi?id=389991

--- Comment #45 from Christoph Feck  ---
Proposed patch at https://phabricator.kde.org/D10668

With immediate positive feedback, this could land in Plasma 5.12.2.

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 389991] Powerdevil global shortcuts not working after 5.12 config migration

[Michał Dybczak]

https://bugs.kde.org/show_bug.cgi?id=389991

--- Comment #44 from Michał Dybczak  ---
(In reply to David Edmundson from comment #42)
> A secondary migration will be in 5.12.2
> 
> >What gives?
> 
> Same reason as my original update script didn't work; kglobalaccel is still
> running whilst you're editing it.

Thanks! I switched to LXQt session, made the changes, relogged to Plasma and
now it works! Power button and screen brightness work as intended. That is the
peace of the advice that is missing in comment 22.

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 389991] Powerdevil global shortcuts not working after 5.12 config migration

[ulusu]

https://bugs.kde.org/show_bug.cgi?id=389991

--- Comment #43 from ulusu  ---
I don't know. Maybe the fact that I'm running Manjaro KDE. I edited the
original file and logged out and in. Then to be sure I rebooted. Still with
brightness keys working since yesterday evening. Rechecked my
~/.config/kglobalshortcutsrc. Still the same.

<>

[kded5]
Decrease Keyboard Brightness=none,Keyboard Brightness Down,Decrease Keyboard
Brightness
Decrease Screen Brightness=none,Monitor Brightness Down,Decrease Screen
Brightness
Hibernate=none,Hibernate,Hibernate
Increase Keyboard Brightness=none,Keyboard Brightness Up,Increase Keyboard
Brightness
Increase Screen Brightness=none,Monitor Brightness Up,Increase Screen
Brightness
PowerOff=none,Power Off,Power Off
Show System Activity=none,Ctrl+Esc,Show System Activity
Sleep=none,Sleep,Suspend
Toggle Keyboard Backlight=none,Keyboard Light On/Off,Toggle Keyboard Backlight
_k_friendly_name=KDE Daemon
display=none,Display\tMeta+P,Switch Display

[org_kde_powerdevil]
Decrease Keyboard Brightness=Keyboard Brightness Down,Keyboard Brightness
Down,Decrease Keyboard Brightness
Decrease Screen Brightness=Monitor Brightness Down,Monitor Brightness
Down,Decrease Screen Brightness
Hibernate=Hibernate,Hibernate,Hibernate
Increase Keyboard Brightness=Keyboard Brightness Up,Keyboard Brightness
Up,Increase Keyboard Brightness
Increase Screen Brightness=Monitor Brightness Up,Monitor Brightness Up,Increase
Screen Brightness
PowerOff=Power Off,Power Off,Power Off
Show System Activity=Ctrl+Esc,Ctrl+Esc,Show System Activity
Sleep=Sleep,Sleep,Suspend
Toggle Keyboard Backlight=Keyboard Light On/Off,Keyboard Light On/Off,Toggle
Keyboard Backlight
_k_friendly_name=Power Management
display=Display\tMeta+P,Display\tMeta+P,Switch Display


Regards,
ulusu

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 389991] Powerdevil global shortcuts not working after 5.12 config migration

[David Edmundson]

https://bugs.kde.org/show_bug.cgi?id=389991

--- Comment #42 from David Edmundson  ---
A secondary migration will be in 5.12.2

>What gives?

Same reason as my original update script didn't work; kglobalaccel is still
running whilst you're editing it.

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 389991] Powerdevil global shortcuts not working after 5.12 config migration

[Michał Dybczak]

https://bugs.kde.org/show_bug.cgi?id=389991

--- Comment #41 from Michał Dybczak  ---
Unfortunately not. I edited original ~/.config/kglobalshortcutsrc file with
kwrite and still the same - it gets overwritten with every logoff/ login so it
never works :(.
I checked permissions and I had oddly 0777 on original file so I lowered it to
0755 but still no luck, it gets overwritten. What gives?

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 389991] Powerdevil global shortcuts not working after 5.12 config migration

[ulusu]

https://bugs.kde.org/show_bug.cgi?id=389991

--- Comment #40 from ulusu  ---
I edited the original file. Not like in the procedure. I don't know if this has
something to do with it.

Regards,
ulusu

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 389991] Powerdevil global shortcuts not working after 5.12 config migration

[Michał Dybczak]

https://bugs.kde.org/show_bug.cgi?id=389991

--- Comment #39 from Michał Dybczak  ---
Thanks, @ulusu, it now makes more sense. However, I switched [kded5] lines with
[org_kde_powerdevil] but it still doesn't work. After I log off/log in, old
lines (or at least some of them) get restored. I did edit it with GUI editor
(kwrite).
How can I prevent restoring old, bugged settings? Why doesn't it stick?

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 389991] Powerdevil global shortcuts not working after 5.12 config migration

[David Edmundson]

https://bugs.kde.org/show_bug.cgi?id=389991

David Edmundson  changed:

   What|Removed |Added

 Ever confirmed|0   |1
Summary|Sleep key no longer |Powerdevil global shortcuts
   |initiates sleep |not working after 5.12
   ||config migration
 Status|UNCONFIRMED |CONFIRMED

-- 
You are receiving this mail because:
You are watching all bug changes.