[sugar] Feature/string freeze exception: uninstall activities from the shell

Marco Pesenti Gritti mpgritti
Wed Jul 16 05:04:37 EDT 2008


On Wed, Jul 16, 2008 at 10:49 AM, Tomeu Vizoso <tomeu at tomeuvizoso.net> wrote:
> Are you sure we should couple so much the feature exception to the
> patch? I would have expected that an exception would be granted just
> in base of the need for it and how much sense it makes to have that
> feature in that release. Later, there would be a code review that
> should take in account the time we have left for stabilizing and how
> invasive is the patch.

IMO the developer who wrote the patch should lead the process, not the
release team, nor the people which are affected by the patch. Those
can participate to the discussion.

He should provide informations about why the patch is needed (for
example, Nepal needs it) and an assessment of risk (which the code
reviewer can disagree with on the mailing list).

> Other projects do code review and exception granting at the same moment?

The projects I know (mozilla and GNOME) does something like what I
described above. Exception granting is after code review.

Marco



More information about the Sugar-devel mailing list