Skip to content
  • Hrvoje Senjan's avatar
    Rename KWin binary to kwin_x11 · 52653aae
    Hrvoje Senjan authored and Martin Flöser's avatar Martin Flöser committed
    This servers two purposes.
    1. it makes KWin/5 co-installable with KWin/4 as now binary and
       all libraries etc. are renamed or installed to a different
       location.
    2. In future we need a dedicated X11 and Wayland main function
       anyway. Thus it makes most sense to rename to kwin_x11 directly
       instead of first renaming to kwin5. The reason why we need to
       have dedicated main functions is that kwin needs to check early
       whether X11 is working or Wayland is working. Right now the first
       thing kwin does is trying to connect to the XServer. This happens
       before the QApplication is constructed and before command line
       args are processed. On Wayland we won't want to test whether we
       can connect to the XServer. As it's too early to check whether we
       are starting kwin for X11 or Wayland the most convenient way is to
       have dedicated binaries - thus a rename is needed. Just renaming
       kwin for wayland is also not a good idea as in future the "main"
       kwin will be for wayland not for X11. Another case for the dedicated
       binaries is the Application class, which right now first tries to
       claim the X11 Window Manager Selection. Again on Wayland even with
       XWayland we won't need that. KWin will be the window manager for
       XWayland if KWin is the Wayland compositor. There is no need to even
       try to support anything else. Most likely it will even be KWin to
       start the XWayland server, so we can be sure that there is no other
       WM running and thus no need to claim the selection and abort if it
       fails.
    
    REVIEW: 118266
    52653aae