Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
If it allows multiple authors.
http://code.google.com/p/gource/
Comment | File | Size | Author |
---|---|---|---|
#1 | contrib_analyzer-1493714-add_gource_support.patch | 4.98 KB | skwashd |
Comments
Comment #1
skwashd CreditAttribution: skwashd commentedThe attached patch for supporting the gource custom log format is attached. The patch also cleans up the git log a little more to produce nicer results.
Comment #2
marvil07 CreditAttribution: marvil07 commentedI really appreciate the patch! :-)
It would be great if the creation of the gource format is at Makefile level.
Based on the last hunk, I think it is possible to do it doing awk scripting(later called from the Makefile), so it will work the same as the generation of the codeswarm input.
On the other side, notice that making it at the same level of Makefile, it will use the sed replacing done via makefile on the drupal_log_generator.py output.
One minor question, any reason to lowercase the author?
Comment #3
skwashd CreditAttribution: skwashd commentedThanks for the feedback. I will look into your scripts and suggestion to see if is makes sense to integrate it with the Makefile.
It keeps the username consistent. While preparing the data for my Drupal history visualisation video I spent a fair bit of time checking the data in an attempt to make it consistent.
Comment #4
marvil07 CreditAttribution: marvil07 commentedI see. While I was doing the same, I ended up using custom replacing(see the *.sed files) to fix some errors, but in theory we should use full drupal.org username, as mentioned on the commit message standards. On the other side, note that given two users on d.o: "Some" ans "some", they are both valid(users.name field uses utf8_bin collation).
Notice this project currently uses 3 targets: D7, D8 and full history. So the idea is to use current infrastructure there to use gource for every target ;-), i.e. see codeswarm output.