{"thread":{"id":"60328","subject":"[PATCH 1/1] [OUTREACHY] Fixed add.c file to conform to guidelines when using die() listed in issue #635","startedAt":"2023-10-09T01:16:18Z","lastAt":"2023-10-11T07:13:58Z","messageCount":7,"participants":["Naomi Ibe","Junio C Hamano","Christian Couder","Dragan Simic"],"isPatch":true,"patchVersion":1,"patchTotal":1},"messages":[{"id":"482835","messageId":"20231009011546.509-1-naomi.ibeh69@gmail.com","threadId":"60328","inReplyTo":null,"subject":"[PATCH 1/1] [OUTREACHY] Fixed add.c file to conform to guidelines when using die() listed in issue #635","fromName":"Naomi Ibe","fromEmail":"naomi.ibeh69@gmail.com","sentAt":"2023-10-09T01:15:45Z","receivedAt":"2023-10-09T01:16:18Z","isPatch":true,"sender":{"key":"naomi.ibeh69@gmail.com","avatar":"https://avatars.githubusercontent.com/u/104323769?v=4"},"body":"From: Naomi \n\nSigned-off-by: Naomi Ibe \n---\n builtin/add.c | 10 +++++-----\n 1 file changed, 5 insertions(+), 5 deletions(-)\n\ndiff --git a/builtin/add.c b/builtin/add.c\nindex c27254a5cd..5126d2ede3 100644\n--- a/builtin/add.c\n+++ b/builtin/add.c\n@@ -182,7 +182,7 @@ static int edit_patch(int argc, const char **argv, const char *prefix)\n \tgit_config(git_diff_basic_config, NULL); /* no \"diff\" UI options */\n \n \tif (repo_read_index(the_repository) < 0)\n-\t\tdie(_(\"Could not read the index\"));\n+\t\tdie(_(\"could not read the index\"));\n \n \trepo_init_revisions(the_repository, &rev, prefix);\n \trev.diffopt.context = 7;\n@@ -200,15 +200,15 @@ static int edit_patch(int argc, const char **argv, const char *prefix)\n \t\tdie(_(\"editing patch failed\"));\n \n \tif (stat(file, &st))\n-\t\tdie_errno(_(\"Could not stat '%s'\"), file);\n+\t\tdie_errno(_(\"could not stat '%s'\"), file);\n \tif (!st.st_size)\n-\t\tdie(_(\"Empty patch. Aborted.\"));\n+\t\tdie(_(\"empty patch. aborted\"));\n \n \tchild.git_cmd = 1;\n \tstrvec_pushl(&child.args, \"apply\", \"--recount\", \"--cached\", file,\n \t\t NULL);\n \tif (run_command(&child))\n-\t\tdie(_(\"Could not apply '%s'\"), file);\n+\t\tdie(_(\"could not apply '%s'\"), file);\n \n \tunlink(file);\n \tfree(file);\n@@ -568,7 +568,7 @@ int cmd_add(int argc, const char **argv, const char *prefix)\n finish:\n \tif (write_locked_index(&the_index, &lock_file,\n \t\t\t COMMIT_LOCK | SKIP_IF_UNCHANGED))\n-\t\tdie(_(\"Unable to write new index file\"));\n+\t\tdie(_(\"unable to write new index file\"));\n \n \tdir_clear(&dir);\n \tclear_pathspec(&pathspec);\n-- \n2.36.1.windows.1\n\n"},{"id":"482893","messageId":"xmqqlecbzl5e.fsf@gitster.g","threadId":"60328","inReplyTo":"20231009011546.509-1-naomi.ibeh69@gmail.com","subject":"Re: [PATCH 1/1] [OUTREACHY] Fixed add.c file to conform to guidelines when using die() listed in issue #635","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2023-10-09T18:49:17Z","receivedAt":"2023-10-09T18:49:23Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Naomi Ibe writes:\n\n> Subject: Re: [PATCH 1/1] [OUTREACHY] Fixed add.c file to conform to guidelines when using d\n\nSubject: [OUTREACHY][PATCH 1/1] builtin/add.c: clean up die() messages\n\n> From: Naomi \n\nThe name and address on this line come from your commit object,\nwhich in turn would have came from your configuration. You have\n\n [user]\n\tname = Naomi\n\temail = naomi.ibeh69@gmail.com\n\nsomewhere in your configuration file, perhaps in $HOME/.gitconfig or\nsomewhere. When contributiong to this project, you want that \"name\"\nline to also include your family name, as it should match what you\nwrite on your Signed-off-by: line. A focused way to do so without\naffecting your author identity for other projects is to add\n\n [user]\n\tname = Naomi Ibe\n\nin .git/config of the repository that you use to contribute to this\nproject, e.g.,\n\n $ cd ... to the working tree of your clone of git you work in ...\n $ git config user.name \"Naomi Ibe\"\n\nThe space above your Sign off is to fill the details you omitted on\nthe title of the message (which would say something about \"conform\nto guidelines\" or \"clean up\" without mentioning what rule the\noriginal violates), making the whole thing something like:\n\n builtin/add.c: clean up die() messages\n\n As described in the CodingGuidelines document, a single line\n message given to die() and its friends should not capitalize its\n first word, and should not add full-stop at the end.\n\n Signed-off-by: ...\n\n> Signed-off-by: Naomi Ibe \n> ---\n> builtin/add.c | 10 +++++-----\n> 1 file changed, 5 insertions(+), 5 deletions(-)\n\nThanks. Otherwise the patch looks good.\n\n>\n> diff --git a/builtin/add.c b/builtin/add.c\n> index c27254a5cd..5126d2ede3 100644\n> --- a/builtin/add.c\n> +++ b/builtin/add.c\n> @@ -182,7 +182,7 @@ static int edit_patch(int argc, const char **argv, const char *prefix)\n> \tgit_config(git_diff_basic_config, NULL); /* no \"diff\" UI options */\n> \n> \tif (repo_read_index(the_repository) < 0)\n> -\t\tdie(_(\"Could not read the index\"));\n> +\t\tdie(_(\"could not read the index\"));\n> \n> \trepo_init_revisions(the_repository, &rev, prefix);\n> \trev.diffopt.context = 7;\n> @@ -200,15 +200,15 @@ static int edit_patch(int argc, const char **argv, const char *prefix)\n> \t\tdie(_(\"editing patch failed\"));\n> \n> \tif (stat(file, &st))\n> -\t\tdie_errno(_(\"Could not stat '%s'\"), file);\n> +\t\tdie_errno(_(\"could not stat '%s'\"), file);\n> \tif (!st.st_size)\n> -\t\tdie(_(\"Empty patch. Aborted.\"));\n> +\t\tdie(_(\"empty patch. aborted\"));\n> \n> \tchild.git_cmd = 1;\n> \tstrvec_pushl(&child.args, \"apply\", \"--recount\", \"--cached\", file,\n> \t\t NULL);\n> \tif (run_command(&child))\n> -\t\tdie(_(\"Could not apply '%s'\"), file);\n> +\t\tdie(_(\"could not apply '%s'\"), file);\n> \n> \tunlink(file);\n> \tfree(file);\n> @@ -568,7 +568,7 @@ int cmd_add(int argc, const char **argv, const char *prefix)\n> finish:\n> \tif (write_locked_index(&the_index, &lock_file,\n> \t\t\t COMMIT_LOCK | SKIP_IF_UNCHANGED))\n> -\t\tdie(_(\"Unable to write new index file\"));\n> +\t\tdie(_(\"unable to write new index file\"));\n> \n> \tdir_clear(&dir);\n> \tclear_pathspec(&pathspec);\n"},{"id":"482987","messageId":"CACS=G2yUGGJwD05KOFZK+AV3TSNDvDEfC=pFRsLwKX_-dgt+gA@mail.gmail.com","threadId":"60328","inReplyTo":"xmqqlecbzl5e.fsf@gitster.g","subject":"Re: [PATCH 1/1] [OUTREACHY] Fixed add.c file to conform to guidelines when using die() listed in issue #635","fromName":"Naomi Ibe","fromEmail":"naomi.ibeh69@gmail.com","sentAt":"2023-10-10T15:19:46Z","receivedAt":"2023-10-10T15:20:06Z","isPatch":true,"sender":{"key":"naomi.ibeh69@gmail.com","avatar":"https://avatars.githubusercontent.com/u/104323769?v=4"},"body":"Thank you very much! I'd definitely make those changes on my next patch.\nShould I begin work on version 2 or should I still wait for additional\ninput on the version 1?\n\nOn Mon, Oct 9, 2023 at 7:49 PM Junio C Hamano wrote:\n>\n> Naomi Ibe writes:\n>\n> > Subject: Re: [PATCH 1/1] [OUTREACHY] Fixed add.c file to conform to guidelines when using d\n>\n> Subject: [OUTREACHY][PATCH 1/1] builtin/add.c: clean up die() messages\n>\n> > From: Naomi \n>\n> The name and address on this line come from your commit object,\n> which in turn would have came from your configuration. You have\n>\n> [user]\n> name = Naomi\n> email = naomi.ibeh69@gmail.com\n>\n> somewhere in your configuration file, perhaps in $HOME/.gitconfig or\n> somewhere. When contributiong to this project, you want that \"name\"\n> line to also include your family name, as it should match what you\n> write on your Signed-off-by: line. A focused way to do so without\n> affecting your author identity for other projects is to add\n>\n> [user]\n> name = Naomi Ibe\n>\n> in .git/config of the repository that you use to contribute to this\n> project, e.g.,\n>\n> $ cd ... to the working tree of your clone of git you work in ...\n> $ git config user.name \"Naomi Ibe\"\n>\n> The space above your Sign off is to fill the details you omitted on\n> the title of the message (which would say something about \"conform\n> to guidelines\" or \"clean up\" without mentioning what rule the\n> original violates), making the whole thing something like:\n>\n> builtin/add.c: clean up die() messages\n>\n> As described in the CodingGuidelines document, a single line\n> message given to die() and its friends should not capitalize its\n> first word, and should not add full-stop at the end.\n>\n> Signed-off-by: ...\n>\n> > Signed-off-by: Naomi Ibe \n> > ---\n> > builtin/add.c | 10 +++++-----\n> > 1 file changed, 5 insertions(+), 5 deletions(-)\n>\n> Thanks. Otherwise the patch looks good.\n>\n> >\n> > diff --git a/builtin/add.c b/builtin/add.c\n> > index c27254a5cd..5126d2ede3 100644\n> > --- a/builtin/add.c\n> > +++ b/builtin/add.c\n> > @@ -182,7 +182,7 @@ static int edit_patch(int argc, const char **argv, const char *prefix)\n> > git_config(git_diff_basic_config, NULL); /* no \"diff\" UI options */\n> >\n> > if (repo_read_index(the_repository) < 0)\n> > - die(_(\"Could not read the index\"));\n> > + die(_(\"could not read the index\"));\n> >\n> > repo_init_revisions(the_repository, &rev, prefix);\n> > rev.diffopt.context = 7;\n> > @@ -200,15 +200,15 @@ static int edit_patch(int argc, const char **argv, const char *prefix)\n> > die(_(\"editing patch failed\"));\n> >\n> > if (stat(file, &st))\n> > - die_errno(_(\"Could not stat '%s'\"), file);\n> > + die_errno(_(\"could not stat '%s'\"), file);\n> > if (!st.st_size)\n> > - die(_(\"Empty patch. Aborted.\"));\n> > + die(_(\"empty patch. aborted\"));\n> >\n> > child.git_cmd = 1;\n> > strvec_pushl(&child.args, \"apply\", \"--recount\", \"--cached\", file,\n> > NULL);\n> > if (run_command(&child))\n> > - die(_(\"Could not apply '%s'\"), file);\n> > + die(_(\"could not apply '%s'\"), file);\n> >\n> > unlink(file);\n> > free(file);\n> > @@ -568,7 +568,7 @@ int cmd_add(int argc, const char **argv, const char *prefix)\n> > finish:\n> > if (write_locked_index(&the_index, &lock_file,\n> > COMMIT_LOCK | SKIP_IF_UNCHANGED))\n> > - die(_(\"Unable to write new index file\"));\n> > + die(_(\"unable to write new index file\"));\n> >\n> > dir_clear(&dir);\n> > clear_pathspec(&pathspec);\n"},{"id":"482998","messageId":"CAP8UFD3R1bDcbE8V9xbVhoEv9JuVLLXBErp9c=CJVvTuBvyykA@mail.gmail.com","threadId":"60328","inReplyTo":"CACS=G2yUGGJwD05KOFZK+AV3TSNDvDEfC=pFRsLwKX_-dgt+gA@mail.gmail.com","subject":"Re: [PATCH 1/1] [OUTREACHY] Fixed add.c file to conform to guidelines when using die() listed in issue #635","fromName":"Christian Couder","fromEmail":"christian.couder@gmail.com","sentAt":"2023-10-10T19:22:01Z","receivedAt":"2023-10-10T19:22:33Z","isPatch":true,"sender":{"key":"christian.couder@gmail.com","avatar":"https://avatars.githubusercontent.com/u/208954?v=4"},"body":"On Tue, Oct 10, 2023 at 5:20 PM Naomi Ibe wrote:\n>\n> Thank you very much! I'd definitely make those changes on my next patch.\n> Should I begin work on version 2 or should I still wait for additional\n> input on the version 1?\n\nI think you can work on version 2. Also please reply inline instead of\ntop-posting (see https://en.wikipedia.org/wiki/Posting_style).\n"},{"id":"483026","messageId":"xmqq34yiqfoh.fsf@gitster.g","threadId":"60328","inReplyTo":"CACS=G2yUGGJwD05KOFZK+AV3TSNDvDEfC=pFRsLwKX_-dgt+gA@mail.gmail.com","subject":"Re: [PATCH 1/1] [OUTREACHY] Fixed add.c file to conform to guidelines when using die() listed in issue #635","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2023-10-10T22:24:30Z","receivedAt":"2023-10-10T22:24:39Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Naomi Ibe writes:\n\n> Thank you very much! I'd definitely make those changes on my next patch.\n\n[administrivia] do not top post.\n\n> Should I begin work on version 2 or should I still wait for additional\n> input on the version 1?\n\nThere is no \"rule\", but based on observations on how people behave,\ne.g.\n\n * for a small patch like this that can be given a good review in 10\n minutes or so, those who do not do so within the first 3 days\n will probably not do so.\n\n * once a reasonably thorough review is given, those who haven't\n responded to the patch and do not have much else to say are\n unlikely to respond.\n\n * on the other hand, after such a review is given, those who do not\n agree with the review tend to respond rather quickly, to get\n their voice in before it becomes too late.\n\nI would say it would be good to start working on it right away and\nuse a couple of days reviewing it yourself before posting it.\n"},{"id":"483035","messageId":"CACS=G2wXkz9OyR5e0ADzWVy3ejibF-Js=sdJYJDRMh8qskO-9A@mail.gmail.com","threadId":"60328","inReplyTo":"xmqq34yiqfoh.fsf@gitster.g","subject":"Re: [PATCH 1/1] [OUTREACHY] Fixed add.c file to conform to guidelines when using die() listed in issue #635","fromName":"Naomi Ibe","fromEmail":"naomi.ibeh69@gmail.com","sentAt":"2023-10-11T05:50:30Z","receivedAt":"2023-10-11T05:50:52Z","isPatch":true,"sender":{"key":"naomi.ibeh69@gmail.com","avatar":"https://avatars.githubusercontent.com/u/104323769?v=4"},"body":"Okay then, I'll work with these guidelines in mind. Thank you very much\n\nOn Tue, Oct 10, 2023 at 11:24 PM Junio C Hamano wrote:\n>\n> Naomi Ibe writes:\n>\n> > Thank you very much! I'd definitely make those changes on my next patch.\n>\n> [administrivia] do not top post.\n>\n> > Should I begin work on version 2 or should I still wait for additional\n> > input on the version 1?\n>\n> There is no \"rule\", but based on observations on how people behave,\n> e.g.\n>\n> * for a small patch like this that can be given a good review in 10\n> minutes or so, those who do not do so within the first 3 days\n> will probably not do so.\n>\n> * once a reasonably thorough review is given, those who haven't\n> responded to the patch and do not have much else to say are\n> unlikely to respond.\n>\n> * on the other hand, after such a review is given, those who do not\n> agree with the review tend to respond rather quickly, to get\n> their voice in before it becomes too late.\n>\n> I would say it would be good to start working on it right away and\n> use a couple of days reviewing it yourself before posting it.\n"},{"id":"483037","messageId":"89c51050c7570439f763f8fc8195aada@manjaro.org","threadId":"60328","inReplyTo":"CACS=G2wXkz9OyR5e0ADzWVy3ejibF-Js=sdJYJDRMh8qskO-9A@mail.gmail.com","subject":"Re: [PATCH 1/1] [OUTREACHY] Fixed add.c file to conform to guidelines when using die() listed in issue #635","fromName":"Dragan Simic","fromEmail":"dsimic@manjaro.org","sentAt":"2023-10-11T06:17:10Z","receivedAt":"2023-10-11T07:13:58Z","isPatch":true,"sender":{"key":"dsimic@manjaro.org","avatar":null},"body":"On 2023-10-11 07:50, Naomi Ibe wrote:\n> Okay then, I'll work with these guidelines in mind. Thank you very much\n\nIf I may notice, avoiding top posting has already been suggested twice, \nIIRC.\n\nI think that various webmails have made top posting popular, and \nactually forced people a bit to do that through the way their user \ninterfaces were laid out, which made inline replies kind of a dying art, \nwhich is really sad to see. It is so much more useful when replies are \nposted inline, i.e. right below the contents they refer to.\n\n\n> On Tue, Oct 10, 2023 at 11:24 PM Junio C Hamano \n> wrote:\n>> \n>> Naomi Ibe writes:\n>> \n>> > Thank you very much! I'd definitely make those changes on my next patch.\n>> \n>> [administrivia] do not top post.\n>> \n>> > Should I begin work on version 2 or should I still wait for additional\n>> > input on the version 1?\n>> \n>> There is no \"rule\", but based on observations on how people behave,\n>> e.g.\n>> \n>> * for a small patch like this that can be given a good review in 10\n>> minutes or so, those who do not do so within the first 3 days\n>> will probably not do so.\n>> \n>> * once a reasonably thorough review is given, those who haven't\n>> responded to the patch and do not have much else to say are\n>> unlikely to respond.\n>> \n>> * on the other hand, after such a review is given, those who do not\n>> agree with the review tend to respond rather quickly, to get\n>> their voice in before it becomes too late.\n>> \n>> I would say it would be good to start working on it right away and\n>> use a couple of days reviewing it yourself before posting it.\n"}]}