---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/121773/#review72897
---
Ship it!
Ship It!
- Martin Gräßlin
On Dec. 31, 2014,
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/121773/
---
(Updated Jan. 2, 2015, 2:03 p.m.)
Status
--
This change has been
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/121773/
---
Review request for KDE Frameworks.
Repository: kdesu
Description
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/121773/#review72841
---
src/kdesud/kdesud.cpp
On Dec. 31, 2014, 12:52 p.m., Jeremy Whiting wrote:
src/kdesud/kdesud.cpp, line 76
https://git.reviewboard.kde.org/r/121773/diff/1/?file=337535#file337535line76
Why not use if HAVE_X11 !defined(Q_OS_MAC) to keep that fix in place?
!defined(Q_OS_MAC) was only added after the
On Dec. 31, 2014, 5:52 a.m., Jeremy Whiting wrote:
src/kdesud/kdesud.cpp, line 76
https://git.reviewboard.kde.org/r/121773/diff/1/?file=337535#file337535line76
Why not use if HAVE_X11 !defined(Q_OS_MAC) to keep that fix in place?
Michael Palimaka wrote:
!defined(Q_OS_MAC)