<html><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">also it would be cool to be able to detect conflicting dependencies.<div><br><div><div>On Oct 9, 2009, at 7:35 AM, Julien Viet wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Oct 9, 2009, at 2:32 AM, Dimitri BAELI wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite">Hello,<br><br>&nbsp;&nbsp; Last commit broke the build. And will probably block the night team in VN :-)<br><br><a href="http://builder.exoplatform.org/hudson/view/All/job/gatein-portal-trunk/97/console">http://builder.exoplatform.org/hudson/view/All/job/gatein-portal-trunk/97/console</a><br></blockquote><div><br></div><div>it is a snapshot issue. Chris has taken the initiative to do a minor refactor of the portlet container (moving the LOCAL_PORTLET_INVOKER_ID constant from one class to another) that is not yet visible to the world depending on it.</div><div><br></div><div>I see two issues here:</div><div><br></div><div>1/ modifying an API used by a consumer (outlined by org.gatein.pc.api package name)</div><div><br></div>2/ the usual issue of using snapshots with a random life cycle driven by trunk development&nbsp;<br><blockquote type="cite"> <span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: 'Times New Roman'; font-size: medium; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px;"><span class="Apple-style-span" style="border-collapse: collapse; font-family: Verdana,Helvetica,'sans serif'; font-size: 11px;"><pre style="white-space: pre-wrap;">/home/hudson/hudson/jobs/gatein-portal-trunk/workspace/gatein/portal/trunk/component/pc/src/main/java/org/exoplatform/portal/pc/ExoKernelIntegration.java:[133,61] cannot find symbol<br>symbol  : variable LOCAL_PORTLET_INVOKER_ID<br>

location: interface org.gatein.pc.api.PortletInvoker<br></pre></span></span><br class="Apple-interchange-newline"><b>Questions:</b><br>&nbsp;&nbsp; How should we behave in such case ?<br>&nbsp;&nbsp; Can we rollback the commit if the responsible is not present for few hours ? Or let it be and update to the last known stable state.<br></blockquote><div><br></div><div>we do not have a general policy for that.</div><div><br></div><div>but in this case it is related to the policy I want to put in place for using branches along with snapshots.</div><br><blockquote type="cite"> <br><b>Experimentation:</b><br>&nbsp;&nbsp; I'll send the build notifications (only failures and back to normal state) to the gatein-dev list. Tell me if that's not acceptable also.<br>&nbsp;&nbsp; Is there another target ML for that ?<br> <br></blockquote><div><br></div><div>sounds acceptable given the policy.</div><div><br></div><div>I would add also that using a snapshot in the dependencies of a project should also send some kind of warn.</div><div><br></div><blockquote type="cite">The notification rules:<br><span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: 'Times New Roman'; font-size: medium; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px;"><span class="Apple-style-span" style="font-family: Verdana,Helvetica,'sans serif'; font-size: 11px;"><ol> <li>Every failed build triggers a new e-mail.</li><li>A successful build after a failed (or unstable) build triggers a new e-mail, indicating that a crisis is over.</li><li>An unstable build after a successful build triggers a new e-mail, indicating that there's a regression.</li> <li>Unless configured, every unstable build triggers a new e-mail, indicating that regression is still there.</li></ol></span></span><br><br clear="all">Dimitri BAELI - eXo Platform SAS<br> _______________________________________________<br>gatein-dev mailing list<br><a href="mailto:gatein-dev@lists.jboss.org">gatein-dev@lists.jboss.org</a><br><a href="https://lists.jboss.org/mailman/listinfo/gatein-dev">https://lists.jboss.org/mailman/listinfo/gatein-dev</a><br></blockquote></div><br></div></blockquote></div><br></div></body></html>