Hi Brent!

As I said, I think this is very close to being hugely helpful to
reviewers and harvesters. Even better, it can make the process of
reviewing so painless we might actually get more active reviewers :-)

Do you think you can do the modifications below, which I think are
critical?

If not, I'll start looking at the code.

BTW, I thought about the bunching part below, it might be implemented by
using a hierarcial list. Bijan's categorical package loader could be a
useful starting point.

Daniel

Daniel Vainsencher <danielv at netvision.net.il> wrote:
> Ooh, this is cool... start to review a changeset in two clicks.. :-)
> 
> A couple of bugs -
> 
> 1. All the updates between the first and the last in excerpt appended
> from sqfixes are missing in the viewer. Ah, known issue 3/4 strikes.
> Seen cs.zip is also used.
> 2. The last entry (MCP-0005) does appear, but the subject is not quite
> right, effectively removing gm's review information :-(
> 
> Generally, it's better to be inclusive than exclusive here - if a mail
> has no attachment or no attachment we know what to do with, we can still
> display it, and let the user save it to disk for manual handling.
> 
> If it displays all mails correctly, lets the user handle some mails
> automatically, and for the rest allows saving, it's already better than
> using sqfixes raw, and should make the review process quicker and more
> effective. 
> 
> Another thing that would help navigation is if the display is more
> similar to sqfixes in some things - 
> - show dates, authors, and subject comment, instead of archive numbers
> - bunch up related things like so -
> **********************
> [ENH] FileInGZDirectly-nk (15 April 2003)
> 	* Ned Konz (13 December 2002)
> 	* Brent Vukmer (1 April 2003) ( [sm][cd][sl][et] )
> 	* Daniel Vainsencher (2 April 2003) ([approved])
> 	* Doug Way (15 April 2003) ([update - 5184]) 
> **********************
> 
> 
> Excerpt -
> **********************
> [ENH] isSymbol [small] (30 April 2003)
> 
> * Marcus Denker (30 April 2003)
> 
> [ENH][Refactoring] InstructionPrinter ([approved] dvf (30 April 2003)
> 
> * Daniel Vainsencher (30 April 2003)
> 
> [ANN] Keynote Speeches for ESUG Smalltalk Conference (30 April 2003)
> 
> * Roel Wuyts (30 April 2003)
> 
> [ANN] Bricks pre-alpha (morphic enhancement effort - Layouts) (30 April
> 2003)
> 
> * tblanchard at mac.com (30 April 2003) (morphic enhancement effort -
> Layouts)
> 
> [BUG][FIX] MissingHashRedefinitions (29 April 2003)
> 
> * Martin Wirblat (29 April 2003)
> 
> [ANN] [ENH] FillInTheBlankMorph Look (28 April 2003)
> 
> * (28 April 2003)
> 
> [ANN] SqueakMap: SPL PDF Library (27 April 2003)
> 
> * Lukas Renggli (27 April 2003)
> 
> [ENH][MCP] MCP-0005-MorphMethodsCategorization-dgd.cs ([cd][sl][er][et])
> (er gm) (no methods changed, onlycategorization) (27 April 2003)
> 
> * German Morales (27 April 2003) (no methods changed,
> onlycategorization) 
> **********************
> 
> Daniel
> 
> Brent Vukmer <bvukmer at blackboard.com> wrote:
> > I have registered the packages and load script for the Bug Fix Archive Viewer on SqueakMap.  The BugFixArchiveViewer automates a good bit of the process of browsing to the Bug Fixes Archive, downloading a fix/enhancement changeset, evaluating the changeset, and posting a comment to the Squeak development email list. The 'post comments' feature creates a mail message that is automatically built with a Bug-Fixes-Archive-parser-compliant subject line.
> > 
> > There are frighteningly-huge screenshots at: http://minnow.cc.gatech.edu/squeak/3204
> > 
> > This is for 3.6alpha, and 3.5 images.  The 1.0 version is definitely *alpha* software.  There are a number of annoying Known Issues.  Test pilots are the only people that may not tear their hair out when using the 1.0 version.
> > 
> > I have tested this load script on a current 3.6alpha image on Windows ( and a little on Unix ).  The "post comment" feature is designed to work after Celeste is removed from the image.
> > 
> > Known Issues:
> > 
> > (1) You have to close the current SMLoader instance after installing the load script.  I subclass from SMLoader, and make a few mods to it which the currently-running instance can't handle.
> > 
> > (2) The viewer takes a good long while to open the first time.  What can I say; screen-scraping takes awhile. 
> > 
> > (3) Currently each new BugFixArchive object is built off a humongous list of Archive changeset URLs, stored as one big String on the class side of BugFixArchive.  There are no URLs that don't end in '.cs.gz', because when I built the big String, I didn't parse for URLs ending in '.cs'. 
> > 
> > (4) Right now, a BugFixArchive object only shows Bug Fix Archive Posts that have a gzipped changeset attachment.