{"thread":{"id":"15789","subject":"gitweb improvements","startedAt":"2008-10-05T15:45:01Z","lastAt":"2008-10-06T22:17:00Z","messageCount":5,"participants":["Tjernlund","Jakub Narebski"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"92336","messageId":"00a201c92701$54ec6980$fec53c80$@se","threadId":"15789","inReplyTo":null,"subject":"gitweb improvements","fromName":"Tjernlund","fromEmail":"tjernlund@tjernlund.se","sentAt":"2008-10-05T15:45:01Z","receivedAt":"2008-10-05T15:45:01Z","isPatch":false,"sender":{"key":"tjernlund@tjernlund.se","avatar":null},"body":"When I browse a repo using gitweb, I miss two things:\n1) looking at the history for a file or directory I really want\n to see the tags there too.\n\n2) looking at a merge like:\nhttp://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=66120005e65eed8a05b14a36ab448bdec42f0d6b\n is somewhat confusing. It really doesn't tell you which commits that is\n included in the merge.\n\n Jocke\n"},{"id":"92345","messageId":"m34p3rq7og.fsf@localhost.localdomain","threadId":"15789","inReplyTo":"00a201c92701$54ec6980$fec53c80$@se","subject":"Re: gitweb improvements","fromName":"Jakub Narebski","fromEmail":"jnareb@gmail.com","sentAt":"2008-10-05T17:22:18Z","receivedAt":"2008-10-05T17:22:18Z","isPatch":false,"sender":{"key":"jnareb@gmail.com","avatar":"https://avatars.githubusercontent.com/u/2706?v=4"},"body":"\"Tjernlund\" writes:\n\n> When I browse a repo using gitweb, I miss two things:\n> 1) looking at the history for a file or directory I really want\n> to see the tags there too.\n\nShould be easy to do, but would probably wait (and be indirect result)\nof refactoring/unification of log-like views code ('log', 'shortlog',\n'history', perhaps also 'rss', 'atom', 'search')\n\nI have added it to my gitweb TODO list...\n \n> 2) looking at a merge like:\n> http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=66120005e65eed8a05b14a36ab448bdec42f0d6b\n> is somewhat confusing. It really doesn't tell you which commits that is\n> included in the merge.\n\nI don't understand you there. First, you have \"(merge: 0d0f3ef 9778e9a)\"\nin the navbar, so you can easily go to commit view for parents. Second, \namong commit headers you have two \"parent\", where SHA-1 of a commit is\nhidden link, and there are also 'commit' and 'diff' link for those.\n\n-- \nJakub Narebski\nPoland\nShadeHawk on #git\n"},{"id":"92370","messageId":"00b201c92739$36585eb0$a3091c10$@se","threadId":"15789","inReplyTo":"m34p3rq7og.fsf@localhost.localdomain","subject":"RE: gitweb improvements","fromName":"Tjernlund","fromEmail":"tjernlund@tjernlund.se","sentAt":"2008-10-05T22:25:03Z","receivedAt":"2008-10-05T22:25:03Z","isPatch":false,"sender":{"key":"tjernlund@tjernlund.se","avatar":null},"body":"\n\n> -----Original Message-----\n> From: Jakub Narebski [mailto:jnareb@gmail.com]\n> Sent: den 5 oktober 2008 19:22\n> To: Tjernlund\n> Cc: 'git'\n> Subject: Re: gitweb improvements\n> \n> \"Tjernlund\" writes:\n> \n> > When I browse a repo using gitweb, I miss two things:\n> > 1) looking at the history for a file or directory I really want\n> > to see the tags there too.\n> \n> Should be easy to do, but would probably wait (and be indirect result)\n> of refactoring/unification of log-like views code ('log', 'shortlog',\n> 'history', perhaps also 'rss', 'atom', 'search')\n> \n> I have added it to my gitweb TODO list...\n\nGreat! Thanks.\n\n> \n> > 2) looking at a merge like:\n> > http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-\n> 2.6.git;a=commit;h=66120005e65eed8a05b14a36ab448bdec42f0d6b\n> > is somewhat confusing. It really doesn't tell you which commits that is\n> > included in the merge.\n> \n> I don't understand you there. First, you have \"(merge: 0d0f3ef 9778e9a)\"\n> in the navbar, so you can easily go to commit view for parents. Second,\n> among commit headers you have two \"parent\", where SHA-1 of a commit is\n> hidden link, and there are also 'commit' and 'diff' link for those.\n\nhmm, looks like I overlooked \"(merge: 0d0f3ef 9778e9a)\" part. However, I can't\nfind the \"ALSA: make the CS4270 driver a new-style I2C driver\" from within\nthis page.\n\n> \n> --\n> Jakub Narebski\n> Poland\n> ShadeHawk on #git\n"},{"id":"92378","messageId":"200810060154.06934.jnareb@gmail.com","threadId":"15789","inReplyTo":"00b201c92739$36585eb0$a3091c10$@se","subject":"Re: gitweb improvements","fromName":"Jakub Narebski","fromEmail":"jnareb@gmail.com","sentAt":"2008-10-05T23:54:06Z","receivedAt":"2008-10-05T23:54:06Z","isPatch":false,"sender":{"key":"jnareb@gmail.com","avatar":"https://avatars.githubusercontent.com/u/2706?v=4"},"body":"On Mon, 6 Oct 2008, Tjernlund wrote:\n> Jakub Narebski wrote:\n>> \"Tjernlund\" writes:\n\n>>> 2) looking at a merge like:\n>>> http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=66120005e65eed8a05b14a36ab448bdec42f0d6b\n>>> is somewhat confusing. It really doesn't tell you which commits that is\n>>> included in the merge.\n>> \n>> I don't understand you there. First, you have \"(merge: 0d0f3ef 9778e9a)\"\n>> in the navbar, so you can easily go to commit view for parents. Second,\n>> among commit headers you have two \"parent\", where SHA-1 of a commit is\n>> hidden link, and there are also 'commit' and 'diff' link for those.\n> \n> hmm, looks like I overlooked \"(merge: 0d0f3ef 9778e9a)\" part. However, I can't\n> find the \"ALSA: make the CS4270 driver a new-style I2C driver\" from within\n> this page.\n\nI think you don't quite understand the situation. The history looks\nlike this:\n\n M Merge branch 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound-2.6\n |\\\n | 2 ALSA: ASoC: Fix another cs4270 error path\n | * ALSA: make the CS4270 driver a new-style I2C driver\n | |\n 1 | Merge git://git.kernel.org/pub/scm/linux/kernel/git/jejb/scsi-rc-fixes-2.6 \n\nParents of commit 'M' (for merge) are '1' and '2', not 2,* or 1,2,*.\n\nNow the fact that commit message for merge contains shortlog of merged\nbranch does not mean that there must be direct link to such shortlog.\nYou can go to shortlog (well, kind of) if you click on second parent,\n_then_ click on shortlog link at top of the page.\n\n-- \nJakub Narebski\nPoland\n"},{"id":"92459","messageId":"013b01c92801$40f5f8f0$c2e1ead0$@se","threadId":"15789","inReplyTo":"200810060154.06934.jnareb@gmail.com","subject":"RE: gitweb improvements","fromName":"Tjernlund","fromEmail":"tjernlund@tjernlund.se","sentAt":"2008-10-06T22:17:00Z","receivedAt":"2008-10-06T22:17:00Z","isPatch":false,"sender":{"key":"tjernlund@tjernlund.se","avatar":null},"body":"> -----Original Message-----\n> From: Jakub Narebski [mailto:jnareb@gmail.com]\n> Sent: den 6 oktober 2008 01:54\n> To: Tjernlund\n> Cc: 'git'\n> Subject: Re: gitweb improvements\n> \n> On Mon, 6 Oct 2008, Tjernlund wrote:\n> > Jakub Narebski wrote:\n> >> \"Tjernlund\" writes:\n> \n> >>> 2) looking at a merge like:\n> >>> http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-\n> 2.6.git;a=commit;h=66120005e65eed8a05b14a36ab448bdec42f0d6b\n> >>> is somewhat confusing. It really doesn't tell you which commits that is\n> >>> included in the merge.\n> >>\n> >> I don't understand you there. First, you have \"(merge: 0d0f3ef 9778e9a)\"\n> >> in the navbar, so you can easily go to commit view for parents. Second,\n> >> among commit headers you have two \"parent\", where SHA-1 of a commit is\n> >> hidden link, and there are also 'commit' and 'diff' link for those.\n> >\n> > hmm, looks like I overlooked \"(merge: 0d0f3ef 9778e9a)\" part. However, I can't\n> > find the \"ALSA: make the CS4270 driver a new-style I2C driver\" from within\n> > this page.\n> \n> I think you don't quite understand the situation. The history looks\n> like this:\n> \n> M Merge branch 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound-2.6\n> |\\\n> | 2 ALSA: ASoC: Fix another cs4270 error path\n> | * ALSA: make the CS4270 driver a new-style I2C driver\n> | |\n> 1 | Merge git://git.kernel.org/pub/scm/linux/kernel/git/jejb/scsi-rc-fixes-2.6\n> \n> Parents of commit 'M' (for merge) are '1' and '2', not 2,* or 1,2,*.\n> \n> Now the fact that commit message for merge contains shortlog of merged\n> branch does not mean that there must be direct link to such shortlog.\n> You can go to shortlog (well, kind of) if you click on second parent,\n> _then_ click on shortlog link at top of the page.\n\nYou are quite right, I didn't quite follow the situation. I do however have one\nobservation, would it be possible to list the commits in a merge the same way\nyou list commits on the top level, that is, more or less make the headings\n \"ALSA: ASoC: Fix another cs4270 error path\"\n \"ALSA: make the CS4270 driver a new-style I2C driver\"\nlinks one can follow?\n\n Jocke\n"}]}