Re: [PATCH weston 0/6] ivi-shell proposal

2013-09-05 Thread Nobuhiko Tanibata
- Original Message - From: "Pekka Paalanen" To: "Nobuhiko Tanibata" Cc: ; Sent: Thursday, September 05, 2013 10:02 PM Subject: Re: [PATCH weston 0/6] ivi-shell proposal On Wed, 4 Sep 2013 09:08:26 +0900 "Nobuhiko Tanibata" wrote: Hi, This series implements ivi-shell to fulfill u

Current Input Method support

2013-09-05 Thread Weng Xuetian
Hi, I've added initial support for wayland input method in fcitx https://github.com/fcitx/fcitx/tree/wayland (still WIP, now it's only handling keyboard event without UI under wayland). While bascially it's quite straight forward to implement that, it's quite painful about some functionality cann

[PATCH weston v2] Copying xkb_info when creating a seat causes problems

2013-09-05 Thread Andrew Wedgbury
Hi Kristian, Here's a new patch for ref counting weston_xkb_info, as suggested. So a seat created with a NULL keymap will now point to the global xkb_info. --- src/compositor.h |5 ++-- src/input.c| 77 src/text-backend.c |

[PATCH 5/5] build: doesn't install wayland-scanner files if the configuration doesn't allow

2013-09-05 Thread mchalain [marc.chal...@gmail.com]
From: mchalain for cross compile the installation of wayland-scanner and other files has to be done by hand. --- Makefile.am |2 ++ 1 file changed, 2 insertions(+) diff --git a/Makefile.am b/Makefile.am index ddf39d1..a56bb48 100644 --- a/Makefile.am +++ b/Makefile.am @@ -6,7 +6,9 @@ SUBDI

[PATCH 4/5] build: use EXEEXT_FOR_BUILD instead EXEEXT for wayland-scanner

2013-09-05 Thread mchalain [marc.chal...@gmail.com]
From: mchalain AX_CC_FOR_BUILD set the EXEEXT_FOR_BUILD variable to define the application extention of build platform. --- src/Makefile.am |7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/src/Makefile.am b/src/Makefile.am index de5b23a..4b6e1dd 100644 --- a/src/Ma

[PATCH 3/5] build: use ax_cc_for_build.m4 script

2013-09-05 Thread mchalain [marc.chal...@gmail.com]
From: mchalain CC_FOR_BUILD could be defined by the ax_cc_for_build.m4 file. This file comes from : http://git.savannah.gnu.org/gitweb/?p=autoconf-archive.git;a=blob_plain;f=m4/ax_cc_for_build.m4 --- configure.ac|1 + src/Makefile.am |2 +- 2 files changed, 2 insertions(+), 1 deleti

[PATCH 2/5] build: cross compile wayland-scanner

2013-09-05 Thread mchalain [marc.chal...@gmail.com]
From: mchalain the wayland-scanner files are build with CC_FOR_BUILD instead CC like the other files. CC_FOR_BUILD could be initialize by autotool but here it's still hard coded to "gcc" --- src/Makefile.am | 13 + 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/src

[PATCH 1/5] build: break the dependecy between wayland-scanner and libwayland-util.la

2013-09-05 Thread mchalain [marc.chal...@gmail.com]
From: mchalain libwayland-util.la is used by other wayland libraries and it contains more functions useless for wayland-scanner. To build wayland-scanner without dependency with wayland libraries, the wayland-util.c is only linked to wayland-scanner. Autotool doesn't allow to compile a file twi

Re: [PATCH weston 0/6] ivi-shell proposal

2013-09-05 Thread Pekka Paalanen
On Wed, 4 Sep 2013 09:08:26 +0900 "Nobuhiko Tanibata" wrote: > Hi, > > This series implements ivi-shell to fulfill use cases of In-Vehicle > Infotainment, IVI. Such use cases are well overviewed in a project; > Genivi IVI layer management. > http://projects.genivi.org/ivi-layer-management/node/

Re: [PATCH 5/5] build: use EXEEXT_FOR_BUILD instead EXEEXT for wayland-scanner

2013-09-05 Thread Marc Chalain
Yes and no, The first one is useful if you stop the patch before the last one. But OK to suppress the first one. Regards, Marc. 2013/9/5 Stefan Schmidt > Hello. > > > On 09/05/2013 11:15 AM, marc.chal...@gmail.com wrote: > >> From: mchalain >> >> AX_CC_FOR_BUILD set the EXEEXT_FOR_BUILD variab

Re: [PATCH 5/5] build: use EXEEXT_FOR_BUILD instead EXEEXT for wayland-scanner

2013-09-05 Thread Stefan Schmidt
Hello. On 09/05/2013 11:15 AM, marc.chal...@gmail.com wrote: From: mchalain AX_CC_FOR_BUILD set the EXEEXT_FOR_BUILD variable to define the application extention of build platform. This effectively makes patch #1 useless. Any special reason for still keeping it? If not I would suggest to m

[PATCH 4/5] build: use ax_cc_for_build.m4 script

2013-09-05 Thread mchalain [marc.chal...@gmail.com]
From: mchalain CC_FOR_BUILD could be defined by the ax_cc_for_build.m4 file. This file comes from : http://git.savannah.gnu.org/gitweb/?p=autoconf-archive.git;a=blob_plain;f=m4/ax_cc_for_build.m4 --- configure.ac |1 + m4/ax_cc_for_build.m4 | 77 ++

[PATCH 5/5] build: use EXEEXT_FOR_BUILD instead EXEEXT for wayland-scanner

2013-09-05 Thread mchalain [marc.chal...@gmail.com]
From: mchalain AX_CC_FOR_BUILD set the EXEEXT_FOR_BUILD variable to define the application extention of build platform. --- src/Makefile.am |9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/src/Makefile.am b/src/Makefile.am index 2921041..4b6e1dd 100644 --- a/src/

[PATCH 3/5] build: cross compile wayland-scanner

2013-09-05 Thread mchalain [marc.chal...@gmail.com]
From: mchalain the wayland-scanner files are build with CC_FOR_BUILD instead CC like the other files. CC_FOR_BUILD could be initialize by autotool but here it's still hard coded to "gcc" --- src/Makefile.am | 13 + 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/src

[PATCH 2/5] build: break the dependecy between wayland-scanner and libwayland-util.la

2013-09-05 Thread mchalain [marc.chal...@gmail.com]
From: mchalain libwayland-util.la is used by other wayland libraries and it contains more functions useless for wayland-scanner. To build wayland-scanner without dependency with wayland libraries, the wayland-util.c is only linked to wayland-scanner. Autotool doesn't allow to compile a file twi

[PATCH 1/5] build: add EXEEXT missing on wayland-scanner

2013-09-05 Thread mchalain [marc.chal...@gmail.com]
From: mchalain Autotool manages different platforms and use $(EXEEXT) to suffix an application. --- src/Makefile.am |8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/Makefile.am b/src/Makefile.am index 4226f63..85006a3 100644 --- a/src/Makefile.am +++ b/src/Makef

[PATCH 0/5] build: wayland cross-compile support

2013-09-05 Thread mchalain [marc.chal...@gmail.com]
wayland-scanner is a source code generator and has to run on the build system. The cross compile of wayland is broken because wayland libraries and wayland-scanner has to use two different compilers. This patches break the dependencies between wayland libraries and wayland-scanner and change the

Re: [PATCH weston 0/6] ivi-shell proposal

2013-09-05 Thread Nobuhiko Tanibata
Hi, Thank you for feedcak. Please see my inline comments. Best regard, Tanibata - Original Message - From: Andreas Pokorny To: Nobuhiko Tanibata Cc: wayland-devel@lists.freedesktop.org ; securitych...@denso.co.jp Sent: Thursday, September 05, 2013 5:34 AM Subject: Re: [PATCH weston 0/