{"thread":{"id":"60330","subject":"[PATCH 0/1] *** EDITED add.c ***","startedAt":"2023-10-09T01:19:07Z","lastAt":"2023-10-09T08:14:36Z","messageCount":5,"participants":["Naomi Ibe","Christian Couder"],"isPatch":true,"patchVersion":1,"patchTotal":1},"messages":[{"id":"482837","messageId":"20231009011842.1956-1-naomi.ibeh69@gmail.com","threadId":"60330","inReplyTo":null,"subject":"[PATCH 0/1] *** EDITED add.c ***","fromName":"Naomi Ibe","fromEmail":"naomi.ibeh69@gmail.com","sentAt":"2023-10-09T01:18:42Z","receivedAt":"2023-10-09T01:19:07Z","isPatch":true,"sender":{"key":"naomi.ibeh69@gmail.com","avatar":"https://avatars.githubusercontent.com/u/104323769?v=4"},"body":"*** BLURB HERE ***\n\nNaomi (1):\n [OUTREACHY] Fixed add.c file to conform to guidelines when using die()\n listed in issue #635\n\n builtin/add.c | 10 +++++-----\n 1 file changed, 5 insertions(+), 5 deletions(-)\n\n-- \n2.36.1.windows.1\n\n"},{"id":"482844","messageId":"CAP8UFD3HZOMJTxW5EkUwKu48GebSKX3-EPD8tjGEQnE2MGaZ7w@mail.gmail.com","threadId":"60330","inReplyTo":"20231009011842.1956-1-naomi.ibeh69@gmail.com","subject":"Re: [PATCH 0/1] *** EDITED add.c ***","fromName":"Christian Couder","fromEmail":"christian.couder@gmail.com","sentAt":"2023-10-09T07:38:52Z","receivedAt":"2023-10-09T07:39:10Z","isPatch":true,"sender":{"key":"christian.couder@gmail.com","avatar":"https://avatars.githubusercontent.com/u/208954?v=4"},"body":"On Mon, Oct 9, 2023 at 3:19 AM Naomi Ibe wrote:\n>\n> *** BLURB HERE ***\n\nA patch with number 0/X (where X is any number) is called a \"cover\nletter\" and is supposed to explain the purpose of the patch series it\nis part of. Usually a cover letter is not useful for a single patch\nlong patch series (when X is 1), as the commit message of the patch\nshould be enough.\n\nAlso things like \"*** BLURB HERE ***\" (including the stars) are\nsupposed to be replaced by a subject or some comments to explain\nthings.\n"},{"id":"482845","messageId":"CACS=G2xfUYVmvhZ_r447uArW-_6yK0r7V9BkzX7i+E7MvruA9Q@mail.gmail.com","threadId":"60330","inReplyTo":"CAP8UFD3HZOMJTxW5EkUwKu48GebSKX3-EPD8tjGEQnE2MGaZ7w@mail.gmail.com","subject":"Re: [PATCH 0/1] *** EDITED add.c ***","fromName":"Naomi Ibe","fromEmail":"naomi.ibeh69@gmail.com","sentAt":"2023-10-09T07:45:48Z","receivedAt":"2023-10-09T07:46:02Z","isPatch":true,"sender":{"key":"naomi.ibeh69@gmail.com","avatar":"https://avatars.githubusercontent.com/u/104323769?v=4"},"body":"Okay I understand it better now ,can I do anything about it? Or should\nI leave it as is?.\nIt's my first time using the \"git patch\" and \"git send-email\"\ncommands, so it was a real struggle understanding the proper way to\nget things setup.\n\nOn Mon, Oct 9, 2023 at 8:39 AM Christian Couder\n wrote:\n>\n> On Mon, Oct 9, 2023 at 3:19 AM Naomi Ibe wrote:\n> >\n> > *** BLURB HERE ***\n>\n> A patch with number 0/X (where X is any number) is called a \"cover\n> letter\" and is supposed to explain the purpose of the patch series it\n> is part of. Usually a cover letter is not useful for a single patch\n> long patch series (when X is 1), as the commit message of the patch\n> should be enough.\n>\n> Also things like \"*** BLURB HERE ***\" (including the stars) are\n> supposed to be replaced by a subject or some comments to explain\n> things.\n"},{"id":"482847","messageId":"CAP8UFD3Cj7nC704Rti8jBp6VxBJDxsBpf_8vvordW8MWXc_GDA@mail.gmail.com","threadId":"60330","inReplyTo":"CACS=G2xfUYVmvhZ_r447uArW-_6yK0r7V9BkzX7i+E7MvruA9Q@mail.gmail.com","subject":"Re: [PATCH 0/1] *** EDITED add.c ***","fromName":"Christian Couder","fromEmail":"christian.couder@gmail.com","sentAt":"2023-10-09T08:03:16Z","receivedAt":"2023-10-09T08:03:32Z","isPatch":true,"sender":{"key":"christian.couder@gmail.com","avatar":"https://avatars.githubusercontent.com/u/208954?v=4"},"body":"On Mon, Oct 9, 2023 at 9:46 AM Naomi Ibe wrote:\n>\n> Okay I understand it better now ,can I do anything about it? Or should\n> I leave it as is?.\n\nI think you should send a version 2 of the patch (so without a cover\nletter) taking into account the comments I made about it. You might\nwant to wait a bit before sending version 2 though, so that others\nmight have time to read your patch and give you more feedback about\nit.\n\n> It's my first time using the \"git patch\n\nI guess you mean `git format-patch`.\n\n> and \"git send-email\"\n> commands, so it was a real struggle understanding the proper way to\n> get things setup.\n\nYeah, it's not easy to set things up, but once that's done it should\nbe much easier. To make things even easier, you could perhaps save\nsomewhere the commands that you have previously used. And to improve\nor make changes to those commands, you could take a look at the\ndocumentation for `git format-patch` and `git send-email` which have a\nnumber of interesting options, and take note of those interesting\noptions.\n"},{"id":"482848","messageId":"CACS=G2yndFLMgZj204q90py9SCGj7bu5_jH9iP2HAWU3tdBJtQ@mail.gmail.com","threadId":"60330","inReplyTo":"CAP8UFD3Cj7nC704Rti8jBp6VxBJDxsBpf_8vvordW8MWXc_GDA@mail.gmail.com","subject":"Re: [PATCH 0/1] *** EDITED add.c ***","fromName":"Naomi Ibe","fromEmail":"naomi.ibeh69@gmail.com","sentAt":"2023-10-09T08:14:19Z","receivedAt":"2023-10-09T08:14:36Z","isPatch":true,"sender":{"key":"naomi.ibeh69@gmail.com","avatar":"https://avatars.githubusercontent.com/u/104323769?v=4"},"body":"Okay then, I'd wait for additional feedback and send in a Version 2\nlater. Thank you\n\nOn Mon, Oct 9, 2023 at 9:03 AM Christian Couder\n wrote:\n>\n> On Mon, Oct 9, 2023 at 9:46 AM Naomi Ibe wrote:\n> >\n> > Okay I understand it better now ,can I do anything about it? Or should\n> > I leave it as is?.\n>\n> I think you should send a version 2 of the patch (so without a cover\n> letter) taking into account the comments I made about it. You might\n> want to wait a bit before sending version 2 though, so that others\n> might have time to read your patch and give you more feedback about\n> it.\n>\n> > It's my first time using the \"git patch\n>\n> I guess you mean `git format-patch`.\n>\n> > and \"git send-email\"\n> > commands, so it was a real struggle understanding the proper way to\n> > get things setup.\n>\n> Yeah, it's not easy to set things up, but once that's done it should\n> be much easier. To make things even easier, you could perhaps save\n> somewhere the commands that you have previously used. And to improve\n> or make changes to those commands, you could take a look at the\n> documentation for `git format-patch` and `git send-email` which have a\n> number of interesting options, and take note of those interesting\n> options.\n"}]}