[PATCH][weston] configure.ac: Add --with-wayland-scanner-path
ppaalanen at gmail.com
Tue Jul 18 13:58:21 UTC 2017
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>
> > 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.
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
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.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 833 bytes
Desc: OpenPGP digital signature
More information about the wayland-devel