Re: [PATCH 0/5] jk/send-email-sender-prompt loose ends
- From
Felipe Contreras <felipe.contreras@gmail.com>
- Date
- Nov 28, 2012, 21:37 UTC
- Message-ID
- <CAMP44s1uVK-m_jczhVui0Z_Tcv-vtSVU8E1uVKY7QdQvQhMQFg@mail.gmail.com>
- In-Reply-To
- <20121128182534.GA21020@sigill.intra.peff.net>
On Wed, Nov 28, 2012 at 7:25 PM, Jeff King <peff@peff.net> wrote:
Show 39 quoted lines
> Here are the cleanups and refactorings split out from my > jk/send-email-sender-prompt series. They can go right on master and are > independent of Felipe's fc/send-email-no-sender-prompt topic. > > [1/5]: test-lib: allow negation of prerequisites > > Same as before. I think this is a useful feature for the test suite (and > 2/5 depends on it). > > [2/5]: t7502: factor out autoident prerequisite > > Same as before. Patch 5/5 depends on it. > > [3/5]: ident: make user_ident_explicitly_given static > > Same as before. Cleanup. > > [4/5]: ident: keep separate "explicit" flags for author and committer > > Same as before. Cleanup. I do not know if anybody will ever care about > the corner cases it fixes, so it is really just being defensive for > future code. > > [5/5]: t: add tests for "git var" > > Tests split out from he "git var can take multiple values" patch. > > Dropped were: > > - "git var" can take multiple values. Nobody really cares about doing > so, and it's an external interface, so we'd have to support it > forever. > > - exporting "explicit ident" flag via "git var"; same reasoning as > above > > - Git.pm supporting explicit ident; ditto > > - send-email prompting change; obsoleted by Felipe's patch
For what it's worth; they look good to me. However, I think it's worth mentioning that there are no functional changes.
-- Felipe Contreras