{"thread":{"id":"17193","subject":"'mail' link on http://repo.or.cz/w/git.git no workee?","startedAt":"2009-01-15T15:53:48Z","lastAt":"2009-01-17T15:19:29Z","messageCount":12,"participants":["Johannes Schindelin","Petr Baudis","Junio C Hamano","Jacob Helwig","Jens Seidel"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"100621","messageId":"alpine.DEB.1.00.0901151651460.3586@pacific.mpi-cbg.de","threadId":"17193","inReplyTo":null,"subject":"'mail' link on http://repo.or.cz/w/git.git no workee?","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2009-01-15T15:53:48Z","receivedAt":"2009-01-15T15:53:48Z","isPatch":false,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi Pasky,\n\nsorry to bother you between two games of Go; Could you have a look at the \n'mail' link with the commit \"Update 1.6.2 draft release notes\"?  It is not \nworking for me...\n\nThat is, it links to marc (not gmane?) but finds no matches...\n\nCiao,\nDscho\n"},{"id":"100683","messageId":"20090116015439.GF12275@machine.or.cz","threadId":"17193","inReplyTo":"alpine.DEB.1.00.0901151651460.3586@pacific.mpi-cbg.de","subject":"Re: 'mail' link on http://repo.or.cz/w/git.git no workee?","fromName":"Petr Baudis","fromEmail":"pasky@suse.cz","sentAt":"2009-01-16T01:54:39Z","receivedAt":"2009-01-16T01:54:39Z","isPatch":false,"sender":{"key":"pasky@ucw.cz","avatar":"https://avatars.githubusercontent.com/u/18439?v=4"},"body":"  Hi,\n\nOn Thu, Jan 15, 2009 at 04:53:48PM +0100, Johannes Schindelin wrote:\n> sorry to bother you between two games of Go; Could you have a look at the \n> 'mail' link with the commit \"Update 1.6.2 draft release notes\"?  It is not \n> working for me...\n> \n> That is, it links to marc (not gmane?) but finds no matches...\n\n  what mailing list post should it point to?\n\n\t\t\t\tPetr \"Pasky\" Baudis\n"},{"id":"100684","messageId":"7vmydsovs5.fsf@gitster.siamese.dyndns.org","threadId":"17193","inReplyTo":"20090116015439.GF12275@machine.or.cz","subject":"Re: 'mail' link on http://repo.or.cz/w/git.git no workee?","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2009-01-16T01:59:54Z","receivedAt":"2009-01-16T01:59:54Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Petr Baudis <pasky@suse.cz> writes:\n\n>> That is, it links to marc (not gmane?) but finds no matches...\n>\n>   what mailing list post should it point to?\n\nI think Dscho's point is that you shouldn't be generating these links that\nlead to nowhere ;-).\n\nIt would really be nice if we can maintain an exterenal database that\nlinks mailing list discussion threads and individual commits, and have\ngitweb and other visualization tools to make use of that information.\n\nI do not care what storage mechanism that external database uses.  It\ncould use notes or it could just be a BDB that can be indexed with commit\nobject names.\n"},{"id":"100687","messageId":"alpine.DEB.1.00.0901160315170.3586@pacific.mpi-cbg.de","threadId":"17193","inReplyTo":"7vmydsovs5.fsf@gitster.siamese.dyndns.org","subject":"Re: 'mail' link on http://repo.or.cz/w/git.git no workee?","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2009-01-16T02:17:14Z","receivedAt":"2009-01-16T02:17:14Z","isPatch":false,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi,\n\nOn Thu, 15 Jan 2009, Junio C Hamano wrote:\n\n> Petr Baudis <pasky@suse.cz> writes:\n> \n> >> That is, it links to marc (not gmane?) but finds no matches...\n> >\n> >   what mailing list post should it point to?\n\nI found gmane to be much nicer to click through than marc.\n\n> I think Dscho's point is that you shouldn't be generating these links \n> that lead to nowhere ;-).\n\nAlmost.\n\nI wondered what the point was, and I would _welcome_ links that pointed to \nthe mails containing the patches (identified either by patch ids or by \nmanual editing; best would be to have both methods at the same time).\n\nCiao,\nDscho\n"},{"id":"100688","messageId":"20090116022408.GG12275@machine.or.cz","threadId":"17193","inReplyTo":"alpine.DEB.1.00.0901160315170.3586@pacific.mpi-cbg.de","subject":"Re: 'mail' link on http://repo.or.cz/w/git.git no workee?","fromName":"Petr Baudis","fromEmail":"pasky@suse.cz","sentAt":"2009-01-16T02:24:08Z","receivedAt":"2009-01-16T02:24:08Z","isPatch":false,"sender":{"key":"pasky@ucw.cz","avatar":"https://avatars.githubusercontent.com/u/18439?v=4"},"body":"On Fri, Jan 16, 2009 at 03:17:14AM +0100, Johannes Schindelin wrote:\n> On Thu, 15 Jan 2009, Junio C Hamano wrote:\n> \n> > Petr Baudis <pasky@suse.cz> writes:\n> > \n> > >> That is, it links to marc (not gmane?) but finds no matches...\n> > >\n> > >   what mailing list post should it point to?\n> \n> I found gmane to be much nicer to click through than marc.\n\nIIRC I found no way to actually make a gmane link since it requires POST\nquery for the searches or something.\n\n> > I think Dscho's point is that you shouldn't be generating these links \n> > that lead to nowhere ;-).\n> \n> Almost.\n> \n> I wondered what the point was, and I would _welcome_ links that pointed to \n> the mails containing the patches (identified either by patch ids or by \n> manual editing; best would be to have both methods at the same time).\n\nWell, but you already have that, or at least an approximation. Do you\nthink the approximation is inadequate? I would expect linking to a\ncommit showing the inadequacy - I don't see how a false positive is\ninadequate at all, actually; even if we had a separate database with the\nmapping, for no records I think falling back to linking to a mailing\nlist search still makes more sense than showing no link.\n\nI was not actually planning to carry the patch over the next gitweb\nupdate since I got no feedback on it (IIRC) since I implemented it.\n\n-- \n\t\t\t\tPetr \"Pasky\" Baudis\nThe average, healthy, well-adjusted adult gets up at seven-thirty\nin the morning feeling just terrible. -- Jean Kerr\n"},{"id":"100689","messageId":"8c9a060901151832l71464185qf9f88afede6f550d@mail.gmail.com","threadId":"17193","inReplyTo":"20090116022408.GG12275@machine.or.cz","subject":"Re: 'mail' link on http://repo.or.cz/w/git.git no workee?","fromName":"Jacob Helwig","fromEmail":"jacob.helwig@gmail.com","sentAt":"2009-01-16T02:32:56Z","receivedAt":"2009-01-16T02:32:56Z","isPatch":false,"sender":{"key":"jacob.helwig@gmail.com","avatar":"https://avatars.githubusercontent.com/u/14557?v=4"},"body":"On Thu, Jan 15, 2009 at 18:24, Petr Baudis <pasky@suse.cz> wrote:\n>\n> IIRC I found no way to actually make a gmane link since it requires POST\n> query for the searches or something.\n>\n\n$ cat bin/gmane-links.pl\n#!/usr/bin/perl\n\nuse strict;\nuse warnings;\n\nuse Mail::Summary::Tools::ArchiveLink::Gmane;\n\nunless ($ARGV[0]) {\n    print STDERR \"Must provide a message-id.\\n\";\n    exit 1;\n}\n\nmy $link = Mail::Summary::Tools::ArchiveLink::Gmane->new(message_id =>\n$ARGV[0]);\n\nprint \"Thread: \" . $link->thread_uri() . \"\\n\";\nprint \"Message: \" . $link->message_uri() . \"\\n\";\n"},{"id":"100691","messageId":"7vd4eootvj.fsf@gitster.siamese.dyndns.org","threadId":"17193","inReplyTo":"8c9a060901151832l71464185qf9f88afede6f550d@mail.gmail.com","subject":"Re: 'mail' link on http://repo.or.cz/w/git.git no workee?","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2009-01-16T02:41:04Z","receivedAt":"2009-01-16T02:41:04Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"\"Jacob Helwig\" <jacob.helwig@gmail.com> writes:\n\n> On Thu, Jan 15, 2009 at 18:24, Petr Baudis <pasky@suse.cz> wrote:\n>>\n>> IIRC I found no way to actually make a gmane link since it requires POST\n>> query for the searches or something.\n>>\n>\n> $ cat bin/gmane-links.pl\n> #!/usr/bin/perl\n>\n> use strict;\n> use warnings;\n>\n> use Mail::Summary::Tools::ArchiveLink::Gmane;\n>\n> unless ($ARGV[0]) {\n>     print STDERR \"Must provide a message-id.\\n\";\n>     exit 1;\n> }\n>\n> my $link = Mail::Summary::Tools::ArchiveLink::Gmane->new(message_id =>\n> $ARGV[0]);\n>\n> print \"Thread: \" . $link->thread_uri() . \"\\n\";\n> print \"Message: \" . $link->message_uri() . \"\\n\";\n\nNot quite, if message_id means what I think it means.\n\nPasky searches for the title of the message, and he does not have the\nMessage-ID header of the original message.\n"},{"id":"100692","messageId":"7v8wpcots2.fsf@gitster.siamese.dyndns.org","threadId":"17193","inReplyTo":"20090116022408.GG12275@machine.or.cz","subject":"Re: 'mail' link on http://repo.or.cz/w/git.git no workee?","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2009-01-16T02:43:09Z","receivedAt":"2009-01-16T02:43:09Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Petr Baudis <pasky@suse.cz> writes:\n\n> I was not actually planning to carry the patch over the next gitweb\n> update since I got no feedback on it (IIRC) since I implemented it.\n\nHeh, now you got it.\n\nAnd I do not think you advertised it since you implemented it, so no\nfeedback does not necessarily mean no interest.  FWIW, I think looking for\na message that has similar words in the mail archive is a nifty feature to\nhave.\n"},{"id":"100708","messageId":"20090116090301.GA13300@merkur.sol.de","threadId":"17193","inReplyTo":"20090116015439.GF12275@machine.or.cz","subject":"Re: 'mail' link on http://repo.or.cz/w/git.git no workee?","fromName":"Jens Seidel","fromEmail":"jensseidel@users.sf.net","sentAt":"2009-01-16T09:03:03Z","receivedAt":"2009-01-16T09:03:03Z","isPatch":false,"sender":{"key":"jensseidel@users.sf.net","avatar":null},"body":"On Fri, Jan 16, 2009 at 02:54:39AM +0100, Petr Baudis wrote:\n> On Thu, Jan 15, 2009 at 04:53:48PM +0100, Johannes Schindelin wrote:\n> > sorry to bother you between two games of Go; Could you have a look at the \n> > 'mail' link with the commit \"Update 1.6.2 draft release notes\"?  It is not \n> > working for me...\n> > \n> > That is, it links to marc (not gmane?) but finds no matches...\n\nJust a side note: The link to the mailing list on http://git-scm.com/ is also\nnot optimal as it refers to gmane. I tried to subscribe but found only a link\nto http://dir.gmane.org/gmane.comp.version-control.git and no instructions how\nto subscribe ...\n\nJens\n"},{"id":"100717","messageId":"alpine.DEB.1.00.0901161224360.3586@pacific.mpi-cbg.de","threadId":"17193","inReplyTo":"20090116022408.GG12275@machine.or.cz","subject":"Re: 'mail' link on http://repo.or.cz/w/git.git no workee?","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2009-01-16T11:27:47Z","receivedAt":"2009-01-16T11:27:47Z","isPatch":false,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi,\n\nOn Fri, 16 Jan 2009, Petr Baudis wrote:\n\n> On Fri, Jan 16, 2009 at 03:17:14AM +0100, Johannes Schindelin wrote:\n> \n> > I wondered what the point was, and I would _welcome_ links that \n> > pointed to the mails containing the patches (identified either by \n> > patch ids or by manual editing; best would be to have both methods at \n> > the same time).\n> \n> Well, but you already have that, or at least an approximation.\n\nOh, I see!  Somehow I missed that this link exists for all non-merge \ncommits...\n\n> I was not actually planning to carry the patch over the next gitweb \n> update since I got no feedback on it (IIRC) since I implemented it.\n\nWell, now you got one: I find it nice.  And I agree with you on the false \npositive thing: it is better to have that link also for commits for which \nno mails can be found.\n\nI just had to scratch my head about what \"mail\" is about...\n\nCiao,\nDscho\n"},{"id":"100718","messageId":"alpine.DEB.1.00.0901161235380.3586@pacific.mpi-cbg.de","threadId":"17193","inReplyTo":"20090116022408.GG12275@machine.or.cz","subject":"Re: 'mail' link on http://repo.or.cz/w/git.git no workee?","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2009-01-16T11:37:03Z","receivedAt":"2009-01-16T11:37:03Z","isPatch":false,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi,\n\nOn Fri, 16 Jan 2009, Petr Baudis wrote:\n\n> On Fri, Jan 16, 2009 at 03:17:14AM +0100, Johannes Schindelin wrote:\n> > On Thu, 15 Jan 2009, Junio C Hamano wrote:\n> > \n> > > Petr Baudis <pasky@suse.cz> writes:\n> > > \n> > > >> That is, it links to marc (not gmane?) but finds no matches...\n> > > >\n> > > >   what mailing list post should it point to?\n> > \n> > I found gmane to be much nicer to click through than marc.\n> \n> IIRC I found no way to actually make a gmane link since it requires POST\n> query for the searches or something.\n\nThe site seems to be down ATM, but it should be\n\nhttp://search.gmane.org/search.php?group=gmane.comp.version-control.git&query=pasky\n\nBTW how about calling the link \"find mail\"?  When I saw \"mail\" first time, \nI thought \"oh, so I can mail that commitdiff to somebody?  Nice, let's \ntry.\"\n\nCiao,\nDscho\n"},{"id":"100834","messageId":"alpine.DEB.1.00.0901171619060.3586@pacific.mpi-cbg.de","threadId":"17193","inReplyTo":"alpine.DEB.1.00.0901161235380.3586@pacific.mpi-cbg.de","subject":"Re: 'mail' link on http://repo.or.cz/w/git.git no workee?","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2009-01-17T15:19:29Z","receivedAt":"2009-01-17T15:19:29Z","isPatch":false,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi,\n\nOn Fri, 16 Jan 2009, Johannes Schindelin wrote:\n\n> The site seems to be down ATM, but it should be\n> \n> http://search.gmane.org/search.php?group=gmane.comp.version-control.git&query=pasky\n\nFWIW, it is up again.\n\nCiao,\nDscho\n"}]}