[Buildroot] [PATCH] graph-depends: fix when $(O) is a relative path
Yann E. MORIN
yann.morin.1998 at free.fr
Mon Jun 23 17:03:50 UTC 2014
Arnout, All,
On 2014-06-23 07:28 +0200, Arnout Vandecappelle spake thusly:
> On 20/06/14 22:55, Yann E. MORIN wrote:
> > On 2014-06-19 00:10 +0200, Arnout Vandecappelle (Essensium/Mind) spake thusly:
> >> The graph-depends commands cd into the CONFIG_DIR and run the script
> >> from there. However, this means that when $(O) is a relative path, it
> >> will no longer be correct. Therefore, use $(BASE_DIR) instead of $(O).
> >>
> >> Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout at mind.be>
> > I tried with thisscript, and did not notice any difference between the
> > two runs, one with master, the second with the patch applied:
[--SNIP long script--]
> First of all, for me all these cases succeed both with and without the patch.
You mean all the 6 cases do work for you? Weird.
I don't see how it can work, since calling this
make -C buildroot O=somewhere goal
will not have the --no-print-directory option, and thus make would print
its usual "Entering/Leaving directory blabla" which confuses
graph-depends.py
There is not much we can do about that, since our out-of-tree Makefile
wrapper is never called.
Even worse, if you do
make -C buildroot O=somewhere defconfig
make -C somewhere graph-depends
then even though we do use our Makefile wrapper in this case, the
--no-print-directory option is not used! So make would still print its
"Entering/Leaving directory blabla" (and thus graph-depends will get
confused.)
> Using any other combination of . and .. a $(O)
> triggers the problem. So try
>
> do_prep br-7
> make -C br-7 O=./br.build-7 defconfig
> make -C br-7 O=./br.build-7 graph-depends
OK, that's a use-case I was missing.
> The graph-depends will fail with:
>
> tee: br.build-7/graphs/graph-depends.dot: No such file or directory
> Getting targets
> Getting dependencies for ['host-binutils', 'host-gmp', 'host-mpc', 'host-mpfr',
> 'uclibc', 'host-fakeroot', 'host-gcc-final', 'host-makedevs',
> 'toolchain-buildroot', 'toolchain', 'busybox', 'rootfs-tar']
> Getting dependencies for ['host-m4', 'linux-headers', 'host-gcc-initial']
> br.build-7/graphs/graph-depends.pdf: No such file or directory
> Makefile:677: recipe for target 'graph-depends' failed
> make: *** [graph-depends] Error 1
OK, it fails here, too, but your patch does not change anything: it
fails without your patch, and it fails with your patch applied.
I'm using make-3.81, if that shall be a differentiator.
Regards,
Yann E. MORIN.
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
More information about the buildroot
mailing list