davidedmundson accepted this revision.
davidedmundson added a comment.
This revision is now accepted and ready to land.


  > [14:12] <d_ed> DavidRedondo1: my understanding is that a system might ship 
"konsole opens with control+t"    . The UI allows you to remove that. This 
would remove the entry in kglobalshortcutsrc, but because it's still  in the 
system defaults file as soon as you log in again it'll add it back
  
  [14:25] <DavidRedondo1> d_ed, fvogt Apparently the runtime writes the hidden 
thing when a component is cleanedUp 
https://cgit.kde.org/kglobalaccel.git/tree/src/runtime/kserviceactioncomponent.cpp#n135
  [14:27] <DavidRedondo1> Does that fail or something when the file is not 
writeable?
  [14:31] <DavidRedondo1> I think it fails
  [14:31] <DavidRedondo1> I just tested it
  
  if it is indeed broken...then we may as well just merge this.

REPOSITORY
  R268 KGlobalAccel

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D24956

To: meven, mart, #plasma, fvogt, apol, davidedmundson
Cc: davidedmundson, davidre, kde-frameworks-devel, LeGast00n, cblack, michaelh, 
ngraham, bruns

Reply via email to