<br><br><div class="gmail_quote">On Wed, Mar 27, 2013 at 11:21 PM, Daniel Narvaez <span dir="ltr"><<a href="mailto:dwnarvaez@gmail.com" target="_blank">dwnarvaez@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 27 March 2013 17:39, Ajay Garg <<a href="mailto:ajay@activitycentral.com">ajay@activitycentral.com</a>> wrote:<br>
> Just one request ::<br>
> =============<br>
><br>
> please include the patch ASAP if it works 90% of the time (although it is<br>
> working 100% for me at this point :P ).<br>
><br>
> If it is thought that some minor-change/bug-fix is needed, it can be done<br>
> "in" the mainline. That would save a lot of maintenance costs in maintaining<br>
> a singular, big patch.<br>
<br>
Hi Ajay,<br>
<br>
I don't know the specific history of this patch but, as a rule,<br>
patches should be pushed to mainline when they are fully working, or<br>
at least they are thought to be. If the maintainers point out issues,<br>
they should be solved before committing. Often the remainning 10% is<br>
the one which takes more time to achieve.<br></blockquote><div><br>Yes, I understand that :)<br><br>"My" 10% intended any minor UI-tweaks (string change, text-positioning, etc), that are often not workflow-critical, and very easy to fix.<br>
Obviously, anything that breaks any use-case is a reason enough to reject the patch straightaway (even if that is 1% of the case).<br> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">

<br>
It's in the maintainers right to pretend they fully like your<br>
contribution before they take responsibility for it. They are going to<br>
take over a lot of the maintenance costs you are complainig about<br>
after all :)<br>
<br>
If the patch is too big, it's often possible to split it up so that<br>
parts of it can be landed earlier.<br></blockquote><div><br><br>Hmm .. it is big .. but I don't really know if any functionality should be broken into several patches, especially if the entire patch is necessary to make the feature work at all.<br>
<br>Anyways, decide upon yourself when I float the patch this weekend :)<br><br><br> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
_______________________________________________<br>
Sugar-devel mailing list<br>
<a href="mailto:Sugar-devel@lists.sugarlabs.org">Sugar-devel@lists.sugarlabs.org</a><br>
<a href="http://lists.sugarlabs.org/listinfo/sugar-devel" target="_blank">http://lists.sugarlabs.org/listinfo/sugar-devel</a><br>
</blockquote></div><br><br clear="all"><br>-- <br><font face="arial, sans-serif">Regards,<br><br>Ajay Garg</font><br style="font-family:arial,sans-serif;font-size:13px;background-color:rgb(255,255,255)"><font face="arial, sans-serif">Dextrose Developer</font><br style="font-family:arial,sans-serif;font-size:13px;background-color:rgb(255,255,255)">
<span style="font-family:arial,sans-serif;font-size:13px;background-color:rgb(255,255,255)">Activity Central: </span><a href="http://activitycentral.com/" style="font-family:arial,sans-serif;font-size:13px;background-color:rgb(255,255,255)" target="_blank">http://activitycentral.com</a>