From: Pranit Bauva Date: Tue, 23 Aug 2016 21:07:34 GMT Subject: Re: [PATCH v14 01/27] bisect--helper: use OPT_CMDMODE instead of OPT_BOOL Message-ID: In-Reply-To: Hey Junio, On Wed, Aug 24, 2016 at 1:58 AM, Junio C Hamano wrote: > Pranit Bauva writes: > >> `--next-all` is meant to be used as a subcommand to support multiple >> "operation mode" though the current implementation does not contain any >> other subcommand along side with `--next-all` but further commits will >> include some more subcommands. >> >> Helped-by: Johannes Schindelin >> Mentored-by: Lars Schneider >> Mentored-by: Christian Couder >> Signed-off-by: Pranit Bauva >> --- > > A series this size really needs a cover letter that describes what > changes were made relative to the previous round to help reviewers. I am actually using SubmitGit these days as I am back in my campus and IMAP/SMTP connections are blocked here. SubmitGit doesn't have the option to include a cover letter. Though I can describe the changes by including a separate email as a reply to 1st patch. I will send the changes according to v12 since v13 wasn't commented on. > I also noticed something curious. All of your messages are dated > > Date: Tue, 23 Aug 2016 11:53:53 +0000 > > and I think that is the same as all the previous series, but how are > you sending your patches? The reason why I ask is because this does > not allow readers to tell their mail reader to sort messages by the > sender's timestamp in order to see the messages in order (as a > matter of fact, git-send-email knows that readers are helped by this > feature, and backdates the first message of 10 patch series by 10 > seconds, stamps the second message as later than the first by one > second, etc. to help them). This might be a bug in SubmitGit. I didn't notice this previously. Regards, Pranit Bauva