<html>
  <head>
    <meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <div class="moz-cite-prefix">On 10/10/2013 16:16, Burton, Ross
      wrote:<br>
    </div>
    <blockquote
cite="mid:CAJTo0LaW6XJThFnBGydYNDBMZ-_ONt=BRgbnosNapadyDyoqFg@mail.gmail.com"
      type="cite">
      <pre wrap="">On 10 October 2013 13:23, Ewan LE BIDEAU-CANEVET
<a class="moz-txt-link-rfc2396E" href="mailto:Ewan.LEBIDEAU-CANEVET@eurogiciel.fr">&lt;Ewan.LEBIDEAU-CANEVET@eurogiciel.fr&gt;</a> wrote:
</pre>
      <blockquote type="cite">
        <pre wrap="">-SRC_URI_append_class-native = <a class="moz-txt-link-rfc2396E" href="file://just-scanner.patch">" file://just-scanner.patch"</a>
</pre>
      </blockquote>
      <pre wrap="">
That patch -  as it says - serves a useful purpose.  What is the
rationale for deleting it?  If there's a good reason, please say what
it is.

</pre>
      <blockquote type="cite">
        <pre wrap="">-# Wayland installs a M4 macro for other projects to use. This M4 macro includes
-# a path to a Makefile fragment to get the rules to generate stubs from protocol
-# description files.  The paths to the sysroot end up incorrect, so fix them.
-do_configure_append_class-native() {
-  sed -e 's,@prefix@,${STAGING_DIR_NATIVE},g' \
-      -e 's,@exec_prefix@,${STAGING_DIR_NATIVE},g' \
-      -e 's,@bindir@,${STAGING_BINDIR_NATIVE},g' \
-      -e 's,@datarootdir@,${STAGING_DATADIR_NATIVE},g' \
-  ${S}/wayland-scanner.m4.in &gt; ${B}/wayland-scanner.m4
-}
</pre>
      </blockquote>
      <pre wrap="">
Wayland is still installing this m4 file, so why delete this fix?
>From a glance at Wayland 1.2 it looks like it isn't required anymore
as pkgconfig is used to determine the paths, but you need to state
this in the commit message.
</pre>
    </blockquote>
    This file <br>
    <pre wrap="">wayland-scanner.m4<b>.</b><b>in</b><b> </b></pre>
    doesn't exist anymore. In fact there is only "wayland-scanner.m4" <br>
    Run sed command on a missing  file wayland-scanner.m4.in  is now
    useless, correct ?<br>
    Furthermore, in wayland-scanner.m4 has no reference to strings <br>
    <pre wrap="">@prefix@ 
@exec_prefix@ 
@bindir@ 
@datarootdir@ 

</pre>
    So if we let this instruction wayland-scanner.m4 will  be cleared.
    So this leads to build faillure.<br>
    <br>
    <blockquote
cite="mid:CAJTo0LaW6XJThFnBGydYNDBMZ-_ONt=BRgbnosNapadyDyoqFg@mail.gmail.com"
      type="cite">
      <pre wrap="">
Also, there's a 1.2.1 release now.
</pre>
    </blockquote>
    You're right :). I have started this work few weeks ago.<br>
    I noticed that there is a 1.3.0 version, and I have started to take
    a look on it. I'll probably post my work in 1 or 2 weeks.<br>
    <br>
    <br>
    As I saw , we need to do the same change (Remove : 
    do_configure_append_class-native function =&gt; last existing
    wayland-scanner.m4.in was in 1.1.0 version.)<br>
    <br>
    Apologies for removing your just-scanner.patch file, my mistake.<br>
    <br>
    <br>
    Ewan<br>
  </body>
</html>