Ok, cool.

If you feel like doing some coding before you start, the MCP/KCP review
work has given me an idea about how reviews can be made much easier...

In case you're interested, here goes.
Bug fixes archive makes finding stuff easier. It still takes time for
every changeset you want to check out to
- choose it
- download it
- open a clean image
before you can test.

So -
Write a small app that shows a list of changesets, html-scrapted off bug
fixes, and when you choose one, uses OSProcess to activate another
Squeak on a clean image, and load the changeset in question into that
image.

The process becomes - look at a sorted list, click your favorite,
review/test the code, go to your favorite mail client to write the
review. Next!

Daniel

Marcus Denker <marcus at ira.uka.de> wrote:
> 
> --Boundary_(ID_TXsa48xY6wvY37QasV8vzg)
> Content-type: multipart/signed; boundary=vkogqOf2sHV7VnPd;
>  protocol="application/pgp-signature"; micalg=pgp-sha1
> Content-disposition: inline
> 
> 
> --vkogqOf2sHV7VnPd
> Content-Type: text/plain; charset=us-ascii
> Content-Disposition: inline
> Content-Transfer-Encoding: quoted-printable
> 
> On Wed, May 07, 2003 at 12:45:46PM +0200, Daniel Vainsencher wrote:
> > > it even doesn't work anymore in 3.5 (it worked fine in 3.4).
> > > I think the main reason is that it adds to many methods that contructs
> > > the user-interface to make it multi-language. That's a nice example
> > > that there is a big difference between a patch and a module: patches
> > > are *extremely* fragile.=3D20
> > We agree this is a patch, I am aware it is not viale for the long term
> > outside the image. OTOH, being a patch by itself doesn't guarantee that
> > it won't screw us up. How do you reconcile these two? popular stuff
> > needs to get reviewed before inclusion, and quick. I hope it will be
> > working out wonderfully for this example.
> >=20
> Ok, actually we are in full agreement here.=20
> 
> The next thing I'l do is actually trying to make some reviews of the smaller
> fixes out there, and I hope someone will review mine. Especially
> the trivial changes are the one that will help us to move forward:=20
> 
> 1) Everyone will get the impression that "something is moving"
> 2) this will help to get the harvesting process going. How will we ever
> manage to add big things if we can't get the simple small changesets
> reviewed and harvested?
> 
>      Marcus
> 
> --=20
> Marcus Denker marcus at ira.uka.de  -- Squeak! http://squeak.de
> 
> 
> --vkogqOf2sHV7VnPd
> Content-Type: application/pgp-signature
> Content-Disposition: inline
> 
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.2.1 (Darwin)
> 
> iD8DBQE+uNybiAAcSb3IlZYRAmxOAJ9f5fQube/y9Z17shQA3MLGSQJuhgCeKQn6
> BJEcm8ynddJvEbJj2UI5418=
> =Yph7
> -----END PGP SIGNATURE-----
> 
> --vkogqOf2sHV7VnPd--
> 
> --Boundary_(ID_TXsa48xY6wvY37QasV8vzg)
> MIME-version: 1.0
> Content-type: text/plain; charset=us-ascii
> Content-transfer-encoding: 7BIT
> 
> 
> 
> --Boundary_(ID_TXsa48xY6wvY37QasV8vzg)--