<div dir="ltr"><div dir="ltr">On Sun, Apr 19, 2020 at 8:16 PM Sebastian Huber <<a href="mailto:sebastian.huber@embedded-brains.de" target="_blank">sebastian.huber@embedded-brains.de</a>> wrote:<br></div><div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
  
    
  
  <div>
    <p>On 19/04/2020 15:38, Gedare Bloom wrote:<br>
    </p>
    <blockquote type="cite">
      <pre>On Sun, Apr 19, 2020 at 3:52 AM Vijay Kumar Banerjee <a href="mailto:vijay@rtems.org" target="_blank"><vijay@rtems.org></a> wrote:
</pre>
      <blockquote type="cite" style="color:rgb(0,0,0)">
        <pre>Hi,

While trying to build the examples with a fresh pull from upstream,
I got the following build error from gccdeps:

```
Build failed
Traceback (most recent call last):
  File "<i><span>/</span>home/lunatic/development/rtems-examples<span>/</span></i>.waf-2.0.19-1f3c580272b15a03d2566843c5fe872a/waflib/Task.py", line 190, in process
    self.post_run()
  File "/home/lunatic/development/rtems-examples/rtems_waf/gccdeps.py", line 144, in post_run
    raise ValueError('could not find %r for %r' % (x, self))
ValueError: could not find ['filesystem', 'fat_ramdisk', 'fs-root-tar.h'] for
</pre>
      </blockquote>
      <pre>It's doing an error check to find the build dependency fs-root-tar.h,
which gets generated in the build directory by bin2c. This has always
been tricky to handle in waf. I don't grok the way Chris reworked the
rootfs support in the example so I don't immediately see the root
(hah) problem or how to try to fix it.</pre></blockquote></div></blockquote><div><br></div><div>Hi,</div><div><br></div><div>I figured out that the tar is generating properly but the error check is failing</div><div>because it's looking for the fs-root-tar.h file in the source file directory but the</div><div>header file is being generated in the build directory.</div><div><br></div><div>Just manually copying the header file to the source directory makes it happy</div><div>and the build works fine. Do we want to generate the header file in the source</div><div>directory as well and maybe add it in gitignore?</div><div><br></div><div>The other approach can be to tell gccdeps to check for the files in the build</div><div>directory instead. I did it with a workaround like the following, and it works fine:</div><div>```</div><div>diff --git a/gccdeps.py b/gccdeps.py<br>index bfabe72..61d9bfd 100644<br>--- a/gccdeps.py<br>+++ b/gccdeps.py<br>@@ -141,7 +141,10 @@ def post_run(self):<br>                        node = path_to_node(path, x, cached_nodes)<br> <br>                if not node:<br>-                       raise ValueError('could not find %r for %r' % (x, self))<br>+                       x = os.path.relpath(str(bld.bldnode)).split('/') + x<br>+                       node = path_to_node(path, x, cached_nodes)<br>+                       if not node:<br>+                           raise ValueError('could not find %r for %r' % (x, self))<br>                if id(node) == id(self.inputs[0]):<br>                        # ignore the source file, it is already in the dependencies<br>                        # this way, successful config tests may be retrieved from the cache<br></div><div>```</div><div><br></div><div>If it doesn't look too hackish, then I'll post a patch. This shouldn't break anything else</div><div>and I'll surely add comments there about the use case of taking this approach.</div><div><br></div><div>Does it look right?</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><blockquote type="cite">
    </blockquote>
    <p>I struggled with this stuff also in the new build system. waf is
      pretty nice, once you figured out how something works, but at
      least for me it was always very hard to figure out solutions. I
      can write Makefiles a hundred times faster. Maybe this helps:</p>
    <p><a href="https://git.rtems.org/sebh/rtems.git/tree/wscript?h=build#n410" target="_blank">https://git.rtems.org/sebh/rtems.git/tree/wscript?h=build#n410</a></p>
    <p><a href="https://git.rtems.org/sebh/rtems.git/tree/spec/build/testsuites/libtests/RTEMS-BUILD-TEST-LIB-TAR01.yml?h=build" target="_blank">https://git.rtems.org/sebh/rtems.git/tree/spec/build/testsuites/libtests/RTEMS-BUILD-TEST-LIB-TAR01.yml?h=build</a> </p></div></blockquote></div></div></div>