| Message ID | patch-1.1-207f1019e70-20211021T195940Z-avarab@gmail.com (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | f7a8389fb3f6afcfcb3ae730016a37818c95b051 |
| Headers | show |
| Series | Makefile: remove redundant GIT-CFLAGS dependency from "sparse" | expand |
Hi Ævar, On Thu, 21 Oct 2021, Ævar Arnfjörð Bjarmason wrote: > The "sparse" target needed the GIT-CFLAGS dependency before my > c234e8a0ecf (Makefile: make the "sparse" target non-.PHONY, > 2021-09-23), but since then it depends on the corresponding *.o files, > which in turn depend on the correct header files, as well as on > GIT-CFLAGS. There's no need to re-state this dependency here. > > Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com> > --- > A tiny redundancy fix for an already-in-master topic. > > Makefile | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/Makefile b/Makefile > index 381bed2c1d2..12be39ac497 100644 > --- a/Makefile > +++ b/Makefile > @@ -2901,7 +2901,7 @@ check-sha1:: t/helper/test-tool$X > > SP_OBJ = $(patsubst %.o,%.sp,$(C_OBJ)) > > -$(SP_OBJ): %.sp: %.c %.o GIT-CFLAGS > +$(SP_OBJ): %.sp: %.c %.o I _guess_ you could say that the `GIT-CFLAGS` dependency is redundant by virtue of the `*.o` targets already depending on it. But then, how much does it hurt? This late in the v2.34 cycle, aren't there any issues that might be worthier of our attention and care? Ciao, Dscho > $(QUIET_SP)cgcc -no-compile $(ALL_CFLAGS) $(EXTRA_CPPFLAGS) \ > -Wsparse-error \ > $(SPARSE_FLAGS) $(SP_EXTRA_FLAGS) $< && \ > -- > 2.33.1.1494.g88b39a443e1 > >
diff --git a/Makefile b/Makefile index 381bed2c1d2..12be39ac497 100644 --- a/Makefile +++ b/Makefile @@ -2901,7 +2901,7 @@ check-sha1:: t/helper/test-tool$X SP_OBJ = $(patsubst %.o,%.sp,$(C_OBJ)) -$(SP_OBJ): %.sp: %.c %.o GIT-CFLAGS +$(SP_OBJ): %.sp: %.c %.o $(QUIET_SP)cgcc -no-compile $(ALL_CFLAGS) $(EXTRA_CPPFLAGS) \ -Wsparse-error \ $(SPARSE_FLAGS) $(SP_EXTRA_FLAGS) $< && \
The "sparse" target needed the GIT-CFLAGS dependency before my c234e8a0ecf (Makefile: make the "sparse" target non-.PHONY, 2021-09-23), but since then it depends on the corresponding *.o files, which in turn depend on the correct header files, as well as on GIT-CFLAGS. There's no need to re-state this dependency here. Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com> --- A tiny redundancy fix for an already-in-master topic. Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)