On Tue, Jan 05, 2021 at 06:17:56PM +0100, Tim Duesterhus wrote: > haproxy: $(OPTIONS_OBJS) $(OBJS) > $(cmd_LD) $(LDFLAGS) -o $@ $^ $(LDOPTS) > +ifeq ($(USE_OPENSSL),) > + $(warning Building without TLS support. Add USE_OPENSSL to add support > for TLS encrypted connections.) > +endif > +
I'm not fond of this one. It tells you after a long build that it's probably not what you want, and is particularly annoying for anyone. I'd rather do it before the build, and only if USE_OPENSSL wasn't forced either way. Again there's no rush on this, and if we figure a nice way to print options before starting the build, we could adopt it to dump several ones instead of just SSL. Willy