{"thread":{"id":"24957","subject":"[RFC PATCH] gitweb.perl: tags snapshot link","startedAt":"2010-09-03T11:42:16Z","lastAt":"2010-09-14T18:06:17Z","messageCount":3,"participants":["WOLfgang Schricker","Giuseppe Bilotta"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"149760","messageId":"i5qmuo$ice$1@dough.gmane.org","threadId":"24957","inReplyTo":null,"subject":"[RFC PATCH] gitweb.perl: tags snapshot link","fromName":"WOLfgang Schricker","fromEmail":"news@wols.org","sentAt":"2010-09-03T11:42:16Z","receivedAt":"2010-09-03T11:42:16Z","isPatch":true,"sender":{"key":"news@wols.org","avatar":null},"body":"Hello,\n\nI missed a snapshot link in my local repo:\n====\ntags\n====\n\ttag | commit | shortlog | log | *snapshot*\n\nI found this threads\n\n\n\n\nand the code for \"Smarter snapshot names\" in 'gitwep.perl'.\n\nMy gitweb.perl-1.7.2.2.patch for \"activating\":\n\n\n--- gitweb.perl.orig\t2010-08-20 01:44:47.000000000 +0200\n+++ gitweb.perl\t2010-09-03 13:20:40.000000000 +0200\n@@ -4909,6 +4909,10 @@\n \t\t} elsif ($tag{'reftype'} eq \"blob\") {\n \t\t\tprint \" | \" . $cgi->a({-href => href(action=>\"blob_plain\", hash=>$tag{'refid'})}, \"raw\");\n \t\t}\n+ my $snapshot_links = format_snapshot_links($tag{'fullname'});\n+ if (defined $snapshot_links) {\n+ print \" | \" . $snapshot_links;\n+ }\n \t\tprint \"\\n\" .\n \t\t \"\";\n \t}\n\n-- \nRegards\n*WOL* fgang *S* chricker\n"},{"id":"150567","messageId":"i6jnhg$epf$1@dough.gmane.org","threadId":"24957","inReplyTo":"i5qmuo$ice$1@dough.gmane.org","subject":"Re: [RFC PATCH] gitweb.perl: tags snapshot link","fromName":"Giuseppe Bilotta","fromEmail":"giuseppe.bilotta@gmail.com","sentAt":"2010-09-12T23:25:30Z","receivedAt":"2010-09-12T23:25:30Z","isPatch":true,"sender":{"key":"giuseppe.bilotta@gmail.com","avatar":"https://avatars.githubusercontent.com/u/1464?v=4"},"body":"WOLfgang Schricker wrote:\n\n> Hello,\n> \n> I missed a snapshot link in my local repo:\n> ====\n> tags\n> ====\n> tag | commit | shortlog | log | *snapshot*\n> \n> I found this threads\n> \n> \n> \n> \n> and the code for \"Smarter snapshot names\" in 'gitwep.perl'.\n> \n> My gitweb.perl-1.7.2.2.patch for \"activating\":\n> \n> \n> --- gitweb.perl.orig\t2010-08-20 01:44:47.000000000 +0200\n> +++ gitweb.perl\t2010-09-03 13:20:40.000000000 +0200\n> @@ -4909,6 +4909,10 @@\n> } elsif ($tag{'reftype'} eq \"blob\") {\n> print \" | \" . $cgi->a({-href => href(action=>\"blob_plain\",\n> hash=>$tag{'refid'})}, \"raw\"); }\n> + my $snapshot_links = format_snapshot_links($tag{'fullname'});\n> + if (defined $snapshot_links) {\n> + print \" | \" . $snapshot_links;\n> + }\n> print \"\\n\" .\n> \"\";\n> }\n\nIf I read the patch correctly, this would add a 'snapshot' link to the tag\nlist view. Although the snapshot can already be obtained by clicking first\non the commit or shortlog view, this would reduce the number of clicks\nnecessary to obtain the snapshot. I believe it would be a good idea to have\nthis. Wolfgang: wold you consider submitting it in a more appropriate form\nfor inclusion?\n\nIf you do, you should probably also consider other places where such a link\ncould be added, such as in tag view, where the nav bar currently reads\n\nsummary | shortlog | log | commit | commitdiff | tree\n\nand \"| snapshot\" could be appended.\n\n-- \nGiuseppe Bilotta\n"},{"id":"150702","messageId":"i6odiq$ire$1@dough.gmane.org","threadId":"24957","inReplyTo":"i6jnhg$epf$1@dough.gmane.org","subject":"Re: [RFC PATCH] gitweb.perl: tags snapshot link","fromName":"WOLfgang Schricker","fromEmail":"news@wols.org","sentAt":"2010-09-14T18:06:17Z","receivedAt":"2010-09-14T18:06:17Z","isPatch":true,"sender":{"key":"news@wols.org","avatar":null},"body":"Giuseppe Bilotta wrote:\n> WOLfgang Schricker wrote:\n> \n>> Hello,\n>>\n>> I missed a snapshot link in my local repo:\n>> ====\n>> tags\n>> ====\n>> tag | commit | shortlog | log | *snapshot*\n>>\n>> I found this threads\n>>\n>> \n>> \n>>\n>> and the code for \"Smarter snapshot names\" in 'gitwep.perl'.\n>>\n>> My gitweb.perl-1.7.2.2.patch for \"activating\":\n>>\n>>\n>> --- gitweb.perl.orig\t2010-08-20 01:44:47.000000000 +0200\n>> +++ gitweb.perl\t2010-09-03 13:20:40.000000000 +0200\n>> @@ -4909,6 +4909,10 @@\n>> } elsif ($tag{'reftype'} eq \"blob\") {\n>> print \" | \" . $cgi->a({-href => href(action=>\"blob_plain\",\n>> hash=>$tag{'refid'})}, \"raw\"); }\n>> + my $snapshot_links = format_snapshot_links($tag{'fullname'});\n>> + if (defined $snapshot_links) {\n>> + print \" | \" . $snapshot_links;\n>> + }\n>> print \"\\n\" .\n>> \"\";\n>> }\n> \n> If I read the patch correctly, this would add a 'snapshot' link to the tag\n> list view. Although the snapshot can already be obtained by clicking first\n> on the commit or shortlog view, this would reduce the number of clicks\n> necessary to obtain the snapshot. I believe it would be a good idea to have\n> this. Wolfgang: wold you consider submitting it in a more appropriate form\n> for inclusion?\n> \n> If you do, you should probably also consider other places where such a link\n> could be added, such as in tag view, where the nav bar currently reads\n> \n> summary | shortlog | log | commit | commitdiff | tree\n> \n> and \"| snapshot\" could be appended.\n>\n\nYes. _Every_ available snapshot link looks -.tgz.\nA 'v0.1' tag brings in the tag list project-v0.1.tgz now.\nI see eg. a relocatable 'latest' tag for project-latest.tgz.\n-- \nRegards\n*WOL* fgang *S* chricker\n"}]}