Re: [PATCH v5 10/15] bugreport: add config values from safelist

From: Martin Ågren <martin.agren@gmail.com>
Date: Jan 30, 2020, 22:36 UTC
Message-ID: <CAN0heSr3WTtGh=gWTiXYEoSGGHPF_YniA9xH989=OtYfM2-Evw@mail.gmail.com>
In-Reply-To: <20200124033436.81097-11-emilyshaffer@google.com>

On Fri, 24 Jan 2020 at 04:40, <emilyshaffer@google.com> wrote:

Show 10 quoted lines

> --- a/Makefile
> +++ b/Makefile
> @@ -2465,7 +2465,7 @@ endif
>  git-%$X: %.o GIT-LDFLAGS $(GITLIBS)
>         $(QUIET_LINK)$(CC) $(ALL_CFLAGS) -o $@ $(ALL_LDFLAGS) $(filter %.o,$^) $(LIBS)
>
> -git-bugreport$X: bugreport.o GIT-LDFLAGS $(GITLIBS)
> +git-bugreport$X: bugreport-config-safelist.h bugreport.o GIT-LDFLAGS $(GITLIBS)
>         $(QUIET_LINK)$(CC) $(ALL_CFLAGS) -o $@ $(ALL_LDFLAGS) $(filter %.o,$^) \
>                 $(LIBS)

Haven't looked at this patch at all, except I've noticed that something is up with the dependencies. I think bugreport.o needs to depend on bugreport-config-safelist.h. As it is, the latter may or may not be available as bugreport.o is built. (Reproduces fairly well for me with something like `make clean && make -j16`.)

I'll try to continue tomorrow. :-)

Martin