Re: [PATCH 2/7] Undocument deprecated alias 'push.default=tracking'
From: Junio C Hamano <hidden>
Date: 2016-06-15 22:56:01
Jonathan Nieder [off-list ref] writes:
Jonathan Nieder wrote:quoted
Is the problem that "deprecated" is not precise enough? For example, would it make sense to say "deprecated synonym for `upstream`. Will be dropped in git 2.1" or something like that?Also, if we plan to remove support soon, we should start warning when this setting is encountered so people know to update their configuration.
I do not think this even needs to be removed. We deprecate something for one of two reasons. One is when it was a bad idea to support it, and we would want to remove the support eventually. This needs a migration plan. The other is when there are better ways available but we do not want to break the old way. We still do not want to encourage the old way to new users. The change from 'upstream' from 'tracking' is the latter. The wording will confuse new users when they want to learn what 'tracking' as a concept is, and it is better spelt 'upstream'. But the breakage is not serious enough to warrant forcing an old timer who can explain these two concepts to newbies when needed to update his configuration files he is not planning to show to newbies.