Re: [PATCH wayland-web 1/3] toolkits: Don't say the core protocol is changing

2015-08-24 Thread Bryce Harrington
On Tue, Aug 25, 2015 at 12:01:46PM +0800, Jonas Ådahl wrote: > The core protocol is not changing. Experimental protocols may be, so say > that instead. > > Signed-off-by: Jonas Ådahl LGTM. For all three: Reviewed-by: Bryce Harrington Pushed to trunk: db4285d..f7cbaeb master -> master >

Re: [PATCH wayland v2 1/2] Extend WAYLAND_DISPLAY and name parameter semantics to support absolute paths.

2015-08-24 Thread Bryce Harrington
On Thu, Aug 06, 2015 at 03:30:58PM +0200, Marek Chalupa wrote: > Hi, > > On 08/06/2015 01:49 PM, Davide Bettio wrote: > >Hello, > > > >On 2015-08-06 10:37, Marek Chalupa wrote: > >>the patch does not apply in the first place: > >> > >>Applying: Extend WAYLAND_DISPLAY and name parameter semantics t

[PATCH wayland-web 1/3] toolkits: Don't say the core protocol is changing

2015-08-24 Thread Jonas Ådahl
The core protocol is not changing. Experimental protocols may be, so say that instead. Signed-off-by: Jonas Ådahl --- toolkits.html | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/toolkits.html b/toolkits.html index 6ada2d8..875ead3 100644 --- a/toolkits.html +++ b/toolk

Re: [PATCH weston 3/3] wayland: port the wayland backend to the new api

2015-08-24 Thread Bryce Harrington
On Thu, Aug 13, 2015 at 01:53:06PM +0300, Pekka Paalanen wrote: > From: Giulio Camuffo So much is being moved around here that it's hard to assure that no bugs got introduced. It feels like there's 2 or 3 refactoring steps being done in one go; if we're dead set on landing this for beta perhaps

Re: [PATCH weston 2/3] drm: port the drm backend to the new init api

2015-08-24 Thread Bryce Harrington
On Thu, Aug 13, 2015 at 01:53:05PM +0300, Pekka Paalanen wrote: > From: Giulio Camuffo This is a big patch with a lot of changes, and I'm worried about landing it right as we're on the eve of beta. If it could be broken up into smaller easy-to-review bits, it might make it more digestible... Th

Re: [PATCH weston 1/3] compositor: prep for migration to new config system

2015-08-24 Thread Bryce Harrington
On Thu, Aug 13, 2015 at 12:30:01PM +0200, Giulio Camuffo wrote: > From: Pekka Paalanen > > Add new configuration argument to the backend_init() function, which > will replace the current argc, argv, and config arguments. > > Next, each backend is converted individually, after which the unused >

Re: [PATCH v3 weston] Introduce pointer locking and confinement protocol

2015-08-24 Thread Jonas Ådahl
On Mon, Aug 24, 2015 at 10:44:55AM -0700, Bill Spitzak wrote: > On Sun, Aug 23, 2015 at 8:19 PM, Jonas Ådahl wrote: > > > > > 1. the pointer "attaches" to the UI element. > > > > One would create a lock region on the scroll handle. When enabled one > > would hide the cursor, create an identical c

Re: [PATCH weston 0/3] start making backends not weston specific

2015-08-24 Thread Bryce Harrington
On Mon, Aug 17, 2015 at 11:29:01AM +0300, Pekka Paalanen wrote: > On Thu, 13 Aug 2015 12:30:00 +0200 > Giulio Camuffo wrote: > > > The first patch by Pekka adds some functions to ease a smooth migration > > from the old way of initializing the backends to a new one which is > > libweston friendly

Re: [PATCH weston v2] ivi-layout: implement surface clipping

2015-08-24 Thread Bryce Harrington
On Mon, Aug 24, 2015 at 01:21:21PM +0300, Pekka Paalanen wrote: > On Mon, 24 Aug 2015 10:24:15 +0900 > Nobuhiko Tanibata wrote: > > > From: Nobuhiko Tanibata > > > > View clip region is set in surface-local coordinates. To compute that > > region, the ivi-layer destination rectangle in the glob

Re: Weston test suite

2015-08-24 Thread Jon A. Cruz
On 08/24/2015 02:50 AM, Pekka Paalanen wrote: > On Sat, 22 Aug 2015 00:00:58 -0700 > Bryce Harrington wrote: > >> On Fri, Aug 21, 2015 at 10:35:34AM +0300, Pekka Paalanen wrote: >>> On Thu, 20 Aug 2015 11:05:39 -0700 >>> Bryce Harrington wrote: >>> > (I wish we had some tests with coverage

Re: [PATCH weston] build, compositor-drm: fix output name constants another way

2015-08-24 Thread Bryce Harrington
On Mon, Aug 24, 2015 at 12:32:33PM -0500, Derek Foreman wrote: > On 24/08/15 06:27 AM, Pekka Paalanen wrote: > > From: Pekka Paalanen > > > > Bumping libdrm requirement by 3 years just for output connector name > > constants was a bit much. Fix the problem introduced in > > 89c49b3060a115e846ba1e

Re: [PATCH weston] build: Require newer version of libdrm

2015-08-24 Thread Bryce Harrington
On Mon, Aug 24, 2015 at 01:06:45PM +0200, Eugen Friedrich wrote: > Hello dear community, > > in our current project we are trying to follow the weston master branch as > much as possible, Hi Eugen, can you tell us more about your project? Bryce > recently introduced dependency to libdrm 2.4.59

Re: Libinput: Halfkey/Mirrorboard implementation

2015-08-24 Thread Daniel Stone
On 24 August 2015 at 18:57, Bill Spitzak wrote: > There is considerable dislike for this idea from the Wayland developers, > though I am stumped why. Not really sure how you're stumped, given that I've reiterated the reason several times. ___ wayland-de

Re: Libinput: Halfkey/Mirrorboard implementation

2015-08-24 Thread Bill Spitzak
On Sun, Aug 23, 2015 at 11:38 AM, Kieran Bingham wrote: > > > This is a very common request, usually for the opposite reason: to make a > > modifier also produce a keysym. In particular it would allow Ctrl or > AltGr > > to act as a compose prefix, something a lot of plugins for Windows does, > >

Re: [PATCH v3 weston] Introduce pointer locking and confinement protocol

2015-08-24 Thread Bill Spitzak
On Sun, Aug 23, 2015 at 8:19 PM, Jonas Ådahl wrote: > > 1. the pointer "attaches" to the UI element. > > One would create a lock region on the scroll handle. When enabled one > would hide the cursor, create an identical cursor surface and add it as > a subsurface positioning relative to the scrol

Re: [PATCH weston] build, compositor-drm: fix output name constants another way

2015-08-24 Thread Derek Foreman
On 24/08/15 06:27 AM, Pekka Paalanen wrote: > From: Pekka Paalanen > > Bumping libdrm requirement by 3 years just for output connector name > constants was a bit much. Fix the problem introduced in > 89c49b3060a115e846ba1e7fbef94d14894244f2 by conditionally using the new > additions. > > Both VI

Re: [PATCH weston] build: Require newer version of libdrm

2015-08-24 Thread Pekka Paalanen
On Mon, 24 Aug 2015 14:16:17 +0300 Pekka Paalanen wrote: > On Mon, 24 Aug 2015 13:06:45 +0200 > Eugen Friedrich wrote: > > > Hello dear community, > > > > in our current project we are trying to follow the weston master branch as > > much as possible, > > recently introduced dependency to libd

[PATCH weston] build, compositor-drm: fix output name constants another way

2015-08-24 Thread Pekka Paalanen
From: Pekka Paalanen Bumping libdrm requirement by 3 years just for output connector name constants was a bit much. Fix the problem introduced in 89c49b3060a115e846ba1e7fbef94d14894244f2 by conditionally using the new additions. Both VIRTUAL and DSI came in the same libdrm commit 566c3ce877a4be7

Re: [PATCH weston] build: Require newer version of libdrm

2015-08-24 Thread Pekka Paalanen
On Mon, 24 Aug 2015 13:06:45 +0200 Eugen Friedrich wrote: > Hello dear community, > > in our current project we are trying to follow the weston master branch as > much as possible, > recently introduced dependency to libdrm 2.4.59 will prevent us to use > weston 1.9, > The patches for naming cor

Re: [PATCH weston] build: Require newer version of libdrm

2015-08-24 Thread Eugen Friedrich
Hello dear community, in our current project we are trying to follow the weston master branch as much as possible, recently introduced dependency to libdrm 2.4.59 will prevent us to use weston 1.9, The patches for naming correction are good and important. so we would love to see a missing defines

Re: [PATCH weston v2] ivi-layout: implement surface clipping

2015-08-24 Thread Pekka Paalanen
On Mon, 24 Aug 2015 10:24:15 +0900 Nobuhiko Tanibata wrote: > From: Nobuhiko Tanibata > > View clip region is set in surface-local coordinates. To compute that > region, the ivi-layer destination rectangle in the global coordinates > are transformed back into the surface-local coordinates. > >

Re: [PATCH wayland] Revert "client: require WAYLAND_DISPLAY to be set"

2015-08-24 Thread Pekka Paalanen
On Mon, 17 Aug 2015 16:17:57 +0300 Pekka Paalanen wrote: > From: Pekka Paalanen > > This reverts commit fb7e13021730d0a5516ecbd3712ea4235e05d24d. > > Developers have been trying to reduce the number of by default required > environment variables, and the mentioned commit is a step backwards in

Weston test suite

2015-08-24 Thread Pekka Paalanen
On Sat, 22 Aug 2015 00:00:58 -0700 Bryce Harrington wrote: > On Fri, Aug 21, 2015 at 10:35:34AM +0300, Pekka Paalanen wrote: > > On Thu, 20 Aug 2015 11:05:39 -0700 > > Bryce Harrington wrote: > > > > > (I wish we had some tests with coverage of this chunk of code, so I could > > > give my usua

Re: wl_surface.attach with NULL wl_buffer behaviour

2015-08-24 Thread Pekka Paalanen
On Fri, 21 Aug 2015 19:04:11 -0500 Prabhu S wrote: > Below is the case where I'm getting stuck with the actual test case. > Wondering why there is no callback wl_callb...@25.done or > wl_buffer@29.release > The EGL implementation depends on these callback/buffer release. You will not get wl_call

Re: wl_surface.attach with NULL wl_buffer behaviour

2015-08-24 Thread Pekka Paalanen
On Fri, 21 Aug 2015 10:16:09 -0700 "Jasper St. Pierre" wrote: > We need to fix that documentation, then, along with fixing Weston. > mutter will give you an error with this event stream, since attaching > a NULL buffer for an xdg_surface is invalid. This applies only to xdg_surface, and perhaps

Re: [PATCH weston] build: Require newer version of libdrm

2015-08-24 Thread Pekka Paalanen
On Fri, 21 Aug 2015 23:24:54 -0700 Bryce Harrington wrote: > On Fri, Aug 21, 2015 at 11:06:02PM -0500, Derek Foreman wrote: > > commit 89c49b3060a115e846ba1e7fbef94d14894244f2 changed the way we name > > outputs, but it also added the new output names VIRTUAL and DSI. > > > > These aren't availa

Re: [PATCH weston v2] clients: Require EGL_MIN_SWAP_INTERVAL to be 0 for subsurfaces

2015-08-24 Thread Pekka Paalanen
On Fri, 21 Aug 2015 22:32:40 +0800 Jonas Ådahl wrote: > On Fri, Aug 21, 2015 at 04:05:34PM +0300, Pekka Paalanen wrote: > > On Tue, 27 Jan 2015 10:47:29 +0800 > > Jonas Ådahl wrote: > > > > > Warn and fail when assert to create sub surfaces when swap interval 0 is > > > not supported by the EGL