[PATCH][weston] configure.ac: Add --with-wayland-scanner-path

[Pekka Paalanen]

On Wed, 24 May 2017 11:32:21 +0300
Jussi Kukkonen <jussi.kukkonen at intel.com> wrote:

> On 23 May 2017 at 14:51, Quentin Glidic <sardemff7+wayland at sardemff7.net>
> wrote:

> > So you want to break the simple (cross-compiling) cases to fix
> > something that doesn’t need fixing?
> 
> I think that is not warranted.
> * My issue was that every wayland-scanner user was doing something
> different, often in a way that required patching in Yocto
> * Weston was fine for us but Pekka gave me feedback that lead me to
> believe the weston method was not the best solution either
> 
> I did my best to try and resolve both issues. It looks like my
> attempt was not successful but there should be no reason to say I
> "want to break" something.

Hi,

I'm with Jussi here.

> > If you want to make a patch, it should not introduce a regression.  
> 
> I think _if_ the goal is to use the same scanner lookup in e.g. mesa
> and weston, some use case in one or the other is just bound to have a
> regression.

Quentin, do you see any way unifying the usage of wayland-scanner
without regressing at least one user?

> > So at this point, we should make a generic patch, ideally (IMO) in
> > wayland-scanner.m4, and have people test it in all their different
> > setup to see if it works as expected.

Is anyone actually using wayland-scanner.m4?

I did not find any evidence of Weston using it. Certainly no-one is
using wayland-scanner.mk anymore because of the structure or
wayland-protocols and the need to search for XML files in both local
and system paths. Since wayland-scanner.m4 has a reference to
wayland-scanner.mk, I doubt anyone is using wayland-scanner.m4 either.

We should make a plan to get rid of wayland-scanner.mk and
wayland-scanner.m4, I do not think they are fixable.

> > If you’re not comfortable making one, I’ll do it by this week-end
> > so you can test it in yours.

Did anything come out of that?

There is also the sysroot patch from Tomek Bury that no-one has
commented on yet.
https://patchwork.freedesktop.org/patch/165478/


Thanks,
pq
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <https://lists.freedesktop.org/archives/wayland-devel/attachments/20170718/7de1d1ef/attachment.sig>