Re: [Avogadro-devel] Change in avogadro[master]: Compilation fixes for Python wrapping.
HI devs, On Sun, Feb 5, 2012 at 9:02 PM, Marcus D. Hanwell (Code Review) rev...@kitware.com wrote: I'd like you to do a code review. Please visit http://review.source.kitware.com/4169 to review the following change. OK, I left my review comment... how does a reviewer sign off the patch, using Gerrit? Egon -- Dr E.L. Willighagen Postdoctoral Researcher Department of Bioinformatics - BiGCaT Maastricht University (http://www.bigcat.unimaas.nl/) Homepage: http://egonw.github.com/ LinkedIn: http://se.linkedin.com/in/egonw Blog: http://chem-bla-ics.blogspot.com/ PubList: http://www.citeulike.org/user/egonw/tag/papers -- Try before you buy = See our experts in action! The most comprehensive online learning library for Microsoft developers is just $99.99! Visual Studio, SharePoint, SQL - plus HTML5, CSS3, MVC3, Metro Style Apps, more. Free future releases when you subscribe now! http://p.sf.net/sfu/learndevnow-dev2 ___ Avogadro-devel mailing list Avogadro-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/avogadro-devel
Re: [Avogadro-devel] Change in avogadro[master]: Compilation fixes for Python wrapping.
On Mon, Feb 6, 2012 at 7:38 AM, Egon Willighagen egon.willigha...@gmail.com wrote: HI devs, On Sun, Feb 5, 2012 at 9:02 PM, Marcus D. Hanwell (Code Review) rev...@kitware.com wrote: I'd like you to do a code review. Please visit http://review.source.kitware.com/4169 to review the following change. OK, I left my review comment... how does a reviewer sign off the patch, using Gerrit? The +1 is enough (and I was emailed). There are different access levels (the +2 code review and +1 verified) which we keep to a core group who is able to merge new patches. This definitely needs fixing before our release, I need to get my head back into Boost Python to fix it properly. Thanks, Marcus -- Try before you buy = See our experts in action! The most comprehensive online learning library for Microsoft developers is just $99.99! Visual Studio, SharePoint, SQL - plus HTML5, CSS3, MVC3, Metro Style Apps, more. Free future releases when you subscribe now! http://p.sf.net/sfu/learndevnow-dev2 ___ Avogadro-devel mailing list Avogadro-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/avogadro-devel