<br><br><div class="gmail_quote">On Thu, Oct 11, 2012 at 6:54 PM, Ruben Rodríguez <span dir="ltr"><<a href="mailto:ruben@activitycentral.com" target="_blank">ruben@activitycentral.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
2012/10/11 Ajay Garg <<a href="mailto:ajay@activitycentral.com">ajay@activitycentral.com</a>>:<br>
<div class="im">> Having so many branches is an overkill; it is too efforts-intensive,<br>
> error-prone and generally daunting.<br>
<br>
</div>It may be a bigger effort when you compare it with how we work now,<br>
but there are good reasons for that scheme:<br>
<br>
* A lot more provision against failure. Think of what would happen if<br>
we ever push a rpm update that unexpectedly makes the machines stop to<br>
boot or causes some other big trouble. I've seen that happen in<br>
several distros, and it is a project killer. At least we need to<br>
separate what code is golden, what is being tested by the QA team and<br>
what is fresh from the coders.<br></blockquote><div><br>Well, I think the onus lies on the developer then, since he/she was responsible for committing a fix that caused the catastrophe.<br>Moreover, I think this is a blessing in disguise (if I may say), as this will cause such catastrophes to be caught much earlier.<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">
* Most of that work is supposed to be automated or to be done by the<br>
RM. Code going up the branches should always be fast-forward.<br></blockquote><div><br>Again, another candidate for time-wastage. Why not work in one branch itself? That would prevent un-necessary duplicate testing by the RM.<br>
<br>It makes MUCH, MUCH, MUCH more sense to concentrate all testing for one codebase (i.e. for one code, corresponding to one branch).<br>(and this applies to everyone, not just the RM).<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">
* The developers only work in their personal clones of the devel<br>
branch,</blockquote><div><br><br>That is understood. As long as developers don't commit, they are free to do anything they want to do on the branch, as long as they don't commit. A code is "effectively" in the branch, only and only after it is remotely-pushed.<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"> so from their perspective there is only *one* branch that<br>
matters. All the rest is risk management. Of course this is a<br>
provision for more developers working on the code in the future, so I<br>
understand it can look overkill now that you do most of the code alone<br>
:)<br>
* Some steps in the scheme are optional and can be removed, like the<br>
master and the production branches being different. As with any<br>
proposal this needs to be tuned up.<br></blockquote><div><br><br>All this I am speaking from personal experience, as giving commit-access to the develper was a big step in saving time (courtesy Anish).<br><br>Earlier, before committing, me and Anish had to go through the testing process individually (meaning double effort; surely no gain). Instead, Anish then started to test on a RPM-level (i.e. building rpms, and deploying on the XO). That served the following purposes ::<br>
<br>a)<br>End-to-End testing<br><br>b)<br>Testing at a rpm-level, not patch-level.<br><br><br><br>Anyways, now that the code-review would be in-place soon, the committer (other than the developer) will anyways test the patch once before committing (thus providing more security against catastrophes).<br>
<br><br>In summary, I am not in favour of multiple branches. As Rafa said, we may split into a devel-branch sometime just before the final release; but even that too if necessary.<br><br>Let's conceentrate on just one branch - meaning one branch to test.<br>
<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">
<div class="HOEnZb"><div class="h5"><br>
--<br>
Rubén Rodríguez<br>
CTO<br>
Activity Central: <a href="http://activitycentral.com" target="_blank">http://activitycentral.com</a><br>
<br>
Facebook: <a href="https://activitycentral.com/facebook" target="_blank">https://activitycentral.com/facebook</a><br>
Google+: <a href="https://activitycentral.com/googleplus" target="_blank">https://activitycentral.com/googleplus</a><br>
Twitter: <a href="https://activitycentral.com/twitter" target="_blank">https://activitycentral.com/twitter</a><br>
_______________________________________________<br>
Dextrose mailing list<br>
<a href="mailto:Dextrose@lists.sugarlabs.org">Dextrose@lists.sugarlabs.org</a><br>
<a href="http://lists.sugarlabs.org/listinfo/dextrose" target="_blank">http://lists.sugarlabs.org/listinfo/dextrose</a><br>
</div></div></blockquote></div><br><br clear="all"><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><br>