search

tildearrow / kwin-lowlatency

archived - X11 full-screen unredirection and lots'a settings for KWin
373 stars 10 forks source link

5.16.90 compatibility? #47

Closed Vistaus closed 4 years ago

Vistaus commented 4 years ago

I like to live on the edge and so I use Plasma 5.16.90, but the only thing I can't use still is kwin-5.16.90 because your patch fails on the kwin-5.16.90 source code. Are you planning to make it compatible soon? :D

tildearrow commented 4 years ago

5.17 will be a re-write since most of the code breaks in that version. And I don't have time right now. Sorry...

Vistaus commented 4 years ago

No problem, mate! I understand. Hope you'll have time again at some point though 'cause I can't imagine Plasma life without kwin-lowlatency :smiley_cat:

tildearrow commented 4 years ago 
/projects/kwin/screenlockerwatcher.cpp: In member function ‘void KWin::ScreenLockerWatcher::serviceOwnerChanged(const QString&, const QString&, const QString&)’:
/projects/kwin/screenlockerwatcher.cpp:84:62: error: ‘AboutToLock’ is not a member of ‘OrgKdeScreensaverInterface’
   84 |         connect(m_kdeInterface, &OrgKdeScreensaverInterface::AboutToLock, this, &ScreenLockerWatcher::aboutToLock);
      |                                                              ^~~~~~~~~~~
make[2]: *** [CMakeFiles/kwin.dir/build.make:1109: CMakeFiles/kwin.dir/screenlockerwatcher.cpp.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:117: CMakeFiles/kwin.dir/all] Error 2
make: *** [Makefile:130: all] Error 2
Izaic commented 4 years ago

Agreed! I can't use plasma without kwin-lowlatency or wayland anymore. And wayland is just too buggy. Maybe roman's patches will solve things, but they seem to be a ways off.

tildearrow commented 4 years ago

I'm so sorry but 5.16.90 will not be possible as the 5.17 tars are already out... 5.17.0 will be available though as soon as they release it.

ShayBox commented 4 years ago

Ah I couldn't figure out why my desktop was so broken when trying kde-unstable, I remembered I was using this today :smile:

KAMiKAZOW commented 4 years ago

The kwin-lowlatency 5.16.5 binary I published in my OBS repo works just fine with Plasma 5.17beta.

Vistaus commented 4 years ago

@KAMiKAZOW Yes, it does. But that's NOT the issue here: the issue is that kwin-lowlatency currently doesn't work with KWIN 5.17

KAMiKAZOW commented 4 years ago

https://github.com/tildearrow/kwin-lowlatency/commit/07a98993b3fb643a912aeb8aa78aa357691884be 🤷

Vistaus commented 4 years ago

07a9899 🤷

The port to KWin 5.17 is still a WIP.

tildearrow commented 4 years ago

Work in progr-

tildearrow commented 4 years ago

Finally!

Vistaus commented 4 years ago

Woohoo! Who da man? You da man!

yellow-octopus-party-hard-meme-8

KAMiKAZOW commented 4 years ago

Currently updating the openSUSE package. Hopefully dependencies or so didn't change much.

robinlahtinen commented 4 years ago

Ubuntu PPA forever forgotten

KAMiKAZOW commented 4 years ago

Ubuntu PPA forever forgotten

Create one yourself. I did for openSUSE. Wasn't that hard.

tildearrow commented 4 years ago

Ubuntu PPA forever forgotten

PPA/Debian packaging concept is too difficult for me to understand and get working right...