Re: git review
Hi David, David Tardon píše v Čt 09. 08. 2012 v 07:17 +0200: We will loose possible contributors that way. Not an option. Patch submittal has to work hasslefree and out of the box. This is really critical: there has to be no extra step at all for patch submittal otherwise we failed. irony It has worked hasslefree and out of the box while we have been using email. /irony It was a precondition for the gerrit work that the email submission + direct push must still be possible, and I for one will make sure this is still so ;-) Regards, Kendy ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
Re: git review
On Thu, Aug 09, 2012 at 09:15:48AM +0200, Jan Holesovsky wrote: David Tardon píše v Čt 09. 08. 2012 v 07:17 +0200: irony It has worked hasslefree and out of the box while we have been using email. /irony It was a precondition for the gerrit work that the email submission + direct push must still be possible, and I for one will make sure this is still so ;-) Agreed. gerrit is a platform and it should integrate all kind of workflows whether they are email-based, web-based, CLI-based, IRC-based or whatever. That doesnt mean the 'gerrit guys' will implement every of your pet features there, but we have to make the basic workflow available there. Best, Bjoern ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
Re: git review
On Wed, Aug 08, 2012 at 11:52:03PM +0200, Bjoern Michaelsen wrote: Like described in link above, you have three options here: pypi-system-wide, pypi-user-local and distro package. No other options please. Most people wont even know what pypi-system-wide is and dont want to know. And they shouldnt need to. We will loose possible contributors that way. Not an option. Patch submittal has to work hasslefree and out of the box. This is really critical: there has to be no extra step at all for patch submittal otherwise we failed. Thinking a bit about this, another possibility would be to do with git-review as we do with other external stuff: Downloading and installing a local copy in ./download. That would ensure it to be universally available and be up to date. Best, Bjoern ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
Re: git review
On Thu, Aug 9, 2012 at 7:04 AM, Bjoern Michaelsen bjoern.michael...@canonical.com wrote: On Wed, Aug 08, 2012 at 11:52:03PM +0200, Bjoern Michaelsen wrote: Like described in link above, you have three options here: pypi-system-wide, pypi-user-local and distro package. No other options please. Most people wont even know what pypi-system-wide is and dont want to know. And they shouldnt need to. We will loose possible contributors that way. Not an option. Patch submittal has to work hasslefree and out of the box. This is really critical: there has to be no extra step at all for patch submittal otherwise we failed. Thinking a bit about this, another possibility would be to do with git-review as we do with other external stuff: Downloading and installing a local copy in ./download. That would ensure it to be universally available and be up to date. I know that michael disagree with me on that, but I prefer dev-tools to be in ... dev-tools.git one can install it as he see fit. and you get the version you want/need regardless where you are in the source tree... Norbert ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
Re: git review
On Thu, Aug 09, 2012 at 07:55:45AM -0500, Norbert Thiebaud wrote: I know that michael disagree with me on that, but I prefer dev-tools to be in ... dev-tools.git one can install it as he see fit. and you get the version you want/need regardless where you are in the source tree... Well, putting it in ./download would be even better, as it would be living upstream at openstack. Best, Bjoern ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
Re: git review
Quoting Norbert Thiebaud nthieb...@gmail.com: On Thu, Aug 9, 2012 at 7:04 AM, Bjoern Michaelsen Thinking a bit about this, another possibility would be to do with git-review as we do with other external stuff: Downloading and installing a local copy in ./download. That would ensure it to be universally available and be up to date. I know that michael disagree with me on that, but I prefer dev-tools to be in ... dev-tools.git one can install it as he see fit. and you get the version you want/need regardless where you are in the source tree... Well to put it in dev-tool is much less painfull as to put it elsewhere. Note this tool must be in your PATH! For all branches. And even if you switch the branches and no matter what your current directory is, you must be able to say: git review --dry-run that it. But who is the person who will put it in dev-tools once and update it all the time? (Last time i contributed to git-review was today morning). But even if that person (not me) or some cron jobs continuously synchronize it, the user must still update it. So the user must now periodically update dev-tool to get the fresh version of git-review? But then what is the difference to say git pull dev-tools/git-review or sudo apt-get update upgrade (place here your distro command) or (*) sudo pip install git-review sudo pip install --upgrade git-review sudo pip uninstall git-review (*) http://www.pip-installer.org/en/latest/index.html Regards David ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
Re: git review
On Thu, Aug 9, 2012 at 8:11 AM, Bjoern Michaelsen bjoern.michael...@canonical.com wrote: On Thu, Aug 09, 2012 at 07:55:45AM -0500, Norbert Thiebaud wrote: I know that michael disagree with me on that, but I prefer dev-tools to be in ... dev-tools.git one can install it as he see fit. and you get the version you want/need regardless where you are in the source tree... Well, putting it in ./download would be even better, as it would be living upstream at openstack. no it would not, because if I swith to branch 3-5... what version do I have then ? not to mention have n-copy of it (I do have typically half-a dozen of clone of core...)... yeah the src directory can be share... still. I tend to do make clean a lot... so I would have to rebuild it every time to make sure it is there when I need it ? Norbert ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
Re: git review
On Thu, Aug 09, 2012 at 08:57:01AM -0500, Norbert Thiebaud wrote: no it would not, because if I swith to branch 3-5... what version do I have then ? If you switch -- the same. ./download would install it into solenv/bin, which is not changed by git checkout. Also I would suggest to backport that to all branches -- it wont affect the build anyway, so should be easy to review. not to mention have n-copy of it (I do have typically half-a dozen of clone of core...)... yeah the src directory can be share... still. I tend to do make clean a lot... so I would have to rebuild it every time to make sure it is there when I need it ? Well, I doubt that small script is the right place to start to look of build dir size. Even killing autodoc will help a lot more. Best, Bjoern ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
Re: git review
Hi Bjoern, On 02.08.2012 23:21, Bjoern Michaelsen wrote: http://wiki.documentfoundation.org/Development/GitReview so: - what are the experiences? - should we recommend it in general? I am really happy with git review. It has killer features that i don't want to miss anymore, in fact they saved my life (and some other guys too ,-) To name the few: 1. called with git review -s (once) it does all the setup for your. It even checks if you have commit-msg hook in place and download and install one, if you don't! 2. it checks if your commit has Change-Id and ammend to recrate it, in case you commited before installing commit-msg hook. 3. it has --dry-run option and shows you what it would do without actually doing anything (like git push command) 4. and last but not least it really save your life by checking how much commits your are going to upload and warn and ask for confirmation if the count is 1. [...] I would suggest, if possible, to dump a copy that can work standalone into the core repo root. Even if it might be outdated compared to openstack its likely better in the long run than ./logerrit for submittal. But IMHO git-review MUST work out of the box then -- without any setup. No! You don't dump a copy of git, bash and binutils, don't you? Like described in link above, you have three options here: pypi-system-wide, pypi-user-local and distro package. No other options please. [...] (*) plus patching to provide something like ./logerrit resubmit i just uploaded the git-review patch: https://review.openstack.org/#/c/11046/ So the simultaneous submissions to release branch and direct push to master should be possible now (once openstack guys apporved my change). And hej, my patch in git-review was shorter then your in logerrit ;-) Regards David ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
Re: git review
On Wed, Aug 08, 2012 at 11:19:09PM +0200, David Ostrovsky wrote: But IMHO git-review MUST work out of the box then -- without any setup. No! You don't dump a copy of git, bash and binutils, don't you? Because they are install with ~all systems and are available on all system as a nicely maintained package. Thats is not the case for git-review yet. Besides, it is small and we should have in our repos anyway for safety when it becomes part of our infrastructure. Besides we _have_ copies of boost, glib, expat and openssl in our repos for valid reasons. Like described in link above, you have three options here: pypi-system-wide, pypi-user-local and distro package. No other options please. Most people wont even know what pypi-system-wide is and dont want to know. And they shouldnt need to. We will loose possible contributors that way. Not an option. Patch submittal has to work hasslefree and out of the box. This is really critical: there has to be no extra step at all for patch submittal otherwise we failed. Best, Bjoern ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
Re: git review
Hi, On Wed, Aug 08, 2012 at 11:52:03PM +0200, Bjoern Michaelsen wrote: On Wed, Aug 08, 2012 at 11:19:09PM +0200, David Ostrovsky wrote: Like described in link above, you have three options here: pypi-system-wide, pypi-user-local and distro package. No other options please. Most people wont even know what pypi-system-wide is and dont want to know. And they shouldnt need to. We will loose possible contributors that way. Not an option. Patch submittal has to work hasslefree and out of the box. This is really critical: there has to be no extra step at all for patch submittal otherwise we failed. irony It has worked hasslefree and out of the box while we have been using email. /irony D. ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice