----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115723/#review49747 -----------------------------------------------------------
src/core/kprotocolmanager.cpp <https://git.reviewboard.kde.org/r/115723/#comment34981> Maybe move the Q_OS_MAC up as the first check, then there is no need for !defined(Q_OS_DARWIN) - Alexander Richardson On Feb. 13, 2014, 2:03 p.m., Martin Gräßlin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/115723/ > ----------------------------------------------------------- > > (Updated Feb. 13, 2014, 2:03 p.m.) > > > Review request for KDE Frameworks, Dawit Alemayehu and Bernhard Beschow. > > > Repository: kio > > > Description > ------- > > Use Q_OS_UNIX instead of HAVE_X11 to determine the platform we are on > > We cannot properly determine the windowing system platform on unix > like systems in kprotocolmanager as it's not linking gui. Thus we > don't know whether we are on X11 or Wayland and there is no proper > way to figure it out, because both DISPLAY and WAYLAND_DISPLAY could > be defined. > > As a solution we just force the platform to be always X11 when we > are on unix like systems (modulo mac). > > > Diffs > ----- > > src/core/kprotocolmanager.cpp f81b6797887eebd868c36b98e867eb055b05a1e2 > > Diff: https://git.reviewboard.kde.org/r/115723/diff/ > > > Testing > ------- > > > Thanks, > > Martin Gräßlin > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel