{"thread":{"id":"37971","subject":"Re: [PATCH] difftool: honor --trust-exit-code for builtin tools","startedAt":"2014-11-15T00:27:11Z","lastAt":"2014-11-15T14:22:31Z","messageCount":2,"participants":["David Aguilar","Adri Farr"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"251926","messageId":"20141115002710.GA72216@gmail.com","threadId":"37971","inReplyTo":null,"subject":"Re: [PATCH] difftool: honor --trust-exit-code for builtin tools","fromName":"David Aguilar","fromEmail":"davvid@gmail.com","sentAt":"2014-11-15T00:27:11Z","receivedAt":"2014-11-15T00:27:11Z","isPatch":true,"sender":{"key":"davvid@gmail.com","avatar":"https://avatars.githubusercontent.com/u/13196?v=4"},"body":"\nAdri sent me this directly but I think it should have gone to the list.\n\nAdri, if you don't mind, Junio can add:\n\nTested-by: Adri Farr <14farresa@gmail.com>\n\n...to the commit message trailer since it looks like it's happy.\n\nThanks for testing!\n\ncheers,\nDavid\n\n----- Forwarded message from Adri Farr <14farresa@gmail.com> -----\n\nDate: Sat, 15 Nov 2014 00:10:12 +0100\nFrom: Adri Farr <14farresa@gmail.com>\nTo: David Aguilar <davvid@gmail.com>\nSubject: Re: [PATCH] difftool: honor --trust-exit-code for builtin tools\n\nI have tested this patch both in vim and meld and it works\nwonderfully. Thank you for the time put into this. I should have\nprovided feedback back when the patch was proposed. I guess it's never\ntoo late :).\n\n2014-11-14 22:57 GMT+01:00 David Aguilar <davvid@gmail.com>:\n> [snip]\n\n----- End forwarded message -----\n"},{"id":"251944","messageId":"CAPfWmhiPgka9ok30eXswcy1jb+g4pcNV80QM-YVvn5r-KebAUQ@mail.gmail.com","threadId":"37971","inReplyTo":"20141115002710.GA72216@gmail.com","subject":"Re: [PATCH] difftool: honor --trust-exit-code for builtin tools","fromName":"Adri Farr","fromEmail":"14farresa@gmail.com","sentAt":"2014-11-15T14:22:31Z","receivedAt":"2014-11-15T14:22:31Z","isPatch":true,"sender":{"key":"14farresa@gmail.com","avatar":null},"body":"Oh, sorry about that. I didn't realize I was directly responding to\nyou. Apologizes. Hopefully I'm doing it right this time. I don't have\nmuch experience with mailing lists, and Gmail doesn't seem to help.\nYou don't need to add the 'Tested-by' line. Testing is the least I can\ndo. If you still want to add that line, my full name is 'Adria\nFarres'. Thank you!\n\n2014-11-15 1:27 GMT+01:00 David Aguilar <davvid@gmail.com>:\n>\n> Adri sent me this directly but I think it should have gone to the list.\n>\n> Adri, if you don't mind, Junio can add:\n>\n> Tested-by: Adri Farr <14farresa@gmail.com>\n>\n> ...to the commit message trailer since it looks like it's happy.\n>\n> Thanks for testing!\n>\n> cheers,\n> David\n>\n> ----- Forwarded message from Adri Farr <14farresa@gmail.com> -----\n>\n> Date: Sat, 15 Nov 2014 00:10:12 +0100\n> From: Adri Farr <14farresa@gmail.com>\n> To: David Aguilar <davvid@gmail.com>\n> Subject: Re: [PATCH] difftool: honor --trust-exit-code for builtin tools\n>\n> I have tested this patch both in vim and meld and it works\n> wonderfully. Thank you for the time put into this. I should have\n> provided feedback back when the patch was proposed. I guess it's never\n> too late :).\n>\n> 2014-11-14 22:57 GMT+01:00 David Aguilar <davvid@gmail.com>:\n>> [snip]\n>\n> ----- End forwarded message -----\n"}]}