Add Wayland support
ClosedPublic
Actions

Authored by stikonas on Feb 10 2019, 7:40 PM.

Details

Reviewers

svuorela

Commits

R436:94c7c9ada210: Add Wayland support

Test Plan

krdc can now connect on Wayland session.

The remote window is not embedded in krdc yet but still better than not connecting at all.

Repository

R436 KRDC

Lint

Lint Skipped

Unit

Unit Tests Skipped

stikonas requested review of this revision.Feb 10 2019, 7:40 PM

stikonas created this revision.

stikonas retitled this revision from Add wayland support to Add Wayland support.

Coding style: it looks like most places in this file uses {} also around one line ifs.

rdp/rdpview.cpp
366–369	Isn't there something in KWindowSystem that does this prettier ? I think I have reviewed something like this in the past there.
367	Are there proper error handling fi the executables aren't found?

Added more error handling if executable is not found.

Use KWindowSystem to detect Wayland

Coding style adjusted

This revision was not accepted when it landed; it landed in state Needs Review.Feb 12 2019, 11:29 PM

This revision was automatically updated to reflect the committed changes.

Diff	ID	Base	Description	Created	Lint	Unit
Base			Base
Diff 1	51345			Feb 10 2019, 7:38 PM	★	★
Diff 2	51355			Feb 10 2019, 9:06 PM	★	★
Diff 3	51369			Feb 10 2019, 10:20 PM	★	★
Diff 4	51560	8b0d28a	R436:94c7c9ada2103ef8274dd6046e69fe82d7b755d2	Feb 12 2019, 11:25 PM	★	★