Mercurial & OpenSolaris: webrev

Mercurial is the distributed source code managment system that has been choosen for OpenSolaris. For the ON consolidation (the one I work in most of the time) this means it is replacing Sun's Teamware which was last shipped with Sun Studio 6 and is EOL. Even though it is officially EOL the ON consolidation (and several others) have continued to use it, with a number of home grown add on packages. One of those addons is a tool called 'webrev'.

The webrev tool is used for generating code changes for peer review prior to integration into a project gate or the main ON consolidation.

The current implementation of it is heavily based on Teamware assumptions and one of the other ON consolidation addon tools 'wx'.

As part of the transition to Mercurial we are going to need a version of 'webrev'.that understands Mercurial changesets between repositories.

I thought it was time I started having a look at Mercurial and also have a bash at updating webrev.

When I started this I had never used Mercurial before and had never read the documentation for it. On the other hand I am pretty familiar with how webrev works and I'm even mentioned in the code change history for webrev from prior to its inital integration as an official ON gate tool.

Well I've made enough progress on this in one afternoon of learning Mercurial and hacking as I go to post a webrev of the changes. This was actually done in a Mercurial repository so it was self generating in effect. I've logged OpenSolaris bug#6446689 to track this work.


Other than learning Mercurial and helping out the transition I had another more pressing nead for doing this now. My Google Summer of Code mente really needs to be able to generate codereview content for me and others to look at. I asked him to generate webrev before I remembered that he wouldn't have access to the Teamware binaries. So I though if I could quickly hack up a webrev he could use that would help me and others review his code. He doesn't know about this yet though :-)

Comments:

glanced at your diff quickly and noted that some parts seem to want to use awk, others nawk... some consistency (eg $awk) would be good.

Posted by oz on July 06, 2006 at 10:06 PM BST #

Post a Comment:
  • HTML Syntax: NOT allowed
About

DarrenMoffat

Search

Categories
Archives
« April 2014
MonTueWedThuFriSatSun
 
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
17
18
19
20
21
22
23
24
25
26
27
28
29
30
    
       
Today