git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [RFC 7/9] send-email: Completely replace --signed-off-cc with --signed-off-by-cc

From
Jeff King <peff@peff.net>
Date
Sep 29, 2008, 16:29 UTC
Message-ID
<20080929162935.GA2628@coredump.intra.peff.net>
In-Reply-To
<1222664781-55763-2-git-send-email-mfwitten@mit.edu>
On Mon, Sep 29, 2008 at 12:06:19AM -0500, Michael Witten wrote:
> This breaks backwards compatibility, but so what---get off your
> lazy arses and remove the cruft.

Sorry, but try to be a lot more careful than that about breaking compatibility. The right sequence is:

  1. Introduce new option (which has already been done for the command
     line, but it looks like you are adding a new config variable).
     Mention the new option in the documentation as the "right way".
     Possibly mention the old version as "deprecated".
  2. Wait a long time, possibly forever. This gives people a chance to
     adjust, and it lets us wait for a major version bump where such an
     incompatibility might be more expected.
  3. Remove the old option.

So I think you are skipping straight to '3' here. However, I think your real purpose is to simply clean up the documentation, so why not just do that? We can still support the old options for historical setups, and just advertise the new ones in the docs.

-Peff
Previous: Michael WittenNext: Michael Witten
Message 30 of 45 in “Docs: send-email's usage text and man page mention same options”
  1. 1/8 Docs: send-email's usage text and man page mention same optionsMichael Witten, Sep 28, 2008
  2. 2/8 Docs: send-email usage text much sexierMichael Witten, Sep 28, 2008
  3. 3/8 Docs: send-email: Man page option orderingMichael Witten, Sep 28, 2008
  4. 4/8 send-email: change --no-validate to boolean --[no-]validateMichael Witten, Sep 28, 2008
  5. 5/8 Docs: send-email: --chain_reply_to -> --[no-]chain-reply-toMichael Witten, Sep 28, 2008
  6. 6/8 Docs: Arranged config options in man pageMichael Witten, Sep 28, 2008
  7. 7/8 Docs: send-email: Added all config variables to man endMichael Witten, Sep 28, 2008
  8. 8/8 Docs: config: send-email config options includedMichael Witten, Sep 28, 2008
  9. Jeff KingSep 28, 2008
  10. Michael WittenSep 28, 2008
  11. Jeff KingSep 28, 2008
  12. Jeff KingSep 28, 2008
  13. Michael WittenSep 28, 2008
  14. Jeff KingSep 28, 2008
  15. Jeff KingSep 28, 2008
  16. Michael WittenSep 28, 2008
  17. bash completion: Add --[no-]-validate to "git send-email"Teemu Likonen, Sep 28, 2008
  18. Teemu LikonenSep 28, 2008
  19. Jakub NarebskiSep 28, 2008
  20. Michael WittenSep 28, 2008
  21. Jakub NarebskiSep 29, 2008
  22. Jeff KingSep 29, 2008
  23. 6/9 Docs: send-email: Remove unnecessary config variable descriptionMichael Witten, Sep 29, 2008
  24. 7/9 send-email: Completely replace --signed-off-cc with --signed-off-by-ccMichael Witten, Sep 29, 2008
  25. 8/9 Docs: send-email: Create logical groupings for --help textMichael Witten, Sep 29, 2008
  26. 9/9 Docs: send-email: Create logical groupings for man textMichael Witten, Sep 29, 2008
  27. Jeff KingSep 29, 2008
  28. Miklos VajnaSep 28, 2008
  29. Michael WittenSep 28, 2008
  30. Jeff KingSep 29, 2008
  31. 8/9 Docs: send-email: Create logical groupings for man textMichael Witten, Sep 29, 2008
  32. 9/9 send-email: signedoffcc -> signedoffbycc, but handle bothMichael Witten, Sep 29, 2008
  33. Jeff KingSep 29, 2008
  34. 1/9 Docs: send-email's usage text and man page mention same optionsMichael Witten, Sep 30, 2008
  35. 2/9 Docs: send-email usage text much sexierMichael Witten, Sep 30, 2008
  36. 3/9 Docs: send-email: Man page option orderingMichael Witten, Sep 30, 2008
  37. 4/9 send-email: change --no-validate to boolean --[no-]validateMichael Witten, Sep 30, 2008
  38. 5/9 Docs: send-email: --chain_reply_to -> --[no-]chain-reply-toMichael Witten, Sep 30, 2008
  39. 6/9 Docs: send-email: Remove unnecessary config variable descriptionMichael Witten, Sep 30, 2008
  40. 7/9 Docs: send-email: Create logical groupings for --help textMichael Witten, Sep 30, 2008
  41. 8/9 Docs: send-email: Create logical groupings for man textMichael Witten, Sep 30, 2008
  42. 9/9 send-email: signedoffcc -> signedoffbycc, but handle bothMichael Witten, Sep 30, 2008
  43. Jeff KingOct 1, 2008
  44. Michael WittenOct 1, 2008
  45. Shawn O. PearceOct 1, 2008

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.