[Sugar-devel] [Dextrose] [PATCH] New suggestion for arrow drawing in Paint activity.olpc4170
bernie at codewiz.org
Thu Oct 21 10:45:12 EDT 2010
On Thu, 2010-10-21 at 12:57 +1100, James Cameron wrote:
> On Wed, Oct 20, 2010 at 10:16:52PM -0200, Bernie Innocenti wrote:
> > On Wed, 2010-10-20 at 21:36 +0530, Anurag Chowdhury wrote:
> > > I have uploaded the icon file at https://dev.laptop.org/ticket/4170
> > > needed for the patch.
> > It would be better if you could keep the icons with the patches.
> I disagree, sometimes they significantly hinder code review. It is up
> to the author of the patch to figure out how to present it best for
> review. Sometimes this might be as a separate patch.
But then what would the maintainer have to do to commit the complete
patch? First "git am", then "git add" the icon, then "git commit
--amend". In order to backport the patch to something like Dextrose, I'd
have to repeat the same job.
To keep patches readable _and_ easy to apply without too much manual
labor, perhaps the submitter could split off the icon to a separate
> Maybe I'm missing something. I use mutt. Given a git format-patch in
> e-mail with an added SVG, what tool chain do you engage to review the
You're right, one would have to apply the patch so they can open the
icon in a viewer. Providing a separate link to the icon would save
time, I agree with you.
// Bernie Innocenti - http://codewiz.org/
\X/ Sugar Labs - http://sugarlabs.org/
More information about the Sugar-devel