Re: [PATCH v2 1/6] unfs3: Add a NFSv3 user mode server for use with runqemu

[Richard Purdie <richard.purdie@linuxfoundation.org>]

Hi Jason,

We're seeing a parallel make race on the autobuilder and it seems its
from this patch:

On Thu, 2014-01-23 at 08:32 -0600, Jason Wessel wrote:
> diff --git a/meta/recipes-devtools/unfs3/unfs3/unfs3_parallel_build.patch b/meta/recipes-devtools/unfs3/unfs3/unfs3_parallel_build.patch
> new file mode 100644
> index 0000000..c7fe3d7
> --- /dev/null
> +++ b/meta/recipes-devtools/unfs3/unfs3/unfs3_parallel_build.patch
> @@ -0,0 +1,36 @@
> +Fix parallel build dependency issue
> +
> +If building with make -j2 the lib.a will not get built in time.
> +
> +Jason Wessel <jason.wessel@windriver.com>
> +
> +Upstream-Status: Pending
> +
> +---
> + Config/Makefile.in |    2 ++
> + Makefile.in        |    3 ++-
> + 2 files changed, 4 insertions(+), 1 deletion(-)
> +
> +--- a/Makefile.in
> ++++ b/Makefile.in
> +@@ -29,7 +29,8 @@ DESTDIR =
> + 
> + VPATH = $(srcdir)
> + 
> +-all: subdirs unfsd$(EXEEXT)
> ++all: subdirs
> ++	$(MAKE) unfsd$(EXEEXT)
> + 
> + unfsd$(EXEEXT): $(OBJS) $(CONFOBJ) $(EXTRAOBJ)
> + 	$(CC) -o $@ $(OBJS) $(CONFOBJ) $(EXTRAOBJ) $(LDFLAGS)
> +--- a/Config/Makefile.in
> ++++ b/Config/Makefile.in
> +@@ -16,6 +16,8 @@ lib.a: $(OBJS)
> + 	$(AR) crs lib.a $(OBJS)
> + 
> + y.tab.h y.tab.c: $(srcdir)/exports.y
> ++
> ++y.tab.c: $(srcdir)/exports.y
> + 	$(YACC) -d $(srcdir)/exports.y
> + 
> + y.tab.o: y.tab.c $(srcdir)/exports.h $(top_srcdir)/nfs.h $(top_srcdir)/mount.h $(top_srcdir)/daemon.h

If you "cd Config; make lex.yy.o" you'll find it errors with y.tab.h not
being found. The reason is the line added above, the y.tab.h dependency
on running YACC is needed.

We're seeing this about once an autobuilder run, e.g.:

http://autobuilder.yoctoproject.org/main/builders/nightly-oecore/builds/7/steps/Building%20Toolchain%20Images_1/logs/stdio

Can we drop this second part of the change?

Cheers,

Richard