Ah, but they don't have to sync. They just click on the link in the email, and get a download dialog in their browser (at least, they will if we construct the dialog correctly). It's actually less error- prone and more streamlined, because there's no possibility of neglecting to add the patch to the review mail.
jim On Oct 12, 2006, at 3:40 PM, P T Withington wrote: > I think this proposal stores less bits on hard drives but makes the > reviewer's job more complicated (they have to sync to get the > review changes). I prefer the existing mechanism. If there were > some way to tell the mailing list not to archive attachments, > perhaps that would be a good compromise? > > On 2006-10-12, at 15:07 EDT, Jim Grandy wrote: > >> I just remembered we had a possibly-complementary proposal on the >> table as well: >> >> Modify the review tools to check the changeset in to Subversion >> instead of sending it as an email attachment. >> >> So svn-review would do a >> >> > svn import patch.ptw.xPLr.tgz http://svn.openlaszlo.org/ >> openlaszlo/patches/patch.ptw.xPLr.tgz >> >> And add a link to the patch to the review email: >> >> http://svn.openlaszlo.org/openlaszlo/patches/patch.ptw.xPLr.tgz >> >> The argument is that we would then be storing the patch once, >> rather than transporting and storing it for each member of >> whichever list it goes to. And yes, these files would live on in >> Subversion, but that's arguably better than living on in many, >> many email archives. >> >> jim >> >> On Oct 12, 2006, at 11:45 AM, P T Withington wrote: >> >>> We've had a few complaints about the 'review mail' noise on the >>> laszlo-dev mailing list, especially the large attachments that go >>> with the review submission. If no one objects, I plan to create a >>> new list <mailto:[EMAIL PROTECTED]> for review mail (and >>> ask that reviewers direct their review requests there). If members >>> of this list want to continue to receive review mail, they will need >>> to join the new list. >>> >>> I'll send mail to the list when the new one is set up and ready to >>> accept subscriptions. >>> >>> P T Withington >>> OpenLazslo.ORG <http://www.openlaszlo.org> >>> mailto:[EMAIL PROTECTED] skype:ptwithy aim:[EMAIL PROTECTED] >>> >>> >>> >>> >>> _______________________________________________ >>> Laszlo-dev mailing list >>> [email protected] >>> http://www.openlaszlo.org/mailman/listinfo/laszlo-dev >> > _______________________________________________ Laszlo-dev mailing list [email protected] http://www.openlaszlo.org/mailman/listinfo/laszlo-dev
