# [ANNOUNCE] Gerrit Code Review 2.0.3 11 messages from 2009-02-17 to 2009-02-18. Participants: Shawn O. Pearce, Frank Li, Gerrit Pape, Jacob Helwig, Johannes Schindelin, Boyd Lynn Gerber, thestar@fussycoder.id.au. Thread: https://gitlist.dev/t/17838 ## Shawn O. Pearce, 2009-02-17 01:12 Subject: [ANNOUNCE] Gerrit Code Review 2.0.3 Message-ID: <20090217011256.GA23314@spearce.org> URL: https://gitlist.dev/e/20090217011256.GA23314%40spearce.org ``` Gerrit is a web based code review system, facilitating online code reviews for projects using the Git version control system. Gerrit makes reviews easier by showing changes in a side-by-side display, and allowing inline comments to be added by any reviewer. Gerrit simplifies Git based project maintainership by permitting any authorized user to submit changes to the master Git repository, rather than requiring all approved changes to be merged in by hand by the project maintainer. This functionality enables a more centralized usage of Git. Gerrit 2.x and later run in any standard Java servlet container, and includes an embedded SSH daemon, providing gitosis like group security and data access. If desired, the review process can be skipped by directly pushing branches/tags, if the necessary access has been granted to the user. The last time I announced Gerrit on this list, it only ran on Google App Engine. Gerrit 2.x and later is designed to run in any environment, without requiring the use of a Google product, or network connectivity to the Internet. This makes it much more suited for corporate deployments, or open source projects which already have their own servers. Currently, Gerrit is under very active development, with stable releases being made about once per week. As such, I won't be announcing future releases here very often. For more information: Homepage: http://code.google.com/p/gerrit/ Live Demo: http://review.source.android.com/open Source: git://android.git.kernel.org/tools/gerrit.git (but see http://code.google.com/p/gerrit/wiki/Source?tm=4) -- Shawn. ``` ## Frank Li, 2009-02-17 07:11 Subject: Re: [ANNOUNCE] Gerrit Code Review 2.0.3 Message-ID: <1976ea660902162311q12e11ec5g7a43ab637186985e@mail.gmail.com> URL: https://gitlist.dev/e/1976ea660902162311q12e11ec5g7a43ab637186985e%40mail.gmail.com In-Reply-To: <20090217011256.GA23314@spearce.org> ``` when I click http://review.source.android.com/open , A Gerrit file download by IE 2009/2/17 Shawn O. Pearce : > Gerrit is a web based code review system, facilitating online code > reviews for projects using the Git version control system. > > Gerrit makes reviews easier by showing changes in a side-by-side > display, and allowing inline comments to be added by any reviewer. > > Gerrit simplifies Git based project maintainership by permitting > any authorized user to submit changes to the master Git repository, > rather than requiring all approved changes to be merged in by > hand by the project maintainer. This functionality enables a more > centralized usage of Git. > > Gerrit 2.x and later run in any standard Java servlet container, > and includes an embedded SSH daemon, providing gitosis like group > security and data access. If desired, the review process can be > skipped by directly pushing branches/tags, if the necessary access > has been granted to the user. > > The last time I announced Gerrit on this list, it only ran on > Google App Engine. Gerrit 2.x and later is designed to run in > any environment, without requiring the use of a Google product, > or network connectivity to the Internet. This makes it much more > suited for corporate deployments, or open source projects which > already have their own servers. > > Currently, Gerrit is under very active development, with stable > releases being made about once per week. As such, I won't be > announcing future releases here very often. > > For more information: > > Homepage: http://code.google.com/p/gerrit/ > Live Demo: http://review.source.android.com/open > Source: git://android.git.kernel.org/tools/gerrit.git > (but see http://code.google.com/p/gerrit/wiki/Source?tm=4) > > -- > Shawn. > -- > To unsubscribe from this list: send the line "unsubscribe git" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > ``` ## Gerrit Pape, 2009-02-17 09:01 Subject: Re: [ANNOUNCE] Gerrit Code Review 2.0.3 Message-ID: <20090217090142.5573.qmail@a4b546cad31603.315fe32.mid.smarden.org> URL: https://gitlist.dev/e/20090217090142.5573.qmail%40a4b546cad31603.315fe32.mid.smarden.org In-Reply-To: <20090217011256.GA23314@spearce.org> ``` On Mon, Feb 16, 2009 at 05:12:56PM -0800, Shawn O. Pearce wrote: > Gerrit is a web based code review system, facilitating online code > Gerrit makes reviews easier by showing changes in a side-by-side > Gerrit simplifies Git based project maintainership by permitting > Gerrit 2.x and later run in any standard Java servlet container, > Currently, Gerrit is under very active development, with stable Somehow this scares me. Regards, Gerrit. ``` ## Shawn O. Pearce, 2009-02-17 15:13 Subject: Re: [ANNOUNCE] Gerrit Code Review 2.0.3 Message-ID: <20090217151350.GH18525@spearce.org> URL: https://gitlist.dev/e/20090217151350.GH18525%40spearce.org In-Reply-To: <20090217090142.5573.qmail@a4b546cad31603.315fe32.mid.smarden.org> ``` Gerrit Pape wrote: > On Mon, Feb 16, 2009 at 05:12:56PM -0800, Shawn O. Pearce wrote: > > Gerrit is a web based code review system, facilitating online code > > Gerrit makes reviews easier by showing changes in a side-by-side > > Gerrit simplifies Git based project maintainership by permitting > > Gerrit 2.x and later run in any standard Java servlet container, > > > Currently, Gerrit is under very active development, with stable > > Somehow this scares me. Me too. :) http://code.google.com/p/gerrit/wiki/Background explains some of Gerrit Code Review's lineage. It goes back to a fork of Rietveld, which was named after the Dutch architect Gerrit Rietveld. Because we forked so far away from Rietveld, we renamed it to Gerrit... -- Shawn. ``` ## Shawn O. Pearce, 2009-02-17 15:18 Subject: Re: [ANNOUNCE] Gerrit Code Review 2.0.3 Message-ID: <20090217151838.GI18525@spearce.org> URL: https://gitlist.dev/e/20090217151838.GI18525%40spearce.org In-Reply-To: <1976ea660902162311q12e11ec5g7a43ab637186985e@mail.gmail.com> ``` Frank Li wrote: > when I click http://review.source.android.com/open , A Gerrit file > download by IE Likely it is this open bug: http://jira.source.android.com/jira/browse/GERRIT-62 Gerrit is known to not work in Internet Explorer. As an open source developer, my primary desktop system runs Linux. My portable laptops all run Mac OS X, because its shiny. I have no system capable of running Internet Explorer, nor do I desire to send dollars towards Redmond in order to obtain an OS license in order to do so. Something is busted in the IE AJAX code used by Gerrit Code Review. I'm guessing that instead of making the data available to the AJAX callback, IE wants to save it as a file on disk. No clue why. And due to the lack of IE, I am unable to debug the problem myself. I am waiting for someone who really cares about using IE to debug it and contribute the patch back. FWIW, there are open source browsers available even for Microsoft Windows. Firefox and Chrome come to mind... both work, as their underlying engines are ported to more OSes and are easier to test. Firefox is currently better tested however. > 2009/2/17 Shawn O. Pearce : > > Gerrit is a web based code review system, facilitating online code > > reviews for projects using the Git version control system. > > > > Live Demo: http://review.source.android.com/open -- Shawn. ``` ## Jacob Helwig, 2009-02-17 16:13 Subject: Re: [ANNOUNCE] Gerrit Code Review 2.0.3 Message-ID: <8c9a060902170813ie8f5bc1gfed53e4f31f64c23@mail.gmail.com> URL: https://gitlist.dev/e/8c9a060902170813ie8f5bc1gfed53e4f31f64c23%40mail.gmail.com In-Reply-To: <20090217151838.GI18525@spearce.org> ``` On Tue, Feb 17, 2009 at 07:18, Shawn O. Pearce wrote: > Gerrit is known to not work in Internet Explorer. As an open source > developer, my primary desktop system runs Linux. My portable laptops > all run Mac OS X, because its shiny. I have no system capable of > running Internet Explorer, nor do I desire to send dollars towards > Redmond in order to obtain an OS license in order to do so. > > Something is busted in the IE AJAX code used by Gerrit Code Review. > I'm guessing that instead of making the data available to the AJAX > callback, IE wants to save it as a file on disk. No clue why. > And due to the lack of IE, I am unable to debug the problem myself. > I am waiting for someone who really cares about using IE to debug > it and contribute the patch back. > I've found IEs4Linux helpful for debugging IE-only issues under Linux. http://www.tatanka.com.br/ies4linux/page/Main_Page I don't recall, off hand, if it'll provide IE 7 (and the site isn't working for me right now, so I can't confirm this). It does provide IE 5.0, 5.5, and 6.0, however. -Jacob ``` ## Johannes Schindelin, 2009-02-17 16:44 Subject: Re: [ANNOUNCE] Gerrit Code Review 2.0.3 Message-ID: URL: https://gitlist.dev/e/alpine.DEB.1.00.0902171743280.6185%40intel-tinevez-2-302 In-Reply-To: <8c9a060902170813ie8f5bc1gfed53e4f31f64c23@mail.gmail.com> ``` Hi, On Tue, 17 Feb 2009, Jacob Helwig wrote: > On Tue, Feb 17, 2009 at 07:18, Shawn O. Pearce wrote: > > Gerrit is known to not work in Internet Explorer. As an open source > > developer, my primary desktop system runs Linux. My portable laptops > > all run Mac OS X, because its shiny. I have no system capable of > > running Internet Explorer, nor do I desire to send dollars towards > > Redmond in order to obtain an OS license in order to do so. > > > > Something is busted in the IE AJAX code used by Gerrit Code Review. > > I'm guessing that instead of making the data available to the AJAX > > callback, IE wants to save it as a file on disk. No clue why. > > And due to the lack of IE, I am unable to debug the problem myself. > > I am waiting for someone who really cares about using IE to debug > > it and contribute the patch back. > > > > I've found IEs4Linux helpful for debugging IE-only issues under Linux. > > http://www.tatanka.com.br/ies4linux/page/Main_Page > > I don't recall, off hand, if it'll provide IE 7 (and the site isn't > working for me right now, so I can't confirm this). It does provide > IE 5.0, 5.5, and 6.0, however. If I understand Shawn (and the motto "scratch your own itch") correctly, Shawn was not so much interested in being pointed to a way of running IE himself, but more in people who absolutely want to run IE themselves to go and fix the issues. Ciao, Dscho ``` ## Shawn O. Pearce, 2009-02-17 17:09 Subject: Re: [ANNOUNCE] Gerrit Code Review 2.0.3 Message-ID: <20090217170915.GJ18525@spearce.org> URL: https://gitlist.dev/e/20090217170915.GJ18525%40spearce.org In-Reply-To: ``` Johannes Schindelin wrote: > On Tue, 17 Feb 2009, Jacob Helwig wrote: > > On Tue, Feb 17, 2009 at 07:18, Shawn O. Pearce wrote: > > > > > > Something is busted in the IE AJAX code used by Gerrit Code Review. > > > > I've found IEs4Linux helpful for debugging IE-only issues under Linux. > > > > http://www.tatanka.com.br/ies4linux/page/Main_Page > > > > I don't recall, off hand, if it'll provide IE 7 (and the site isn't > > working for me right now, so I can't confirm this). It does provide > > IE 5.0, 5.5, and 6.0, however. > > If I understand Shawn (and the motto "scratch your own itch") correctly, > Shawn was not so much interested in being pointed to a way of running IE > himself, but more in people who absolutely want to run IE themselves to go > and fix the issues. Indeed. There are probably also license issues with running IE under Linux. IIRC Microsoft only licenses IE for use under Windows itself, and with a valid Windows OS license. As a professional developer, I do actually try to respect the license under which software is delivered to me, even if I have no respect for the vendor... ;-) -- Shawn. ``` ## Boyd Lynn Gerber, 2009-02-17 18:53 Subject: Re: [ANNOUNCE] Gerrit Code Review 2.0.3 Message-ID: URL: https://gitlist.dev/e/alpine.LNX.2.00.0902171152270.21161%40suse104.zenez.com In-Reply-To: <8c9a060902170813ie8f5bc1gfed53e4f31f64c23@mail.gmail.com> ``` On Tue, 17 Feb 2009, Jacob Helwig wrote: > On Tue, Feb 17, 2009 at 07:18, Shawn O. Pearce wrote: >> Gerrit is known to not work in Internet Explorer. As an open source >> developer, my primary desktop system runs Linux. My portable laptops >> all run Mac OS X, because its shiny. I have no system capable of >> running Internet Explorer, nor do I desire to send dollars towards >> Redmond in order to obtain an OS license in order to do so. >> >> Something is busted in the IE AJAX code used by Gerrit Code Review. >> I'm guessing that instead of making the data available to the AJAX >> callback, IE wants to save it as a file on disk. No clue why. >> And due to the lack of IE, I am unable to debug the problem myself. >> I am waiting for someone who really cares about using IE to debug >> it and contribute the patch back. >> > > I've found IEs4Linux helpful for debugging IE-only issues under Linux. > > http://www.tatanka.com.br/ies4linux/page/Main_Page > > I don't recall, off hand, if it'll provide IE 7 (and the site isn't > working for me right now, so I can't confirm this). It does provide > IE 5.0, 5.5, and 6.0, however. You can get IE 7 if you go to the advanced option and choose it. They say it is experimental. I find it works well for when I have to test IE. I only test IE 6 and IE 7 to make sure web pages work in them as well. -- Boyd Gerber 801 849-0213 ZENEZ 1042 East Fort Union #135, Midvale Utah 84047 ``` ## Jacob Helwig, 2009-02-17 21:06 Subject: Re: [ANNOUNCE] Gerrit Code Review 2.0.3 Message-ID: <8c9a060902171306w5926c20ej439489de450e04f5@mail.gmail.com> URL: https://gitlist.dev/e/8c9a060902171306w5926c20ej439489de450e04f5%40mail.gmail.com In-Reply-To: ``` On Tue, Feb 17, 2009 at 08:44, Johannes Schindelin wrote: > Hi, > > On Tue, 17 Feb 2009, Jacob Helwig wrote: > >> On Tue, Feb 17, 2009 at 07:18, Shawn O. Pearce wrote: >> > Gerrit is known to not work in Internet Explorer. As an open source >> > developer, my primary desktop system runs Linux. My portable laptops >> > all run Mac OS X, because its shiny. I have no system capable of >> > running Internet Explorer, nor do I desire to send dollars towards >> > Redmond in order to obtain an OS license in order to do so. >> > >> > Something is busted in the IE AJAX code used by Gerrit Code Review. >> > I'm guessing that instead of making the data available to the AJAX >> > callback, IE wants to save it as a file on disk. No clue why. >> > And due to the lack of IE, I am unable to debug the problem myself. >> > I am waiting for someone who really cares about using IE to debug >> > it and contribute the patch back. >> > >> >> I've found IEs4Linux helpful for debugging IE-only issues under Linux. >> >> http://www.tatanka.com.br/ies4linux/page/Main_Page >> >> I don't recall, off hand, if it'll provide IE 7 (and the site isn't >> working for me right now, so I can't confirm this). It does provide >> IE 5.0, 5.5, and 6.0, however. > > If I understand Shawn (and the motto "scratch your own itch") correctly, > Shawn was not so much interested in being pointed to a way of running IE > himself, but more in people who absolutely want to run IE themselves to go > and fix the issues. > > Ciao, > Dscho > > Oh, I understand "scratch your own itch", but there's also a difference between unable to, and unwilling to. ;-) I wouldn't have questioned it, or said anything, if it was simply put as: "Known issue, but we don't use IE, so it's not a priority for us. Patches welcome." (Note: I'm not saying that Shawn should be willing to, just that he's not "unable" to. I also don't claim to know the full nuances of any possible licensing issues.) ``` ## thestar@fussycoder.id.au, 2009-02-18 00:13 Subject: Re: [ANNOUNCE] Gerrit Code Review 2.0.3 Message-ID: <20090218111356.uabo6aff6ss00kcg@webmail.fussycoder.id.au> URL: https://gitlist.dev/e/20090218111356.uabo6aff6ss00kcg%40webmail.fussycoder.id.au In-Reply-To: <8c9a060902171306w5926c20ej439489de450e04f5@mail.gmail.com> ``` Quoting Jacob Helwig : > On Tue, Feb 17, 2009 at 08:44, Johannes Schindelin > wrote: >> Hi, >> >> On Tue, 17 Feb 2009, Jacob Helwig wrote: >>> >>> I've found IEs4Linux helpful for debugging IE-only issues under Linux. >>> >>> http://www.tatanka.com.br/ies4linux/page/Main_Page >>> >>> I don't recall, off hand, if it'll provide IE 7 (and the site isn't >>> working for me right now, so I can't confirm this). It does provide >>> IE 5.0, 5.5, and 6.0, however. >> >> If I understand Shawn (and the motto "scratch your own itch") correctly, >> Shawn was not so much interested in being pointed to a way of running IE >> himself, but more in people who absolutely want to run IE themselves to go >> and fix the issues. >> >> Ciao, >> Dscho >> >> > > Oh, I understand "scratch your own itch", but there's also a > difference between unable to, and unwilling to. ;-) I wouldn't have > questioned it, or said anything, if it was simply put as: "Known > issue, but we don't use IE, so it's not a priority for us. Patches > welcome." > > (Note: I'm not saying that Shawn should be willing to, just that he's > not "unable" to. I also don't claim to know the full nuances of any > possible licensing issues.) I would've thought that impossible license situations would result in one being unable to use it. ```