在 2012-10-08一的 20:13 -0600,Eric Blake写道: > On 09/28/2012 06:25 AM, Eric Blake wrote: > > On 09/27/2012 09:51 PM, liguang wrote: > >> if gnulib submodule happened to be dirty, build > >> process will fall into '_autogen' target trap > >> in cfg.mk recursively, so break this dead-loop. > > > > Yes, I've run into this before; thanks for trying to tackle it. > > > >> > >> Signed-off-by: liguang <lig.fnst@xxxxxxxxxxxxxx> > >> --- > >> cfg.mk | 2 +- > >> 1 files changed, 1 insertions(+), 1 deletions(-) > >> > >> diff --git a/cfg.mk b/cfg.mk > >> index bbfd4a2..92966d5 100644 > >> --- a/cfg.mk > >> +++ b/cfg.mk > >> @@ -699,7 +699,7 @@ ifeq (0,$(MAKELEVEL)) > >> test "$$stamp" = "$$actual"; echo $$?) > >> _clean_requested = $(filter %clean,$(MAKECMDGOALS)) > >> ifeq (1,$(_update_required)$(_clean_requested)) > >> - $(info INFO: gnulib update required; running ./autogen.sh first) > >> + $(error ERR: gnulib update required; running ./autogen.sh first) > > > > However, this is not the right fix - it will error out even when .gnulib > > is not dirty, but just out of date. I'll spend some time on this > > getting it right today. > > Thanks again for forcing me to fix this; if you hadn't noticed, it was > fixed here: > > https://www.redhat.com/archives/libvir-list/2012-October/msg00018.html > Good! pretty complex fix-up. -- liguang lig.fnst@xxxxxxxxxxxxxx FNST linux kernel team -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list