Re: Review Request 120579: [KWayland] Install headers for WaylandServer

2014-10-14 Thread Martin Gräßlin
> On Oct. 14, 2014, 1:50 p.m., Martin Gräßlin wrote: > > -2, it's not installed as the WaylandServer still needs work and is not in > > a state yet to provide ABI. > > Sebastian Kügler wrote: > Well, neither is WaylandServer, the point at this point in time is to > allow sharing code, no?

Re: Review Request 120579: [KWayland] Install headers for WaylandServer

2014-10-14 Thread Sebastian Kügler
> On Oct. 14, 2014, 11:50 a.m., Martin Gräßlin wrote: > > -2, it's not installed as the WaylandServer still needs work and is not in > > a state yet to provide ABI. > > Sebastian Kügler wrote: > Well, neither is WaylandServer, the point at this point in time is to > allow sharing code, no?

Re: Review Request 120579: [KWayland] Install headers for WaylandServer

2014-10-14 Thread Martin Gräßlin
> On Oct. 14, 2014, 1:50 p.m., Martin Gräßlin wrote: > > -2, it's not installed as the WaylandServer still needs work and is not in > > a state yet to provide ABI. > > Sebastian Kügler wrote: > Well, neither is WaylandServer, the point at this point in time is to > allow sharing code, no?

Re: Review Request 120579: [KWayland] Install headers for WaylandServer

2014-10-14 Thread Sebastian Kügler
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120579/ --- (Updated Oct. 14, 2014, 12:49 p.m.) Status -- This change has been m

Re: Review Request 120579: [KWayland] Install headers for WaylandServer

2014-10-14 Thread Martin Gräßlin
> On Oct. 14, 2014, 1:50 p.m., Martin Gräßlin wrote: > > -2, it's not installed as the WaylandServer still needs work and is not in > > a state yet to provide ABI. > > Sebastian Kügler wrote: > Well, neither is WaylandServer, the point at this point in time is to > allow sharing code, no?

Re: Review Request 120579: [KWayland] Install headers for WaylandServer

2014-10-14 Thread Sebastian Kügler
> On Oct. 14, 2014, 11:54 a.m., Martin Gräßlin wrote: > > that said: I'm happy to take a patch to fix the export header ;-) Sure, I can push that part separately (with commented INSTALL directive), but let's sort that thread of the discussion out, first. - Sebastian

Re: Review Request 120579: [KWayland] Install headers for WaylandServer

2014-10-14 Thread Sebastian Kügler
> On Oct. 14, 2014, 11:50 a.m., Martin Gräßlin wrote: > > -2, it's not installed as the WaylandServer still needs work and is not in > > a state yet to provide ABI. Well, neither is WaylandServer, the point at this point in time is to allow sharing code, no? Is there another way to easily sta

Re: Review Request 120579: [KWayland] Install headers for WaylandServer

2014-10-14 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120579/#review68384 --- that said: I'm happy to take a patch to fix the export header

Re: Review Request 120579: [KWayland] Install headers for WaylandServer

2014-10-14 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120579/#review68383 --- -2, it's not installed as the WaylandServer still needs work a

Review Request 120579: [KWayland] Install headers for WaylandServer

2014-10-14 Thread Sebastian Kügler
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120579/ --- Review request for kwin, Plasma and Martin Gräßlin. Repository: kwayland