Re: [PATCH] Update hard-coded header dependencies
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Aug 10, 2014, 23:39 UTC
- Message-ID
- <xmqqy4uv7w4j.fsf@gitster.dls.corp.google.com>
- In-Reply-To
- <xmqq38d39b2d.fsf@gitster.dls.corp.google.com>
Junio C Hamano <gitster@pobox.com> writes:
Show 14 quoted lines
> Jonathan Nieder <jrnieder@gmail.com> writes: > >> ? People using ancient compilers that never change headers wouldn't >> be hurt, people using modern compilers that do change headers also >> wouldn't be hurt, and we could stop pretending to maintain an >> up-to-date list. > > I agree that it is very tempting to declare that we do not manually > "maintain" the dependency list and force people without -MMD to > recompile whenever any unrelated header changes. Especially that > this patch only works on the 'master' branch and upwards, and does > not even work on 'maint', let alone 1.9 or 1.8.5 maintenance tracks. > > Let's consider the merit of that approach after 2.1 is out. Thanks.
Actually "upwards" is not even true; the 'next' branch already wants e.g. trace.h to build credential-store.o, which is not needed for the 'master' branch.