Ticket #629 (closed enhancement: fixed)
svn blame support
| Reported by: | anonymous | Owned by: | cboos |
|---|---|---|---|
| Priority: | normal | Milestone: | 0.11 |
| Component: | version control/browser | Version: | 0.7.1 |
| Severity: | major | Keywords: | annotation blame |
| Cc: | zilvinas@…, pjenvey@…, maze@…, trac@…, domonom@…, gary.wilson@…, tapted@…, mithrandir@…, alec.munro@… |
Description
Having svn blame (aka annotate) in the source browser would be real nice.
Thanks for nice job so far, guys.
Attachments
Change History
comment:4 Changed 5 years ago by cmlenz
- Owner changed from jonas to cmlenz
- Status changed from new to assigned
comment:5 Changed 5 years ago by cmlenz
- Milestone changed from 2.0 to 1.0
Experimental implementation of blame support is in source:branches/cmlenz-dev/blame.
comment:6 Changed 5 years ago by Russ Brown
For extra brownie points on this, see if there is an svk:merge property somewhere and attempt to resolve merges to find the true authors of the lines merged.
That would be *very* smart.
comment:8 Changed 5 years ago by anonymous
Any news on when this is going to be integrated. It would be wonderful to have for the next milestone.
comment:9 Changed 5 years ago by joe@…
Sorry update! The last comment was mine, but I forgot to add my email address.
comment:10 Changed 5 years ago by cboos
- Milestone changed from 1.0 to 0.11
It's currently scheduled for milestone:0.11
cmlenz: can you move your branch to the sandbox, as blame support is also something I'd like to contribute to.
comment:11 Changed 5 years ago by sven.reimers@…
I started hacking away at a plugin proving a IHTMLPreviewAnnotator to show blame information along the lines cmlenz provided in his branch. But still there is a problem to access the req object during annotating, just line number and content is passed, additionally the only lineno is requested from inside trac. Anybody any idea if at least the api will be adopted (should not be that complex, since there seems to be only one annotator) - then it should impose no further problem developing this as a plugin (maybe some *.css, etc.). So anybody?
comment:12 Changed 5 years ago by sven.reimers@…
Forgot one other problem. The user should be able to select which annotaters are used, so some gui changes would be necessary here as well, but I think cmlenz has already provided some interesting things in his branch as well. Should this be tracked in a different ticket? Maybe this can be fixed in 0.10 time frame (allowing for plugin)
comment:15 Changed 4 years ago by cboos
Another duplicate request for this: #3023
comment:17 Changed 4 years ago by anonymous
Any news on this? Quite strange for Trac not to have the blame (I prefer the term "praise") feature since cvsweb et al have it.
comment:18 Changed 4 years ago by dombot@…
- Cc dombot@… added
This would be a very handy feature and would be a good potential advantage to convince the dev team from moving from CVS + Bugzilla + Bonsai to Subversion + Trac.
comment:21 Changed 4 years ago by Trent Apted <tapted@…>
- Cc tapted@… added
I am marking a groupwork software project/assignment currently (and have been for about 12 hours straight now). Students must use Trac/SVN and we generate some interesting visualizations from Trac and SVN logs/history
there are technical reports if people are interested: Wattle Tree: What’ll it Tell Us? and [longer] The Big Five and Visualisations of Team Work Activity (the "Big Five" are components of teamwork, according to Salas et al. in the Journal of Small Group Research 36(5), 2005).
However, sometimes you really want the details from an SVN blame.. so.. my fingers are crossed, in the hope 0.11 makes it out before August in time for the next semester :)
comment:23 in reply to: ↑ description Changed 4 years ago by alec.munro@…
- Cc alec.munro@… added
What needs to be done to get this working? I would really like to see more progress on 0.11, and I think I could probably help out with this particulur issue.
comment:24 Changed 4 years ago by cboos
#3023 has been marked as duplicate... again. In particular, check this requirement.
comment:25 follow-up: ↓ 28 Changed 4 years ago by garyo@…
Not a requirement, but a "would be nice" feature for blame: in Emacs, the blame ("annotate") mode colors source lines by their age. Old lines are blue, new are bright red, and so on. This way the recent changes stand out nicely. I would suggest doing this only for the rev numbers or dates columns in the blame output in trac, since the source lines already have their own syntax colorization. I think a default color scheme of 6 months ago gets blue, today gets red, and linearly in between, would be fine. Not sure arbitrary colors like this are easy with CSS/html though?
comment:26 Changed 4 years ago by cboos
- Severity changed from normal to major
comment:27 Changed 4 years ago by cboos
- Keywords annotation added; svn removed
- Owner changed from cmlenz to cboos
- Status changed from assigned to new
See sandbox/blame.
comment:28 in reply to: ↑ 25 Changed 4 years ago by cboos
Replying to garyo@genarts.com:
Not a requirement, but a "would be nice" feature for blame: in Emacs, the blame ("annotate") mode colors source lines by their age. Old lines are blue, new are bright red, and so on. This way the recent changes stand out nicely. I would suggest doing this only for the rev numbers or dates columns in the blame output in trac, since the source lines already have their own syntax colorization.
Done so in r4484, looks great, thanks for the suggestion!
comment:29 Changed 4 years ago by cboos
- Status changed from new to closed
- Resolution set to fixed
Blame branch merged in r4561.
Issues with the blame feature should be opened as new defect tickets. Feedback and improvements on the feature are welcomed of course, but the base feature is there and seems to be solid.
Note that the TracMercurial plugin (the 0.11 variant, i.e. source:sandbox/mercurial-plugin-0.11) also supports the new blame feature.
comment:30 Changed 2 years ago by elmarco
\o/ thanks
