Jeff King <peff@xxxxxxxx> writes: > On Tue, May 22, 2012 at 02:00:05PM -0700, Junio C Hamano wrote: > >> It however seems to me that git-credential-osxkeychain.o does honor >> $(CFLAGS), either from the user "make CFLAGS=..." or the default the >> Makefile in question supplies. >> >> The line you quoted is not using $(CC) as the compiler, but is using it to >> link the final build product. It may not hurt to have $(CFLAGS) on that >> line, but shouldn't the line also have $(LDFLAGS) on it? > > I think so. Why don't we do this? I am OK with that. CFLAGS/LDFLAGS that come from ../../../config.mak _might_ not be suitable for this binary, but I do not think of a concrete example how that could be. Besides, this is just an example in contrib/ so I'd not worry about it too much. Thanks. > -- >8 -- > Subject: osxkeychain: pull make config from top-level directory > > The default compiler and cflags were mostly "works for me" > when I built the original version. We need to be much less > careful here than usual, because we know we are building > only on OS X. But it's only polite to at least respect the > CFLAGS and CC definitions that the user may have provided > earlier. > > While we're at it, let's update our definitions and rules to > be more like the top-level Makefile; default our CFLAGS to > include -O2, and make sure we use CFLAGS and LDFLAGS when > linking. > > Signed-off-by: Jeff King <peff@xxxxxxxx> > --- > contrib/credential/osxkeychain/Makefile | 7 +++++-- > 1 file changed, 5 insertions(+), 2 deletions(-) > > diff --git a/contrib/credential/osxkeychain/Makefile b/contrib/credential/osxkeychain/Makefile > index 75c07f8..4b3a08a 100644 > --- a/contrib/credential/osxkeychain/Makefile > +++ b/contrib/credential/osxkeychain/Makefile > @@ -2,10 +2,13 @@ all:: git-credential-osxkeychain > > CC = gcc > RM = rm -f > -CFLAGS = -g -Wall > +CFLAGS = -g -O2 -Wall > + > +-include ../../../config.mak.autogen > +-include ../../../config.mak > > git-credential-osxkeychain: git-credential-osxkeychain.o > - $(CC) -o $@ $< -Wl,-framework -Wl,Security > + $(CC) $(CFLAGS) -o $@ $< $(LDFLAGS) -Wl,-framework -Wl,Security > > git-credential-osxkeychain.o: git-credential-osxkeychain.c > $(CC) -c $(CFLAGS) $< -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html