{"thread":{"id":"60185","subject":"[RFC] New configuration option \"diff.statNameWidth\"","startedAt":"2023-09-02T18:11:56Z","lastAt":"2023-09-12T02:38:19Z","messageCount":8,"participants":["Dragan Simic","Junio C Hamano"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"481329","messageId":"eb8f524eca3975f086715ec32a8a1fbb@manjaro.org","threadId":"60185","inReplyTo":null,"subject":"[RFC] New configuration option \"diff.statNameWidth\"","fromName":"Dragan Simic","fromEmail":"dsimic@manjaro.org","sentAt":"2023-09-02T18:03:11Z","receivedAt":"2023-09-02T18:11:56Z","isPatch":false,"sender":{"key":"dsimic@manjaro.org","avatar":null},"body":"Hello,\n\nI'd like to implement support for a new configuration option named \n\"diff.statNameWidth\" and submit the patch, so I'd like to check first \nwould that patch be accepted and merged.\n\nSimilarly to the already existing configuration option \n\"diff.statGraphWidth\",[1][2] \"diff.statNameWidth\" as a new configuration \noption would limit the width of the filename part in generated diffstat \noutputs, just like the command-line option \"--stat-name-width=\" \nalready does.\n\nHaving \"diff.statNameWidth\" as a new configuration option would be quite \nusable, and it would fit together nicely with the already existing \n\"diff.statGraphWidth\" configuration option. For exaple, I have quite a \nfew aliases in my ~/.gitconfig that contain \"--stat-name-width=\", and \nhaving this new configuration option would allow those aliases to be \nmuch shorter. Of course, it would benefit all other uses in which \ndiffstats are generated.\n\nPlease advise.\n\nLinks:\n[1] \nhttps://git-scm.com/docs/git-diff#Documentation/git-diff.txt---statltwidthgtltname-widthgtltcountgt\n[2] \nhttps://git-scm.com/docs/git-config#Documentation/git-config.txt-diffstatGraphWidth\n"},{"id":"481331","messageId":"xmqq8r9ommyt.fsf@gitster.g","threadId":"60185","inReplyTo":"eb8f524eca3975f086715ec32a8a1fbb@manjaro.org","subject":"Re: [RFC] New configuration option \"diff.statNameWidth\"","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2023-09-02T18:47:22Z","receivedAt":"2023-09-02T18:47:31Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Dragan Simic writes:\n\n> I'd like to implement support for a new configuration option named\n> \"diff.statNameWidth\" and submit the patch, so I'd like to check first\n> would that patch be accepted and merged.\n\nIn general, we do not give promises or estimates. The devil is in\nthe details and until we see the design we may not know if an\noverall idea is good. Even when it is obviously a good idea, we\nwould not know the quality of the implementation until we see it.\n\n - If something is worth adding, even if we do not accept it in the\n upstream first, it will spread among users and developers, and\n eventually we may realize the mistake of initially not taking it\n and we may come begging to the contributor for upstreaming.\n\n - On the other hand, a new thing that even the contributor\n themselves are unsure if it is worth investing their work in, if\n it is only to use it for themselves, is very unlikely to be of\n interest to us or our users.\n\n\"If this will be accepted, I'll work on it\" is a very counter\nproductive thing to say around here. It is easily (mis)taken as a\nsign that it is the latter case. \"This is a good idea, I believe in\nit, and I'll work on it whether you initially show interest or not\"\nis what we want to see, and such a patch will not need a \"check\nfirst\" letter.\n\nIn other words, make it so good that we would come to you, begging\n;-).\n\nThanks.\n"},{"id":"481333","messageId":"72c114086590b9b15a3fdd9e0d6bd67e@manjaro.org","threadId":"60185","inReplyTo":"xmqq8r9ommyt.fsf@gitster.g","subject":"Re: [RFC] New configuration option \"diff.statNameWidth\"","fromName":"Dragan Simic","fromEmail":"dsimic@manjaro.org","sentAt":"2023-09-02T18:56:58Z","receivedAt":"2023-09-02T18:57:02Z","isPatch":false,"sender":{"key":"dsimic@manjaro.org","avatar":null},"body":"On 2023-09-02 20:47, Junio C Hamano wrote:\n> Dragan Simic writes:\n> \n>> I'd like to implement support for a new configuration option named\n>> \"diff.statNameWidth\" and submit the patch, so I'd like to check first\n>> would that patch be accepted and merged.\n> \n> In general, we do not give promises or estimates. The devil is in\n> the details and until we see the design we may not know if an\n> overall idea is good. Even when it is obviously a good idea, we\n> would not know the quality of the implementation until we see it.\n\nOh, totally! A good idea with a poor implementation is something that \nsimply can't be accepted and merged. No promises can be made until the \ncode is available for review.\n\n> - If something is worth adding, even if we do not accept it in the\n> upstream first, it will spread among users and developers, and\n> eventually we may realize the mistake of initially not taking it\n> and we may come begging to the contributor for upstreaming.\n> \n> - On the other hand, a new thing that even the contributor\n> themselves are unsure if it is worth investing their work in, if\n> it is only to use it for themselves, is very unlikely to be of\n> interest to us or our users.\n\nI agree, the mantra of open-source could be formulated as something like \n\"find an itch and scratch it\" or \"build it and they will come\". In \nother words, if someone finds something useful and worth investing their \ntime, perhaps other people will find it useful, too.\n\n> \"If this will be accepted, I'll work on it\" is a very counter\n> productive thing to say around here. It is easily (mis)taken as a\n> sign that it is the latter case. \"This is a good idea, I believe in\n> it, and I'll work on it whether you initially show interest or not\"\n> is what we want to see, and such a patch will not need a \"check\n> first\" letter.\n> \n> In other words, make it so good that we would come to you, begging\n> ;-).\n\nThank you very much for your thoughts and the time required to write it \nall down. I'll do my best to make my patch(es) irresistible. :)\n"},{"id":"481334","messageId":"xmqq4jkcmdaw.fsf@gitster.g","threadId":"60185","inReplyTo":"72c114086590b9b15a3fdd9e0d6bd67e@manjaro.org","subject":"Re: [RFC] New configuration option \"diff.statNameWidth\"","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2023-09-02T22:16:07Z","receivedAt":"2023-09-02T22:16:17Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Dragan Simic writes:\n\n> Thank you very much for your thoughts and the time required to write\n> it all down. I'll do my best to make my patch(es) irresistible. :)\n\nHeh, I have to send the same from time to time, so it didn't take\nthat much time. Having said all the above, once we start seeing the\nactual patches and its sales pitch (aka proposed commit log\nmessage), we do guide and help polishing the patch into a better\nshape, so it will not be like the contributor has to work in the\ndark without knowing what level of bar their contribution has to\npass.\n\nThanks.\n"},{"id":"481345","messageId":"a879c9c7d3b9bdae9a49f67fbe6316fc@manjaro.org","threadId":"60185","inReplyTo":"xmqq4jkcmdaw.fsf@gitster.g","subject":"Re: [RFC] New configuration option \"diff.statNameWidth\"","fromName":"Dragan Simic","fromEmail":"dsimic@manjaro.org","sentAt":"2023-09-03T03:43:18Z","receivedAt":"2023-09-03T03:46:56Z","isPatch":false,"sender":{"key":"dsimic@manjaro.org","avatar":null},"body":"On 2023-09-03 00:16, Junio C Hamano wrote:\n> Dragan Simic writes:\n> \n>> Thank you very much for your thoughts and the time required to write\n>> it all down. I'll do my best to make my patch(es) irresistible. :)\n> \n> Heh, I have to send the same from time to time, so it didn't take\n> that much time.\n\nQuite frankly, I suspected that to be a canned response, :) but anyway, \npreparing that response, and possibly refining it over a couple of \niterations, surely already took a fair amount of time and effort.\n\n> Having said all the above, once we start seeing the actual patches\n> and its sales pitch (aka proposed commit log message), we do guide\n> and help polishing the patch into a better shape, so it will not\n> be like the contributor has to work in the dark without knowing\n> what level of bar their contribution has to pass.\n\nThanks, everything sounds great and welcoming to the new contributors, \nwho of course need to be willing to put in the required amount of skill \nand effort.\n"},{"id":"481693","messageId":"261907dbfafc8d7b3b048854b663e82d@manjaro.org","threadId":"60185","inReplyTo":"a879c9c7d3b9bdae9a49f67fbe6316fc@manjaro.org","subject":"Re: [RFC] New configuration option \"diff.statNameWidth\"","fromName":"Dragan Simic","fromEmail":"dsimic@manjaro.org","sentAt":"2023-09-11T18:56:43Z","receivedAt":"2023-09-11T21:39:13Z","isPatch":false,"sender":{"key":"dsimic@manjaro.org","avatar":null},"body":"On 2023-09-03 05:43, Dragan Simic wrote:\n> On 2023-09-03 00:16, Junio C Hamano wrote:\n>> Having said all the above, once we start seeing the actual patches\n>> and its sales pitch (aka proposed commit log message), we do guide\n>> and help polishing the patch into a better shape, so it will not\n>> be like the contributor has to work in the dark without knowing\n>> what level of bar their contribution has to pass.\n> \n> Thanks, everything sounds great and welcoming to the new contributors,\n> who of course need to be willing to put in the required amount of\n> skill and effort.\n\nI sent a patch to the git mailing list today, about five hours ago, but \nit hasn't appeared on the list yet. Could something be wrong with the \nmail server(s), as I also received no other messages from the list in \nthe last six hours or so?\n"},{"id":"481724","messageId":"0d62fba447dfb1c776d3c4a3d269e974@manjaro.org","threadId":"60185","inReplyTo":"xmqqo7i8s48f.fsf@gitster.g","subject":"Re: [RFC] New configuration option \"diff.statNameWidth\"","fromName":"Dragan Simic","fromEmail":"dsimic@manjaro.org","sentAt":"2023-09-12T00:26:10Z","receivedAt":"2023-09-12T01:35:10Z","isPatch":false,"sender":{"key":"dsimic@manjaro.org","avatar":null},"body":"On 2023-09-12 01:02, Junio C Hamano wrote:\n> Dragan Simic writes:\n> \n>> On 2023-09-03 05:43, Dragan Simic wrote:\n>>> On 2023-09-03 00:16, Junio C Hamano wrote:\n>>>> Having said all the above, once we start seeing the actual patches\n>>>> and its sales pitch (aka proposed commit log message), we do guide\n>>>> and help polishing the patch into a better shape, so it will not\n>>>> be like the contributor has to work in the dark without knowing\n>>>> what level of bar their contribution has to pass.\n>>> Thanks, everything sounds great and welcoming to the new\n>>> contributors,\n>>> who of course need to be willing to put in the required amount of\n>>> skill and effort.\n>> \n>> I sent a patch to the git mailing list today, about five hours ago,\n>> but it hasn't appeared on the list yet. Could something be wrong with\n>> the mail server(s), as I also received no other messages from the list\n>> in the last six hours or so?\n> \n> It is a high-volume mailing list server and occasional hiccup is\n> part of regular life. Waiting a bit and then poking the postmater\n> at vger.kernel.org may be needed from time to time, but I am seeing\n> your message on the archive, so it seems \"waiting\" has worked fine?\n\nYes, remaining patient did the trick this time. A whole lot of messages \nfrom the list arrived at once after the hiccup resolved itself.\n"},{"id":"481734","messageId":"xmqqo7i8s48f.fsf@gitster.g","threadId":"60185","inReplyTo":"261907dbfafc8d7b3b048854b663e82d@manjaro.org","subject":"Re: [RFC] New configuration option \"diff.statNameWidth\"","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2023-09-11T23:02:40Z","receivedAt":"2023-09-12T02:38:19Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Dragan Simic writes:\n\n> On 2023-09-03 05:43, Dragan Simic wrote:\n>> On 2023-09-03 00:16, Junio C Hamano wrote:\n>>> Having said all the above, once we start seeing the actual patches\n>>> and its sales pitch (aka proposed commit log message), we do guide\n>>> and help polishing the patch into a better shape, so it will not\n>>> be like the contributor has to work in the dark without knowing\n>>> what level of bar their contribution has to pass.\n>> Thanks, everything sounds great and welcoming to the new\n>> contributors,\n>> who of course need to be willing to put in the required amount of\n>> skill and effort.\n>\n> I sent a patch to the git mailing list today, about five hours ago,\n> but it hasn't appeared on the list yet. Could something be wrong with\n> the mail server(s), as I also received no other messages from the list\n> in the last six hours or so?\n\nIt is a high-volume mailing list server and occasional hiccup is\npart of regular life. Waiting a bit and then poking the postmater\nat vger.kernel.org may be needed from time to time, but I am seeing\nyour message on the archive, so it seems \"waiting\" has worked fine?\n\n"}]}