---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123075/
---
(Updated June 1, 2017, 1:20 p.m.)
Status
--
This change has been
On March 20, 2015, 8:07 a.m., Martin Gräßlin wrote:
as in other similar requests: -2 from my side
Martin Gräßlin wrote:
To extend: I think the way is wrong. If it now builds on MacOS the
required is wrong. It should be an optional find_package properly ifdefed.
Christoph Cullmann
On March 20, 2015, 8:07 a.m., Martin Gräßlin wrote:
as in other similar requests: -2 from my side
Martin Gräßlin wrote:
To extend: I think the way is wrong. If it now builds on MacOS the
required is wrong. It should be an optional find_package properly ifdefed.
Christoph Cullmann
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123075/#review9
---
as in other similar requests: -2 from my side
- Martin
On March 20, 2015, 8:07 a.m., Martin Gräßlin wrote:
as in other similar requests: -2 from my side
To extend: I think the way is wrong. If it now builds on MacOS the required is
wrong. It should be an optional find_package properly ifdefed.
- Martin
On March 20, 2015, 8:07 a.m., Martin Gräßlin wrote:
as in other similar requests: -2 from my side
Martin Gräßlin wrote:
To extend: I think the way is wrong. If it now builds on MacOS the
required is wrong. It should be an optional find_package properly ifdefed.
Christoph Cullmann
On March 20, 2015, 8:07 a.m., Martin Gräßlin wrote:
as in other similar requests: -2 from my side
Martin Gräßlin wrote:
To extend: I think the way is wrong. If it now builds on MacOS the
required is wrong. It should be an optional find_package properly ifdefed.
Christoph Cullmann
On March 20, 2015, 8:07 a.m., Martin Gräßlin wrote:
as in other similar requests: -2 from my side
Martin Gräßlin wrote:
To extend: I think the way is wrong. If it now builds on MacOS the
required is wrong. It should be an optional find_package properly ifdefed.
Christoph Cullmann
On March 20, 2015, 7:07 a.m., Martin Gräßlin wrote:
as in other similar requests: -2 from my side
Martin Gräßlin wrote:
To extend: I think the way is wrong. If it now builds on MacOS the
required is wrong. It should be an optional find_package properly ifdefed.
Christoph Cullmann
On March 20, 2015, 7:07 a.m., Martin Gräßlin wrote:
as in other similar requests: -2 from my side
Martin Gräßlin wrote:
To extend: I think the way is wrong. If it now builds on MacOS the
required is wrong. It should be an optional find_package properly ifdefed.
Actually, you don't
On March 20, 2015, 7:07 a.m., Martin Gräßlin wrote:
as in other similar requests: -2 from my side
Martin Gräßlin wrote:
To extend: I think the way is wrong. If it now builds on MacOS the
required is wrong. It should be an optional find_package properly ifdefed.
Christoph Cullmann
On March 20, 2015, 8:07 a.m., Martin Gräßlin wrote:
as in other similar requests: -2 from my side
Martin Gräßlin wrote:
To extend: I think the way is wrong. If it now builds on MacOS the
required is wrong. It should be an optional find_package properly ifdefed.
Christoph Cullmann
On March 20, 2015, 7:07 a.m., Martin Gräßlin wrote:
as in other similar requests: -2 from my side
Martin Gräßlin wrote:
To extend: I think the way is wrong. If it now builds on MacOS the
required is wrong. It should be an optional find_package properly ifdefed.
Christoph Cullmann
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123075/#review3
---
Ship it!
Ship It!
- Jeremy Whiting
On March 19, 2015,
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123075/
---
Review request for KDE Frameworks and Michael Palimaka.
Repository:
15 matches
Mail list logo