[PATCH wayland] introduce new headers wayland-client/server-core.h

2015-04-25 Thread Giulio Camuffo
wayland-client.h and wayland-server.h include the protocol headers generated at build time. This means that a libwayland user cannot generate and use protocol code created from a wayland.xml newer than the installed libwayand, because it is not possible to only include the API header. This commit a

Re: [PATCH wayland] introduce new headers wayland-client/server-core.h

2015-04-27 Thread Bill Spitzak
On 04/26/2015 11:48 PM, Marek Chalupa wrote: Is the --include-core-headers option necessary? Until now the scanner included wayland-client/server.h which included the %-protocol.h, but since there are inclusion guards, the protocol header was not included again - so including wayland-client/ser

Re: [PATCH wayland] introduce new headers wayland-client/server-core.h

2015-04-27 Thread Giulio Camuffo
2015-04-27 21:53 GMT+03:00 Bill Spitzak : > On 04/26/2015 11:48 PM, Marek Chalupa wrote: > >> >> Is the --include-core-headers option necessary? Until now the scanner >> included wayland-client/server.h >> which included the %-protocol.h, but since there are inclusion guards, >> the protocol header

Re: [PATCH wayland] introduce new headers wayland-client/server-core.h

2015-04-27 Thread Bill Spitzak
On 04/27/2015 01:06 PM, Giulio Camuffo wrote: 2015-04-27 21:53 GMT+03:00 Bill Spitzak : On 04/26/2015 11:48 PM, Marek Chalupa wrote: Is the --include-core-headers option necessary? Until now the scanner included wayland-client/server.h which included the %-protocol.h, but since there are in

Re: [PATCH wayland] introduce new headers wayland-client/server-core.h

2015-04-27 Thread Giulio Camuffo
2015-04-28 3:34 GMT+03:00 Bill Spitzak : > > > On 04/27/2015 01:06 PM, Giulio Camuffo wrote: >> >> 2015-04-27 21:53 GMT+03:00 Bill Spitzak : >>> >>> On 04/26/2015 11:48 PM, Marek Chalupa wrote: >>> Is the --include-core-headers option necessary? Until now the scanner included wayland

Re: [PATCH wayland] introduce new headers wayland-client/server-core.h

2015-04-27 Thread Marek Chalupa
On Mon, Apr 27, 2015 at 4:06 PM, Giulio Camuffo wrote: > 2015-04-27 21:53 GMT+03:00 Bill Spitzak : > > On 04/26/2015 11:48 PM, Marek Chalupa wrote: > > > >> > >> Is the --include-core-headers option necessary? Until now the scanner > >> included wayland-client/server.h > >> which included the %-p

Re: [PATCH wayland] introduce new headers wayland-client/server-core.h

2015-04-27 Thread Giulio Camuffo
2015-04-28 9:29 GMT+03:00 Marek Chalupa : > > > On Mon, Apr 27, 2015 at 4:06 PM, Giulio Camuffo > wrote: >> >> 2015-04-27 21:53 GMT+03:00 Bill Spitzak : >> > On 04/26/2015 11:48 PM, Marek Chalupa wrote: >> > >> >> >> >> Is the --include-core-headers option necessary? Until now the scanner >> >> in

Re: [PATCH wayland] introduce new headers wayland-client/server-core.h

2015-04-28 Thread Pekka Paalanen
On Tue, 28 Apr 2015 09:40:39 +0300 Giulio Camuffo wrote: > 2015-04-28 9:29 GMT+03:00 Marek Chalupa : > > > > > > On Mon, Apr 27, 2015 at 4:06 PM, Giulio Camuffo > > wrote: > >> > >> 2015-04-27 21:53 GMT+03:00 Bill Spitzak : > >> > On 04/26/2015 11:48 PM, Marek Chalupa wrote: > >> > > >> >> > >>

Re: [PATCH wayland] introduce new headers wayland-client/server-core.h

2015-04-28 Thread Giulio Camuffo
2015-04-28 12:48 GMT+03:00 Pekka Paalanen : > On Tue, 28 Apr 2015 09:40:39 +0300 > Giulio Camuffo wrote: > >> 2015-04-28 9:29 GMT+03:00 Marek Chalupa : >> > >> > >> > On Mon, Apr 27, 2015 at 4:06 PM, Giulio Camuffo >> > wrote: >> >> >> >> 2015-04-27 21:53 GMT+03:00 Bill Spitzak : >> >> > On 04/26

Re: [PATCH wayland] introduce new headers wayland-client/server-core.h

2015-04-28 Thread Jason Ekstrand
Pekka, Thanks for adding me to the CC. I have one general comment that I would have made inline with the code had it lasted this far. Please take this opportunity to remove the deprecated stuff by *not* putting anything deprecated into wayland-*-core.h. We have to keep it to keep old apps buildi

Re: [PATCH wayland] introduce new headers wayland-client/server-core.h

2015-04-28 Thread Bill Spitzak
The problem is when you have an extension.xml, and generate extension-client-protocol.h. Then you have some preexisting code including that and relying on it including wayland-client.h and then wayland-client-protocol.h, and if you use something from that it will not build anymore, because wayland

Re: [PATCH wayland] introduce new headers wayland-client/server-core.h

2015-04-29 Thread Giulio Camuffo
2015-04-29 2:17 GMT+03:00 Bill Spitzak : >> The problem is when you have an extension.xml, and generate >> extension-client-protocol.h. Then you have some preexisting code >> including that and relying on it including wayland-client.h and then >> wayland-client-protocol.h, and if yo

Re: [PATCH wayland] introduce new headers wayland-client/server-core.h

2015-04-29 Thread Bill Spitzak
On 04/29/2015 12:21 AM, Giulio Camuffo wrote: 2015-04-29 2:17 GMT+03:00 Bill Spitzak : The problem is when you have an extension.xml, and generate extension-client-protocol.h. Then you have some preexisting code including that and relying on it including wayland-client.h and then wayland-client-

Re: [PATCH wayland] introduce new headers wayland-client/server-core.h

2015-04-29 Thread Giulio Camuffo
2015-04-29 19:06 GMT+03:00 Bill Spitzak : > On 04/29/2015 12:21 AM, Giulio Camuffo wrote: >> >> 2015-04-29 2:17 GMT+03:00 Bill Spitzak : The problem is when you have an extension.xml, and generate extension-client-protocol.h. Then you have some preexisting code i

Re: [PATCH wayland] introduce new headers wayland-client/server-core.h

2015-04-29 Thread Bill Spitzak
On 04/29/2015 09:18 AM, Giulio Camuffo wrote: You commented it out from the wrong file. Comment it in iirc desktop-shell-client-protocol.h instead, or some other file generated from a protocol in weston. Okay I did this in weston: make clean make protocol/desktop-shell-client-

Re: [PATCH wayland] introduce new headers wayland-client/server-core.h

2015-04-30 Thread Pekka Paalanen
On Wed, 29 Apr 2015 09:06:13 -0700 Bill Spitzak wrote: > On 04/29/2015 12:21 AM, Giulio Camuffo wrote: > > Just to be clear, are you suggesting the scanner should generate two > > headers for an xml, one including the old headers and one including > > the core ones? I'm not sure i see the advanta

Re: [PATCH wayland] introduce new headers wayland-client/server-core.h

2015-04-30 Thread Bill Spitzak
On 04/30/2015 03:34 AM, Pekka Paalanen wrote: Just to be clear, my proposal is: wayland-client.h : hand-written, includes: wayland-client-core.h wayland-client-protocol.h deprecated api wayland-client-core.h : hand-written, includes wl_event_queue, wl_pr

Re: [PATCH wayland] introduce new headers wayland-client/server-core.h

2015-04-30 Thread Giulio Camuffo
2015-04-30 20:41 GMT+03:00 Bill Spitzak : > On 04/30/2015 03:34 AM, Pekka Paalanen wrote: > >>> Just to be clear, my proposal is: >>> >>> wayland-client.h : hand-written, includes: >>> wayland-client-core.h >>> wayland-client-protocol.h >>> deprecated api >>> >>> way

Re: [PATCH wayland] introduce new headers wayland-client/server-core.h

2015-04-30 Thread Bill Spitzak
On 04/30/2015 11:03 AM, Giulio Camuffo wrote: Functionally the initial patch and the latest ones are exactly the same, so not sure what you're talking about No they are not. The original patch and my proposal has *-client-protocol.h file including wayland-client-core.h. The latest one ha

Re: [PATCH wayland] introduce new headers wayland-client/server-core.h

2015-04-30 Thread Giulio Camuffo
2015-04-30 21:18 GMT+03:00 Bill Spitzak : > On 04/30/2015 11:03 AM, Giulio Camuffo wrote: > >> Functionally the initial patch and the latest ones are exactly the >> same, so not sure what you're talking about > > > No they are not. The original patch and my proposal has *-client-protocol.h > fi