{"thread":{"id":"35233","subject":"git-contacts questions","startedAt":"2013-10-29T08:34:19Z","lastAt":"2013-10-29T20:42:14Z","messageCount":10,"participants":["Sebastian Schuberth","Felipe Contreras","Matthieu Moy","Eric Sunshine"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"229713","messageId":"CAHGBnuPW3dtiL-=6BgvaOB4VvOAPyZhrVS3WGreVH_Pt81=wyg@mail.gmail.com","threadId":"35233","inReplyTo":null,"subject":"git-contacts questions","fromName":"Sebastian Schuberth","fromEmail":"sschuberth@gmail.com","sentAt":"2013-10-29T08:34:19Z","receivedAt":"2013-10-29T08:34:19Z","isPatch":false,"sender":{"key":"sschuberth@gmail.com","avatar":"https://avatars.githubusercontent.com/u/349154?v=4"},"body":"Hi,\n\nI'm currently trying to get git-contacts to work under Windows / MSYS.\nI have a few issues that may be due to me using the script in a wrong\nway, or the script having some bugs, so I'd like to ask you for help.\n\n1) Passing just \"HEAD\" as a committish like in \"git contacts HEAD\"\ndoes not output anything for me, but using the SHA1 for HEAD does\nneither. My HEAD commit does not add any files, but only modifies\npreviously existing files, so I would have expected some output. In\ncase it turns out to be correct to have no output in my case, could we\nprobably say that in some message to the user?\n\n2) For some commits I get error messages from \"git blame\" in function\nget_blame because multiple -L options are specified:\n\n$ git contacts b0783baacd20be7007df40cf274985c4863d63fb\nfatal: More than one '-L n,m' option given\nfatal: More than one '-L n,m' option given\nfatal: More than one '-L n,m' option given\nfatal: More than one '-L n,m' option given\n\n>From reading git-blame's man page it indeed seems to me as if only one\n-L option is allowed, so is this something that needs to be fixed in\ngit-contacts?\n\nThanks.\n\n-- \nSebastian Schuberth\n"},{"id":"229716","messageId":"CAMP44s1+npNnR8-exARL8kamQ81JwyLhhQnnkv6DvK8e3qOfHA@mail.gmail.com","threadId":"35233","inReplyTo":"CAHGBnuPW3dtiL-=6BgvaOB4VvOAPyZhrVS3WGreVH_Pt81=wyg@mail.gmail.com","subject":"Re: git-contacts questions","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2013-10-29T08:57:48Z","receivedAt":"2013-10-29T08:57:48Z","isPatch":false,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"On Tue, Oct 29, 2013 at 2:34 AM, Sebastian Schuberth\n wrote:\n\n> 1) Passing just \"HEAD\" as a committish like in \"git contacts HEAD\"\n> does not output anything for me, but using the SHA1 for HEAD does\n> neither. My HEAD commit does not add any files, but only modifies\n> previously existing files, so I would have expected some output. In\n> case it turns out to be correct to have no output in my case, could we\n> probably say that in some message to the user?\n\nIt should be HEAD^, or -1, like with 'git format-patch'.\n\n> 2) For some commits I get error messages from \"git blame\" in function\n> get_blame because multiple -L options are specified:\n>\n> $ git contacts b0783baacd20be7007df40cf274985c4863d63fb\n> fatal: More than one '-L n,m' option given\n> fatal: More than one '-L n,m' option given\n> fatal: More than one '-L n,m' option given\n> fatal: More than one '-L n,m' option given\n>\n> From reading git-blame's man page it indeed seems to me as if only one\n> -L option is allowed, so is this something that needs to be fixed in\n> git-contacts?\n\nYou are probably using a newer version of the script on an older version of Git.\n\nYou can also try git-related instead[1].\n\n[1] https://github.com/felipec/git-related\n\n-- \nFelipe Contreras\n"},{"id":"229718","messageId":"CAHGBnuP6nyTCGGNntvWVYAtxNX9xgt6=F3HheiEUcPU_9TP-1g@mail.gmail.com","threadId":"35233","inReplyTo":"CAMP44s1+npNnR8-exARL8kamQ81JwyLhhQnnkv6DvK8e3qOfHA@mail.gmail.com","subject":"Re: git-contacts questions","fromName":"Sebastian Schuberth","fromEmail":"sschuberth@gmail.com","sentAt":"2013-10-29T09:12:10Z","receivedAt":"2013-10-29T09:12:10Z","isPatch":false,"sender":{"key":"sschuberth@gmail.com","avatar":"https://avatars.githubusercontent.com/u/349154?v=4"},"body":"On Tue, Oct 29, 2013 at 9:57 AM, Felipe Contreras\n wrote:\n\n>> 1) Passing just \"HEAD\" as a committish like in \"git contacts HEAD\"\n>> does not output anything for me, but using the SHA1 for HEAD does\n>> neither. My HEAD commit does not add any files, but only modifies\n>> previously existing files, so I would have expected some output. In\n>> case it turns out to be correct to have no output in my case, could we\n>> probably say that in some message to the user?\n>\n> It should be HEAD^, or -1, like with 'git format-patch'.\n\nOh, that's pretty much unexpected. Wouldn't it be much more natural if\nI had to specify the commit(s) that introduce(s) the changes that I\nwant others to look at?\n\n>> 2) For some commits I get error messages from \"git blame\" in function\n>> get_blame because multiple -L options are specified:\n>>\n>> $ git contacts b0783baacd20be7007df40cf274985c4863d63fb\n>> fatal: More than one '-L n,m' option given\n>> fatal: More than one '-L n,m' option given\n>> fatal: More than one '-L n,m' option given\n>> fatal: More than one '-L n,m' option given\n>>\n>> From reading git-blame's man page it indeed seems to me as if only one\n>> -L option is allowed, so is this something that needs to be fixed in\n>> git-contacts?\n>\n> You are probably using a newer version of the script on an older version of Git.\n\nWell, I'm running Git for Windows 1.8.4, and according to [1]\ngit-blame was not changed after 1.8.4.\n\n> You can also try git-related instead[1].\n>\n> [1] https://github.com/felipec/git-related\n\nI just did very quickly (with Ruby 1.8.7) and I get:\n\n$ git related b0783baacd20be7007df40cf274985c4863d63fb\nC:/Program Files (x86)/Git/libexec/git-core/git-related:309:in\n`popen': can't convert Array into String (TypeError)\n from C:/Program Files\n(x86)/Git/libexec/git-core/git-related:309:in `from_rev_args'\n from C:/Program Files (x86)/Git/libexec/git-core/git-related:345\n\n[1] http://git-scm.com/docs/git-blame.html\n\n-- \nSebastian Schuberth\n"},{"id":"229719","messageId":"CAMP44s2KdHPH-UaSwdiJtT8WNo7uT9K_ArXwqHhCryPRsX1qjA@mail.gmail.com","threadId":"35233","inReplyTo":"CAHGBnuP6nyTCGGNntvWVYAtxNX9xgt6=F3HheiEUcPU_9TP-1g@mail.gmail.com","subject":"Re: git-contacts questions","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2013-10-29T10:35:49Z","receivedAt":"2013-10-29T10:35:49Z","isPatch":false,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"On Tue, Oct 29, 2013 at 3:12 AM, Sebastian Schuberth\n wrote:\n> On Tue, Oct 29, 2013 at 9:57 AM, Felipe Contreras\n> wrote:\n>\n>>> 1) Passing just \"HEAD\" as a committish like in \"git contacts HEAD\"\n>>> does not output anything for me, but using the SHA1 for HEAD does\n>>> neither. My HEAD commit does not add any files, but only modifies\n>>> previously existing files, so I would have expected some output. In\n>>> case it turns out to be correct to have no output in my case, could we\n>>> probably say that in some message to the user?\n>>\n>> It should be HEAD^, or -1, like with 'git format-patch'.\n>\n> Oh, that's pretty much unexpected. Wouldn't it be much more natural if\n> I had to specify the commit(s) that introduce(s) the changes that I\n> want others to look at?\n\nYeah, that's exactly what you are doing. How do you tell 'git log' to\nshow you certain changes?\n\n>> You can also try git-related instead[1].\n>>\n>> [1] https://github.com/felipec/git-related\n>\n> I just did very quickly (with Ruby 1.8.7) and I get:\n>\n> $ git related b0783baacd20be7007df40cf274985c4863d63fb\n> C:/Program Files (x86)/Git/libexec/git-core/git-related:309:in\n> `popen': can't convert Array into String (TypeError)\n> from C:/Program Files\n> (x86)/Git/libexec/git-core/git-related:309:in `from_rev_args'\n> from C:/Program Files (x86)/Git/libexec/git-core/git-related:345\n\nFixed.\n\n-- \nFelipe Contreras\n"},{"id":"229725","messageId":"CAHGBnuNWtgH_aYxv0gX44AkGTRo6vsKdd4w1cViLGD9O9RLaNQ@mail.gmail.com","threadId":"35233","inReplyTo":"CAMP44s2KdHPH-UaSwdiJtT8WNo7uT9K_ArXwqHhCryPRsX1qjA@mail.gmail.com","subject":"Re: git-contacts questions","fromName":"Sebastian Schuberth","fromEmail":"sschuberth@gmail.com","sentAt":"2013-10-29T14:17:57Z","receivedAt":"2013-10-29T14:17:57Z","isPatch":false,"sender":{"key":"sschuberth@gmail.com","avatar":"https://avatars.githubusercontent.com/u/349154?v=4"},"body":"On Tue, Oct 29, 2013 at 11:35 AM, Felipe Contreras\n wrote:\n> On Tue, Oct 29, 2013 at 3:12 AM, Sebastian Schuberth\n> wrote:\n>> On Tue, Oct 29, 2013 at 9:57 AM, Felipe Contreras\n>> wrote:\n>>\n>>>> 1) Passing just \"HEAD\" as a committish like in \"git contacts HEAD\"\n>>>> does not output anything for me, but using the SHA1 for HEAD does\n>>>> neither. My HEAD commit does not add any files, but only modifies\n>>>> previously existing files, so I would have expected some output. In\n>>>> case it turns out to be correct to have no output in my case, could we\n>>>> probably say that in some message to the user?\n>>>\n>>> It should be HEAD^, or -1, like with 'git format-patch'.\n>>\n>> Oh, that's pretty much unexpected. Wouldn't it be much more natural if\n>> I had to specify the commit(s) that introduce(s) the changes that I\n>> want others to look at?\n>\n> Yeah, that's exactly what you are doing. How do you tell 'git log' to\n> show you certain changes?\n\nI'm not sure what you're trying to point me at. It's clear that from\nan implementation view you need to blame HEAD^ if you need to know\nwhich poeple should review your changes in HEAD. But IMHO that is an\nimplementation detail that should be hidden from the user. Like I\nsaid, I think it would be much more natural if it worked by specifying\nthe commits that *I* introduced, and then lists all people to contact\nfor a review.\n\n>> $ git related b0783baacd20be7007df40cf274985c4863d63fb\n>> C:/Program Files (x86)/Git/libexec/git-core/git-related:309:in\n>> `popen': can't convert Array into String (TypeError)\n>> from C:/Program Files\n>> (x86)/Git/libexec/git-core/git-related:309:in `from_rev_args'\n>> from C:/Program Files (x86)/Git/libexec/git-core/git-related:345\n>\n> Fixed.\n\nThanks, but now I'm getting\n\n$ git related b0783baacd20be7007df40cf274985c4863d63fb\nfatal: ambiguous argument '\\^b0783baacd20be7007df40cf274985c4863d63fb': unknown\nrevision or path not in the working tree.\nUse '--' to separate paths from revisions, like this:\n'git [...] -- [...]'\n\nI've checked of course that the commit exists.\n\n-- \nSebastian Schuberth\n"},{"id":"229727","messageId":"vpqsivkywdx.fsf@anie.imag.fr","threadId":"35233","inReplyTo":"CAHGBnuNWtgH_aYxv0gX44AkGTRo6vsKdd4w1cViLGD9O9RLaNQ@mail.gmail.com","subject":"Re: git-contacts questions","fromName":"Matthieu Moy","fromEmail":"matthieu.moy@grenoble-inp.fr","sentAt":"2013-10-29T14:39:06Z","receivedAt":"2013-10-29T14:39:06Z","isPatch":false,"sender":{"key":"matthieu.moy@grenoble-inp.fr","avatar":"https://gravatar.com/avatar/72c8a2705971a25dfaff23cece15130d405685845d911aedd5667ace277f3fc5?d=mp&s=160"},"body":"Sebastian Schuberth writes:\n\n> On Tue, Oct 29, 2013 at 11:35 AM, Felipe Contreras\n> wrote:\n>> On Tue, Oct 29, 2013 at 3:12 AM, Sebastian Schuberth\n>> wrote:\n>>> On Tue, Oct 29, 2013 at 9:57 AM, Felipe Contreras\n>>> wrote:\n>>>\n>>>>> 1) Passing just \"HEAD\" as a committish like in \"git contacts HEAD\"\n>>>>> does not output anything for me, but using the SHA1 for HEAD does\n>>>>> neither. My HEAD commit does not add any files, but only modifies\n>>>>> previously existing files, so I would have expected some output. In\n>>>>> case it turns out to be correct to have no output in my case, could we\n>>>>> probably say that in some message to the user?\n>>>>\n>>>> It should be HEAD^, or -1, like with 'git format-patch'.\n>>>\n>>> Oh, that's pretty much unexpected. Wouldn't it be much more natural if\n>>> I had to specify the commit(s) that introduce(s) the changes that I\n>>> want others to look at?\n>>\n>> Yeah, that's exactly what you are doing. How do you tell 'git log' to\n>> show you certain changes?\n>\n> I'm not sure what you're trying to point me at. It's clear that from\n> an implementation view you need to blame HEAD^ if you need to know\n> which poeple should review your changes in HEAD.\n\nI agree that the situation when providing only HEAD is really\ndisappointing...\n\n> But IMHO that is an implementation detail that should be hidden from\n> the user.\n\n... but it's not just an implementation detail: git-contacts takes a\nrange of commits, so you can ask for people to Cc for a whole patch\nseries for example.\n\nIf I understand correctly, \"git contact $ONE_COMMIT\" does\n\"git contact $ONE_COMMIT..HEAD\" implicitly, and this is weird when\n$ONE_COMMIT is HEAD.\n\n-- \nMatthieu Moy\nhttp://www-verimag.imag.fr/~moy/\n"},{"id":"229729","messageId":"CAHGBnuOT1ttzSuzCt4GDxJ7iap6VM1Sbyuve2FGR7pdf2z40bg@mail.gmail.com","threadId":"35233","inReplyTo":"vpqsivkywdx.fsf@anie.imag.fr","subject":"Re: git-contacts questions","fromName":"Sebastian Schuberth","fromEmail":"sschuberth@gmail.com","sentAt":"2013-10-29T15:02:29Z","receivedAt":"2013-10-29T15:02:29Z","isPatch":false,"sender":{"key":"sschuberth@gmail.com","avatar":"https://avatars.githubusercontent.com/u/349154?v=4"},"body":"On Tue, Oct 29, 2013 at 3:39 PM, Matthieu Moy\n wrote:\n\n>> I'm not sure what you're trying to point me at. It's clear that from\n>> an implementation view you need to blame HEAD^ if you need to know\n>> which poeple should review your changes in HEAD.\n>\n> I agree that the situation when providing only HEAD is really\n> disappointing...\n>\n>> But IMHO that is an implementation detail that should be hidden from\n>> the user.\n>\n> ... but it's not just an implementation detail: git-contacts takes a\n> range of commits, so you can ask for people to Cc for a whole patch\n> series for example.\n>\n> If I understand correctly, \"git contact $ONE_COMMIT\" does\n> \"git contact $ONE_COMMIT..HEAD\" implicitly, and this is weird when\n> $ONE_COMMIT is HEAD.\n\nIt's not only weird if $ONE_COMMIT is HEAD, but for any single commit.\nAt least I would expect \"git contacts $ONE_COMMIT\" to list the poeple\ninterested in the single commit $ONE_COMMIT. Instead, what it seems to\ndo is list the people who are interested in the commits *after*\n$ONE_COMMIT.\n\n-- \nSebastian Schuberth\n"},{"id":"229734","messageId":"CAMP44s0Ou4S5Bbf9sHwnXFJazu12uHvjuTOTT4YK_8bg4MG2UQ@mail.gmail.com","threadId":"35233","inReplyTo":"CAHGBnuNWtgH_aYxv0gX44AkGTRo6vsKdd4w1cViLGD9O9RLaNQ@mail.gmail.com","subject":"Re: git-contacts questions","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2013-10-29T16:59:56Z","receivedAt":"2013-10-29T16:59:56Z","isPatch":false,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"On Tue, Oct 29, 2013 at 8:17 AM, Sebastian Schuberth\n wrote:\n> On Tue, Oct 29, 2013 at 11:35 AM, Felipe Contreras\n> wrote:\n>> On Tue, Oct 29, 2013 at 3:12 AM, Sebastian Schuberth\n>> wrote:\n\n>>> $ git related b0783baacd20be7007df40cf274985c4863d63fb\n>>> C:/Program Files (x86)/Git/libexec/git-core/git-related:309:in\n>>> `popen': can't convert Array into String (TypeError)\n>>> from C:/Program Files\n>>> (x86)/Git/libexec/git-core/git-related:309:in `from_rev_args'\n>>> from C:/Program Files (x86)/Git/libexec/git-core/git-related:345\n>>\n>> Fixed.\n>\n> Thanks, but now I'm getting\n>\n> $ git related b0783baacd20be7007df40cf274985c4863d63fb\n> fatal: ambiguous argument '\\^b0783baacd20be7007df40cf274985c4863d63fb': unknown\n> revision or path not in the working tree.\n> Use '--' to separate paths from revisions, like this:\n> 'git [...] -- [...]'\n>\n> I've checked of course that the commit exists.\n\nI've pushed a fix, it's not complete, but should work for most cases.\n\n-- \nFelipe Contreras\n"},{"id":"229737","messageId":"CAPig+cRpEvAjiQWZ08kW2oqwR6XZ=u7dE=3OfVyB9SZzf8+saw@mail.gmail.com","threadId":"35233","inReplyTo":"CAMP44s1+npNnR8-exARL8kamQ81JwyLhhQnnkv6DvK8e3qOfHA@mail.gmail.com","subject":"Re: git-contacts questions","fromName":"Eric Sunshine","fromEmail":"sunshine@sunshineco.com","sentAt":"2013-10-29T19:02:48Z","receivedAt":"2013-10-29T19:02:48Z","isPatch":false,"sender":{"key":"sunshine@sunshineco.com","avatar":"https://avatars.githubusercontent.com/u/163641?v=4"},"body":"On Tue, Oct 29, 2013 at 4:57 AM, Felipe Contreras\n wrote:\n> On Tue, Oct 29, 2013 at 2:34 AM, Sebastian Schuberth\n> wrote:\n>> 2) For some commits I get error messages from \"git blame\" in function\n>> get_blame because multiple -L options are specified:\n>>\n>> $ git contacts b0783baacd20be7007df40cf274985c4863d63fb\n>> fatal: More than one '-L n,m' option given\n>> fatal: More than one '-L n,m' option given\n>> fatal: More than one '-L n,m' option given\n>> fatal: More than one '-L n,m' option given\n>>\n>> From reading git-blame's man page it indeed seems to me as if only one\n>> -L option is allowed, so is this something that needs to be fixed in\n>> git-contacts?\n>\n> You are probably using a newer version of the script on an older version of Git.\n\nRight. git-blame learned to accept multiple -L's in 58dbfa2e59a1, and\ngit-contacts started taking advantage of that feature in 4c70cfbfbc2d.\nA git-contacts prior to 4c70cfbfbc2d might work for you.\n"},{"id":"229746","messageId":"CAHGBnuOZF+poa1OpgmayqAkwAuZWMESPQJPVqNjWwkS1i3t-OA@mail.gmail.com","threadId":"35233","inReplyTo":"CAPig+cRpEvAjiQWZ08kW2oqwR6XZ=u7dE=3OfVyB9SZzf8+saw@mail.gmail.com","subject":"Re: git-contacts questions","fromName":"Sebastian Schuberth","fromEmail":"sschuberth@gmail.com","sentAt":"2013-10-29T20:42:14Z","receivedAt":"2013-10-29T20:42:14Z","isPatch":false,"sender":{"key":"sschuberth@gmail.com","avatar":"https://avatars.githubusercontent.com/u/349154?v=4"},"body":"On Tue, Oct 29, 2013 at 8:02 PM, Eric Sunshine wrote:\n\n>>> $ git contacts b0783baacd20be7007df40cf274985c4863d63fb\n>>> fatal: More than one '-L n,m' option given\n>>> fatal: More than one '-L n,m' option given\n>>> fatal: More than one '-L n,m' option given\n>>> fatal: More than one '-L n,m' option given\n>>>\n>>> From reading git-blame's man page it indeed seems to me as if only one\n>>> -L option is allowed, so is this something that needs to be fixed in\n>>> git-contacts?\n>>\n>> You are probably using a newer version of the script on an older version of Git.\n>\n> Right. git-blame learned to accept multiple -L's in 58dbfa2e59a1, and\n> git-contacts started taking advantage of that feature in 4c70cfbfbc2d.\n> A git-contacts prior to 4c70cfbfbc2d might work for you.\n\nAh, and 58dbfa2e59a1 is not in any release yet. Thanks.\n\n-- \nSebastian Schuberth\n"}]}