Difference between revisions of "Mercurial history browsing"

From CDOT Wiki
Jump to: navigation, search
m (Project News)
m (Project News)
Line 152: Line 152:
  
 
'''October 1st, 2008'''
 
'''October 1st, 2008'''
* Chatted with ted about getting a hold of the pushlog database
+
* [http://sidkalra.com/files/mercurial/chats/ted/hgChat_ted_Oct1-2008.txt Chatted with ted] about getting a hold of the pushlog database
 
* Email conversation with jorendorff about setting up pushloghtml on my development environment and some questions/concerns regarding pushloghtml
 
* Email conversation with jorendorff about setting up pushloghtml on my development environment and some questions/concerns regarding pushloghtml
  
Line 185: Line 185:
 
* Filed the bug: [https://bugzilla.mozilla.org/show_bug.cgi?id=459727 pushloghtml should show more than 10 entries at a time]
 
* Filed the bug: [https://bugzilla.mozilla.org/show_bug.cgi?id=459727 pushloghtml should show more than 10 entries at a time]
 
* ted has already filed a bug for the expand/collapse feature: [https://bugzilla.mozilla.org/show_bug.cgi?id=445560  html pushlog should fold merge changesets into one line]
 
* ted has already filed a bug for the expand/collapse feature: [https://bugzilla.mozilla.org/show_bug.cgi?id=445560  html pushlog should fold merge changesets into one line]
* [http://sidkalra.com/files/mercurial/chats/jorendorff/hgChat_jorendorff&ted_oct13-08.txt Chatted with jorendorff and ted] about setting up my development environment and the expand/collapse feature
+
* [http://sidkalra.com/files/mercurial/chats/jorendorff/hgChat_jorendorff&ted_oct13-08.txt Chatted with jorendorff and ted] about setting up my development environment
 +
* [http://sidkalra.com/files/mercurial/chats/ted/hgChat_ted_Oct13-08.txt Chatted with ted] about the expand/collapse feature
  
 
'''October 14th, 2008'''
 
'''October 14th, 2008'''
 
* Applied Ted's patch to my development environment to fix the bug: [https://bugzilla.mozilla.org/show_bug.cgi?id=445560  html pushlog should fold merge changesets into one line], check out my [http://blog.sidkalra.com/2008/10/applying-a-patch-to-a-repo-using-mercurial/ blog post] for details
 
* Applied Ted's patch to my development environment to fix the bug: [https://bugzilla.mozilla.org/show_bug.cgi?id=445560  html pushlog should fold merge changesets into one line], check out my [http://blog.sidkalra.com/2008/10/applying-a-patch-to-a-repo-using-mercurial/ blog post] for details
* Chatted with ted and djc on IRC about pushloghtml
+
* djc on IRC about pushloghtml
 +
* [http://sidkalra.com/files/mercurial/chats/ted/hgChat_ted_Oct14-2008.txt Chatted with ted] about a problem with applying his patch
  
 
'''October 16th, 2008'''
 
'''October 16th, 2008'''
Line 209: Line 211:
 
* Had a conversation with djc regarding [https://bugzilla.mozilla.org/show_bug.cgi?id=459727 bug 459727]
 
* Had a conversation with djc regarding [https://bugzilla.mozilla.org/show_bug.cgi?id=459727 bug 459727]
 
* Had a conversation with ted regarding [https://bugzilla.mozilla.org/show_bug.cgi?id=445560 bug 445560]
 
* Had a conversation with ted regarding [https://bugzilla.mozilla.org/show_bug.cgi?id=445560 bug 445560]
 +
* [http://sidkalra.com/files/mercurial/chats/ted/hgChat_ted_Oct29-08.txt Had a conversation with ted] regarding the expand/collapse feature
 +
** Asked him what his patch doesn't do properly
 +
** Wanted to find out what needed to be fixed
  
 
'''November 3rd, 2008'''
 
'''November 3rd, 2008'''
Line 247: Line 252:
 
* Edited the blog post [http://blog.sidkalra.com/2008/11/v02-release-showing-the-correct-author-name-for-bug-459727/ about not getting the right author name]. The edit provides a solution to the problem
 
* Edited the blog post [http://blog.sidkalra.com/2008/11/v02-release-showing-the-correct-author-name-for-bug-459727/ about not getting the right author name]. The edit provides a solution to the problem
 
* Edited the [http://blog.sidkalra.com/2008/11/a-puzzling-problem-with-ctxparentsidescription/ blog post] detailing my problem with ctx.parents()[i].description()
 
* Edited the [http://blog.sidkalra.com/2008/11/a-puzzling-problem-with-ctxparentsidescription/ blog post] detailing my problem with ctx.parents()[i].description()
 +
* [http://sidkalra.com/files/mercurial/chats/ted/hgChat_ted_Nov12_08.txt Chatted with ted] about a problem with not getting the correct author name. Sometimes the email was being shown while other times the actual name was showing up
  
 
'''November 13th, 2008'''
 
'''November 13th, 2008'''
Line 269: Line 275:
 
* Posted new hg_templates patch for [https://bugzilla.mozilla.org/show_bug.cgi?id=445560 bug 445560]   
 
* Posted new hg_templates patch for [https://bugzilla.mozilla.org/show_bug.cgi?id=445560 bug 445560]   
 
** Made a [http://blog.sidkalra.com/2008/11/v03-release-fix-for-my-patch-for-bug-445560/ blog post] about the two new patches for [https://bugzilla.mozilla.org/show_bug.cgi?id=445560 bug 445560]
 
** Made a [http://blog.sidkalra.com/2008/11/v03-release-fix-for-my-patch-for-bug-445560/ blog post] about the two new patches for [https://bugzilla.mozilla.org/show_bug.cgi?id=445560 bug 445560]
 +
 +
'''November 29th, 2008'''
 +
* [http://sidkalra.com/files/mercurial/chats/ted/hgChat_ted_Nov29_08.txt Chatted with ted] about the expand/collapse patch
 +
** ted identified a problem with patch
 +
** I said I would fix it
  
 
'''December 1st, 2008'''
 
'''December 1st, 2008'''

Revision as of 15:49, 5 December 2008

Project Name

Adding Functionality to the Browser Based UI for Mercurial History Browsing, hgweb

Project Description

Mozilla is just starting to use an exciting new distributed version control tool: Mercurial.

One of the exciting things about Mercurial is that the history—the list of all the changes that have ever been checked in—is not linear. It frequently has branches and merges. This is actually a good, important feature, as you know if you've read a little about distributed version control. The downside is that the history becomes a maze of twisty little passages. Benjamin Smedberg's demo shows the history of a repository with lots of merges (each box is a check-in; you can click on the boxes to move around).

There have been a couple attempts at showing history in an intuitive, graphical way. Mercurial comes with a web UI for browsing the repository, including history (here's what it looks like). It also comes with an "hg glog" extension that draws history as ASCII art, and an "hg view" extension that does roughly the same thing with a little GUI. Sadly, none of these applications qualifies as awesome.

The project's goal is to rectify this sad situation by writing an awesome browser-based UI for navigating Mercurial repository history.

Resources: jorendorff, bsmedberg

Project Leader(s)

Siddharth Kalra

Project Contributor(s)

  • Tiago Moreira

Project Mentors

  • Jason Ordendorff (jorendorff)
  • Dirkjan Ochtman (djc)
  • Ted Mielczarek (ted)
  • Benjamin Smedberg (bsmedberg)

Project Details

September 23rd, 2008

v0.1 Release Details

  • to get pushloghtml working on my machine (some progress made on this part)
  • to locate the source files that dictate the amount of pushlog entries that get displayed and then alter it to display more entries
    • Turns out that there was some miscommunication and I need to add an AJAX scrolling feature that dynamically loads more changesets
  • to add code that allows expand/collapse functionality (apparently all I need is a js lib according to humph)
    • Turns out that there was another miscommunication as Ted has already added this feature. Look here. However the patch has some problems that will need to be fixed. I might fix this for my 0.2 release (undecided for now)
  • To release v0.1 on time

v0.2 Release Details (More Information) (Final result)

  • To continue work on bug 459727
    • Implement a loader GIF (View here)
    • Getting the correct Chronological order when adding more entries OnScroll (View here) (Final result)
    • Getting merge changesets to show up correctly when adding more entries OnScroll (View here) (final result)
      • Making sure the new entries that load OnScroll comply with bug 445560 (Expand/Collapse feature) ==> REMOVED from v0.2 - this functionality should wait until the expand/collapse functionality itself is approved and goes live. This should be a separate bug
    • To make buglinks work
    • To make changeset links work
    • To get the correct date formating
    • On first page load show 20 entries instead of 10 (View here)
    • To get the correct author names (right now, sometimes it returns the email address while other times the name of the author) (View solution here)
  • To fix ted's patch for bug 445560 (View here)
  • To implement "the which files were touched by a changeset" feature for bug 448707 (View here)

v0.3 Release Details (View here)

  • Problem with my patch for bug 445560 (View here)
    • Fixed problems with my patch by adding 2 new patches (View here)
      • Fix 1: Should use ids instead of dates
      • Fix 2: Should store unique identifiers in the class instead of the id of the tag
  • My patch for bug 448707 approved! (View here)
  • Implementing the Graph view for the pushlog
  • Problem with my patch for bug 459727 (View here) (fix for my patch)
    • To calculate the max use SELECT COUNT(*) FROM ... instead of asking for all the rows and then counting them
    • getMaxEntries called, then 'start' used once the user scrolls. But the user might scroll before the first result comes back, while start is still zero -- a race condition.
    • The JS code isn't consistently indented
    • Patch seems to contain some non-ASCII characters. Bugzilla doesn't render it properly. Consider using an HTML entity reference, like – or —, instead of an actual Unicode character.
    • In a few places the b tag is used where the server uses the strong tag; be consistent
    • Loads more results when the user scrolls. What if the initial results don't fill up the window?
    • Better to move CSS and formatting to stylesheets where possible
  • Problem #1 explained in comment #9 by jorendorff still existed with my new patch (it is fixed, view here)
  • Improved the bug link feature for bug 459727 (View here)
  • My patch for bug 445560 approved (View here)

Project News

September 18th, 2008

  • Picked the Mercurial History Browsing project

September 23rd, 2008

September 25th, 2008

  • Chatted with jorendorff on setting my development environment
  • Email conversation with jorendorff, asked some preliminary questions about the project and a hello world type of activity

September 26th, 2008

  • Email conversation with jorendorff about setting up the development environment

September 28th, 2008

  • Played around with hg. Details here
  • Email conversation with jorendorff, asked for help on setting up the development environment and suggestions on a v0.1 release
  • Email conversation with djc about hg_templates

September 29th, 2008

  • Email conversation with jorendorff about v0.1 release ideas

October 1st, 2008

  • Chatted with ted about getting a hold of the pushlog database
  • Email conversation with jorendorff about setting up pushloghtml on my development environment and some questions/concerns regarding pushloghtml

October 2nd, 2008

  • Development Environment almost setup and running except for pushloghtml
  • Decided on v0.1 release (tentative, still need to clear it with humph)
  • Email conversation with jorendorff clarifying some concerns about pushloghtml
  • Email conversation with ted explaining how to setup pushloghtml

October 3rd, 2008

  • Decided on v0.1 release (confirmed with humph). Updated v0.1 release blog post
  • Email conversation asking about simplejson and pysqlite2 modules
  • Email conversation with bsmedberg explaining simplejson and pysqlite2 questions

October 5th, 2008

  • Problems with getting pushloghtml to work here. Work in progress...

October 6th, 2008

  • Multiple email conversations asking djc and bsmedberg about a "no module found" error relating to simplejson and pysqlite2

October 7th, 2008

  • Email conversation with djc, discussing ideas about the causes of the "no module found" error

October 8th, 2008

  • Email conversation with djc asking questions about building Mercurial from source

October 9th, 2008

October 13th, 2008

October 14th, 2008

October 16th, 2008

October 17th, 2008

  • Email conversation asking questions about the json-pushes script

October 18th, 2008

October 19th, 2008

October 23rd, 2008

October 29th, 2008

  • Had a conversation with djc regarding bug 459727
  • Had a conversation with ted regarding bug 445560
  • Had a conversation with ted regarding the expand/collapse feature
    • Asked him what his patch doesn't do properly
    • Wanted to find out what needed to be fixed

November 3rd, 2008

  • Had a conversation with djc regarding the date formatting for bug 459727

November 4th, 2008

  • Had a conversation relating to v0.2 release with jorendorff (decided what I will include in v0.2)

November 5th, 2008

November 6th, 2008

November 7th, 2008

  • Made a blog post discussing the details for v0.2 release
  • Made a blog post about a problem with implementing merge changesets for bug 459727

November 8th, 2008

November 10th, 2008

November 11th, 2008

November 12th, 2008

  • Had a discussion with ted and bsmedberg about the issue explained by the blog post here
  • Edited the blog post about not getting the right author name. The edit provides a solution to the problem
  • Edited the blog post detailing my problem with ctx.parents()[i].description()
  • Chatted with ted about a problem with not getting the correct author name. Sometimes the email was being shown while other times the actual name was showing up

November 13th, 2008

November 15th, 2008

November 16th, 2008

November 19th, 2008

November 24th, 2008

November 29th, 2008

  • Chatted with ted about the expand/collapse patch
    • ted identified a problem with patch
    • I said I would fix it

December 1st, 2008

  • Had a discussion with jorendorff on the Graph view
  • Made celebratory blog post about my patch for bug 448707 being accepted
  • Made a blog post about understanding how graph data is passed from the server side to the client side

December 2nd, 2008

  • Made a blog post about completing the 1st iteration for the graph (View here)
  • Made a blog post about problems identified by jorendorff concerning my patch for bug 459727

December 3rd, 2008

  • Added a new patch for hgpoller for bug 459727
  • Added a new patch for hg_templates for bug 459727
  • Blogged about the above patches here
  • Made a blog post explaining a fix concerning a problem with my newly submitted patch (see above) for bug 459727. Problem #1 explained in comment #9 by jorendorff still existed even with my new patch

December 4th, 2008

December 5th, 2008

  • Had a chat with jorendorff and ted
    • Talked about who should be the reviewer for bug
    • Talked about continuing on and working on this project next semester
    • Final discussion for the project for this semester
  • Made a blog post about another patch of mine getting approved for bug 445560

External Links

Other