Edgewall Software

Ticket #629 (closed enhancement: fixed)

Opened 4 years ago

Last modified 5 months ago

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

  Changed 4 years ago by daniel

  • milestone set to 2.0

  Changed 4 years ago by cmlenz

#1215 has been marked as duplicate of this ticket.

  Changed 4 years ago by anonymous

  • cc zilvinas@… added

  Changed 3 years ago by cmlenz

  • owner changed from jonas to cmlenz
  • status changed from new to assigned

  Changed 3 years ago by cmlenz

  • milestone changed from 2.0 to 1.0

Experimental implementation of blame support is in source:branches/cmlenz-dev/blame.

  Changed 3 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.

  Changed 3 years ago by anonymous

#2160 has been marked as duplicate of this ticket.

  Changed 3 years ago by anonymous

Any news on when this is going to be integrated. It would be wonderful to have for the next milestone.

  Changed 3 years ago by joe@…

Sorry update! The last comment was mine, but I forgot to add my email address.

  Changed 3 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.

  Changed 3 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?

  Changed 3 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)

  Changed 3 years ago by anonymous

  • cc pjenvey@… added

  Changed 3 years ago by anonymous

  • cc maze@… added

  Changed 3 years ago by cboos

Another duplicate request for this: #3023

  Changed 3 years ago by anonymous

  • cc trac@… added

  Changed 2 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.

  Changed 2 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.

  Changed 2 years ago by anonymous

  • cc domonom@… added; dombot@… removed

  Changed 2 years ago by anonymous

  • cc gary.wilson@… added

  Changed 2 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 :)

  Changed 2 years ago by anonymous

  • cc mithrandir@… added

in reply to: ↑ description   Changed 2 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.

  Changed 2 years ago by cboos

#3023 has been marked as duplicate... again. In particular, check this requirement.

follow-up: ↓ 28   Changed 2 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?

  Changed 2 years ago by cboos

  • severity changed from normal to major

  Changed 2 years ago by cboos

  • keywords annotation added; svn removed
  • owner changed from cmlenz to cboos
  • status changed from assigned to new

in reply to: ↑ 25   Changed 2 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!

  Changed 23 months 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.

  Changed 5 months ago by elmarco

\o/ thanks

Add/Change #629 (svn blame support)

Author



Change Properties
<Author field>
Action
as closed
Next status will be 'reopened'
to The owner will change from cboos. Next status will be 'closed'
 
Note: See TracTickets for help on using tickets.