)]}'
{
  "commit": "aab845424ef4b9a3fd29406e3d722324440e4a00",
  "tree": "4f6f3b3f28ac87f40eb8953081b02a508a6a8ad6",
  "parents": [
    "b5e823594cff190bc18361207a89b08d57b038d7"
  ],
  "author": {
    "name": "Junio C Hamano",
    "email": "gitster@pobox.com",
    "time": "Sat Sep 05 19:56:20 2015 -0700"
  },
  "committer": {
    "name": "Junio C Hamano",
    "email": "gitster@pobox.com",
    "time": "Sun Sep 06 19:59:40 2015 -0700"
  },
  "message": "am: match --signoff to the original scripted version\n\nLinus noticed that the recently reimplemented \"git am -s\" defines\nthe trailer block too rigidly, resulting in an unnecessary blank\nline between the existing sign-offs and his new sign-off.  An e-mail\nsubmission sent to Linus in real life ends with mixture of sign-offs\nand commentaries, e.g.\n\n\ttitle here\n\n\tmessage here\n\n\tSigned-off-by: Original Author \u003coriginal@auth.or\u003e\n\t[rv: tweaked frotz and nitfol]\n\tSigned-off-by: Re Viewer \u003crv@ew.er\u003e\n\tSigned-off-by: Other Reviewer \u003cother@rev.ewer\u003e\n\t---\n\tpatch here\n\nBecause the reimplementation reused append_signoff() helper that is\nused by other codepaths, which is unaware that people intermix such\ncomments with their sign-offs in the trailer block, such a message\nwas judged to end with a non-trailer, resulting in an extra blank\nline before adding a new sign-off.\n\nThe original scripted version of \"git am\" used a lot looser\ndefinition, i.e. \"if and only if there is no line that begins with\nSigned-off-by:, add a blank line before adding a new sign-off\".  For\nthe upcoming release, stop using the append_signoff() in \"git am\"\nand reimplement the looser definition used by the scripted version\nto use only in \"git am\" to fix this regression in \"am\" while\navoiding new regressions to other users of append_signoff().\n\nIn the longer term, we should look into loosening append_signoff()\nso that other codepaths that add a new sign-off behave the same way\nas \"git am -s\", but that is a task for post-release.\n\nReported-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\nSigned-off-by: Junio C Hamano \u003cgitster@pobox.com\u003e\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "634f7a7aa717e6da7ff9a549cb7d5c5585e10873",
      "old_mode": 33188,
      "old_path": "builtin/am.c",
      "new_id": "e7828e5d9beb225ece1feeaefd271baec2dda840",
      "new_mode": 33188,
      "new_path": "builtin/am.c"
    },
    {
      "type": "modify",
      "old_id": "dd627c42d3f5b73a088f91dd6ab8c1ce9ddcb88c",
      "old_mode": 33261,
      "old_path": "t/t4150-am.sh",
      "new_id": "5e4855595bf31bc1de1355b856d531669e7f7ae1",
      "new_mode": 33261,
      "new_path": "t/t4150-am.sh"
    }
  ]
}
