Re: [PATCH] btrfs-progs: Add dependencies explicitly to fix a parallel build issue

2013-09-23 Thread Rongqing Li
"version.h" ./btrfs-calc-size.c:#include "version.h" ./btrfs-map-logical.c:#include "version.h" ./cmds-check.c:#include "version.h" ./btrfs-debug-tree.c:#include "version.h" Signed-off-by: Roy Li --- Makefile |2 +

Re: [PATCH] btrfs-progs: Add dependencies explicitly to fix a parallel build issue

2013-09-21 Thread Rongqing Li
an implicit rule .c -> .o.d, so there should be no problem for any of the files listed above. Do you means the below: .c.o: $(Q)$(check) $< @echo "[CC] $@" $(Q)$(CC) $(DEPFLAGS) $(AM_CFLAGS) $(CFLAGS) -c $< -Roy If you think the patch &q

Re: [PATCH] btrfs-progs: Add version.h dependency to fix parallel builds

2013-09-17 Thread Rongqing Li
@echo "[CC] $@" $(Q)$(CC) $(AM_CFLAGS) $(STATIC_CFLAGS) -c $< -o $@ -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.h

Re: [PATCH] btrfs-progs: Add dependencies explicitly to fix a parallel build issue

2013-09-17 Thread Rongqing Li
On 09/18/2013 09:51 AM, Eric Sandeen wrote: @echo "[CC] $@" >$(Q)$(CC) $(DEPFLAGS) $(AM_CFLAGS) $(STATIC_CFLAGS) -c $< -o $@ > I think this can be done more cleanly, I'll send a patch. -Eric Thanks, expect your patch -Roy -- Best Reagrds,