{"thread":{"id":"43138","subject":"Re: [PATCH] gitweb: Add \"next\" link to commitdiff view","startedAt":"2006-10-28T16:10:36Z","lastAt":"2006-10-29T02:10:29Z","messageCount":6,"participants":["Luben Tuikov","Jakub Narebski","Junio C Hamano"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"298208","messageId":"200610281810.36892.jnareb@gmail.com","threadId":"43138","inReplyTo":"200610230037.57183.jnareb@gmail.com","subject":"Re: [PATCH] gitweb: Add \"next\" link to commitdiff view","fromName":"Jakub Narebski","fromEmail":"jnareb@gmail.com","sentAt":"2006-10-28T16:10:36Z","receivedAt":"2006-10-28T16:10:36Z","isPatch":true,"sender":{"key":"jnareb@gmail.com","avatar":"https://avatars.githubusercontent.com/u/2706?v=4"},"body":"Jakub Narebski wrote:\n> Add a kind of \"next\" view in the bottom part of navigation bar for\n> \"commitdiff\" view.\n> \n> For commitdiff between two commits:\n>   (from: _commit_)\n> For commitdiff for one single parent commit:\n>   (parent: _commit_)\n> For commitdiff for one merge commit\n>   (merge: _commit_ _commit_ ...)\n> For commitdiff for root (parentless) commit\n>   (initial)\n> where _link_ denotes hyperlink. SHA1 is shortened to 7 characters on\n> display, everything is perhaps unnecessary esc_html on display.\n> \n> Signed-off-by: Jakub Narebski \n> ---\n> \n> Junio (and others), is it what you wanted? The idea was to change\n> \"commitdiff\" view only in minimal way, and preserve similarity\n> to \"commit\" format.\n\nAny reasons not to accept this patch? I find it very useful.\n\nThe idea to use fixed string instead of shortened SHA-1 of commit\nwas abandoned after some discussion in this thread.\n-- \nJakub Narebski\n"},{"id":"297880","messageId":"7vzmbg8dum.fsf@assigned-by-dhcp.cox.net","threadId":"43138","inReplyTo":"200610281810.36892.jnareb@gmail.com","subject":"Re: [PATCH] gitweb: Add \"next\" link to commitdiff view","fromName":"Junio C Hamano","fromEmail":"junkio@cox.net","sentAt":"2006-10-28T19:00:49Z","receivedAt":"2006-10-28T19:00:49Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Jakub Narebski writes:\n\n> Jakub Narebski wrote:\n>> Add a kind of \"next\" view in the bottom part of navigation bar for\n>> \"commitdiff\" view.\n>> \n>> For commitdiff between two commits:\n>>   (from: _commit_)\n>> For commitdiff for one single parent commit:\n>>   (parent: _commit_)\n>> For commitdiff for one merge commit\n>>   (merge: _commit_ _commit_ ...)\n>> For commitdiff for root (parentless) commit\n>>   (initial)\n>> where _link_ denotes hyperlink. SHA1 is shortened to 7 characters on\n>> display, everything is perhaps unnecessary esc_html on display.\n>> \n>> Signed-off-by: Jakub Narebski \n>> ---\n>> \n>> Junio (and others), is it what you wanted? The idea was to change\n>> \"commitdiff\" view only in minimal way, and preserve similarity\n>> to \"commit\" format.\n>\n> Any reasons not to accept this patch?\n\nNothing other than it was lost in the noise (I am partly\nresponsible for) that followed the patch.\n\n\tMessage-ID: <200610230037.57183.jnareb@gmail.com>\n\nWho acked this in the discussion? I'd like to add \"Acked-by:\"\nfor them when I make the commit.\n"},{"id":"293905","messageId":"196807.75961.qm@web31801.mail.mud.yahoo.com","threadId":"43138","inReplyTo":"200610281810.36892.jnareb@gmail.com","subject":"Re: [PATCH] gitweb: Add \"next\" link to commitdiff view","fromName":"Luben Tuikov","fromEmail":"ltuikov@yahoo.com","sentAt":"2006-10-29T01:50:26Z","receivedAt":"2006-10-29T01:50:26Z","isPatch":true,"sender":{"key":"ltuikov@yahoo.com","avatar":null},"body":"--- Jakub Narebski wrote:\n> Jakub Narebski wrote:\n> > Add a kind of \"next\" view in the bottom part of navigation bar for\n> > \"commitdiff\" view.\n> > \n> > For commitdiff between two commits:\n> > (from: _commit_)\n> > For commitdiff for one single parent commit:\n> > (parent: _commit_)\n> > For commitdiff for one merge commit\n> > (merge: _commit_ _commit_ ...)\n> > For commitdiff for root (parentless) commit\n> > (initial)\n> > where _link_ denotes hyperlink. SHA1 is shortened to 7 characters on\n> > display, everything is perhaps unnecessary esc_html on display.\n> > \n> > Signed-off-by: Jakub Narebski \n> > ---\n> > \n> > Junio (and others), is it what you wanted? The idea was to change\n> > \"commitdiff\" view only in minimal way, and preserve similarity\n> > to \"commit\" format.\n> \n> Any reasons not to accept this patch? I find it very useful.\n> \n> The idea to use fixed string instead of shortened SHA-1 of commit\n> was abandoned after some discussion in this thread.\n\nI prefer using the commit-8 without any \"...\" postfixed. Anyone who\nknows squat about git knows very well what a commit-8 is when they\nsee one -- the first 8 hexadecimal digits of the full SHA-1.\n\nI like using \"next\" only when there is a \"prev\" right next to it,\ni.e. based on _context_, something like this:\n << prev next >>\nwhere \"<< prev\" is hyperlinked, and \"next >>\" is also.\n\nI looked at your patch (first email in this thread) and from\nwhat I gathered that it does, I like it (haven't tried it yet).\n\nWhat I like about it is the clear information it conveys:\nit gives me both the commit-8 and what that commit-8 _is_.\n\nI sometimes remember commit-8s and this helps me a lot when\nI move the mouse pointer to an xterm and want to do\ngit-show , and it futhermore helps me _know_ the\nrelationship between that commit-8 and the rest of the\ninformation in the window frame gitweb is showing, (parent, etc).\n\nAcked-by: Luben Tuikov \n(not that it matters in any way ;-) )\n\n Luben\n\n\n"},{"id":"295380","messageId":"200610290258.31771.jnareb@gmail.com","threadId":"43138","inReplyTo":"196807.75961.qm@web31801.mail.mud.yahoo.com","subject":"Re: [PATCH] gitweb: Add \"next\" link to commitdiff view","fromName":"Jakub Narebski","fromEmail":"jnareb@gmail.com","sentAt":"2006-10-29T01:58:31Z","receivedAt":"2006-10-29T01:58:31Z","isPatch":true,"sender":{"key":"jnareb@gmail.com","avatar":"https://avatars.githubusercontent.com/u/2706?v=4"},"body":"Luben Tuikov wrote:\n> --- Jakub Narebski wrote:\n>> Jakub Narebski wrote:\n>>> Add a kind of \"next\" view in the bottom part of navigation bar for\n>>> \"commitdiff\" view.\n>>> \n>>> For commitdiff between two commits:\n>>> (from: _commit_)\n>>> For commitdiff for one single parent commit:\n>>> (parent: _commit_)\n>>> For commitdiff for one merge commit\n>>> (merge: _commit_ _commit_ ...)\n>>> For commitdiff for root (parentless) commit\n>>> (initial)\n>>> where _link_ denotes hyperlink. SHA1 is shortened to 7 characters on\n>>> display, everything is perhaps unnecessary esc_html on display.\n[...]\n>> \n>> Any reasons not to accept this patch? I find it very useful.\n>> \n>> The idea to use fixed string instead of shortened SHA-1 of commit\n>> was abandoned after some discussion in this thread.\n> \n> I prefer using the commit-8 without any \"...\" postfixed. Anyone who\n> knows squat about git knows very well what a commit-8 is when they\n> see one -- the first 8 hexadecimal digits of the full SHA-1.\n> \n> I like using \"next\" only when there is a \"prev\" right next to it,\n> i.e. based on _context_, something like this:\n> << prev next>>\n> where \"<< prev\" is hyperlinked, and \"next>>\" is also.\n\nUnfortunately this is simply not possible in this case, as links in git\nare only from commit to paren(a), in one direction only.\n\nBesides as you can see we can have more than one \"next>>\" link: for merge\ncommit there are multiple parents. Well, for branch points there are\nmultiple commits which have given commit as (one of) its parent(s), so\nthere can be multiple \"< I looked at your patch (first email in this thread) and from\n> what I gathered that it does, I like it (haven't tried it yet).\n> \n> What I like about it is the clear information it conveys:\n> it gives me both the commit-8 and what that commit-8 _is_.\n\nIt is commit-7, but that can be easily changed.\n\n> Acked-by: Luben Tuikov \n> (not that it matters in any way ;-) )\n\nIIRC Junio asked for ACKs.\n-- \nJakub Narebski\n"},{"id":"296723","messageId":"672246.46875.qm@web31808.mail.mud.yahoo.com","threadId":"43138","inReplyTo":"200610290258.31771.jnareb@gmail.com","subject":"Re: [PATCH] gitweb: Add \"next\" link to commitdiff view","fromName":"Luben Tuikov","fromEmail":"ltuikov@yahoo.com","sentAt":"2006-10-29T02:04:40Z","receivedAt":"2006-10-29T02:04:40Z","isPatch":true,"sender":{"key":"ltuikov@yahoo.com","avatar":null},"body":"--- Jakub Narebski wrote:\n> Luben Tuikov wrote:\n> > --- Jakub Narebski wrote:\n> >> Jakub Narebski wrote:\n> >>> Add a kind of \"next\" view in the bottom part of navigation bar for\n> >>> \"commitdiff\" view.\n> >>> \n> >>> For commitdiff between two commits:\n> >>> (from: _commit_)\n> >>> For commitdiff for one single parent commit:\n> >>> (parent: _commit_)\n> >>> For commitdiff for one merge commit\n> >>> (merge: _commit_ _commit_ ...)\n> >>> For commitdiff for root (parentless) commit\n> >>> (initial)\n> >>> where _link_ denotes hyperlink. SHA1 is shortened to 7 characters on\n> >>> display, everything is perhaps unnecessary esc_html on display.\n> [...]\n> >> \n> >> Any reasons not to accept this patch? I find it very useful.\n> >> \n> >> The idea to use fixed string instead of shortened SHA-1 of commit\n> >> was abandoned after some discussion in this thread.\n> > \n> > I prefer using the commit-8 without any \"...\" postfixed. Anyone who\n> > knows squat about git knows very well what a commit-8 is when they\n> > see one -- the first 8 hexadecimal digits of the full SHA-1.\n> > \n> > I like using \"next\" only when there is a \"prev\" right next to it,\n> > i.e. based on _context_, something like this:\n> > << prev next>>\n> > where \"<< prev\" is hyperlinked, and \"next>>\" is also.\n> \n> Unfortunately this is simply not possible in this case, as links in git\n> are only from commit to paren(a), in one direction only.\n\nI was not suggesting that. I simply saw a \"next\" suggestion in this\nthread and wanted to mention about \"next\", \"prev\" and context.\n\n> It is commit-7, but that can be easily changed.\n\nPlease do change it to \"commit-8\" -- it'd be consistent with the rest\nof GIT.\n\n> > Acked-by: Luben Tuikov \n> > (not that it matters in any way ;-) )\n> \n> IIRC Junio asked for ACKs.\n\nIn this email, it seems that he's asking for, quoting,\n \"Acked-by:\":\nhttp://marc.theaimsgroup.com/?l=git&m=116206206008374&w=2\n\n Luben\n"},{"id":"296942","messageId":"200610290310.30265.jnareb@gmail.com","threadId":"43138","inReplyTo":"672246.46875.qm@web31808.mail.mud.yahoo.com","subject":"Re: [PATCH] gitweb: Add \"next\" link to commitdiff view","fromName":"Jakub Narebski","fromEmail":"jnareb@gmail.com","sentAt":"2006-10-29T02:10:29Z","receivedAt":"2006-10-29T02:10:29Z","isPatch":true,"sender":{"key":"jnareb@gmail.com","avatar":"https://avatars.githubusercontent.com/u/2706?v=4"},"body":"Luben Tuikov wrote:\n> --- Jakub Narebski wrote:\n>>\n>> It is commit-7, but that can be easily changed.\n> \n> Please do change it to \"commit-8\" -- it'd be consistent with the rest\n> of GIT.\n\nWell, some parts of GIT use \"commit-8\" (for example git-describe),\nother use \"commit-7...\": this includes git-diff both in --raw\n(git diff --raw, as opposed to git-diff-tree), git-diff in it's patch \nformat in \"index ..\" extended header, in \"Merge:\" line for \nmerges in git-show output.\n\n-- \nJakub Narebski\n"}]}