[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Avoid `SCM_VALIDATE_LIST ()'
From: |
Han-Wen Nienhuys |
Subject: |
Re: [PATCH] Avoid `SCM_VALIDATE_LIST ()' |
Date: |
Sun, 07 Sep 2008 16:25:43 -0300 |
User-agent: |
Thunderbird 2.0.0.16 (X11/20080723) |
Neil Jerram escreveu:
> Since you mention 'cleanups', I must say that I agree with Ludovic,
> that it would have been preferable to post the patch for
> review/discussion before committing it, since that is our (majority)
> current practice. Sure there may have been a few exceptions, but only
> for trivial changes, I believe, and I don't believe that this was -
> overall - a trivial change. (I'm aware that it has lots of trivial
> bits in it, but I don't think it's all trivial.)
>
> (I also think it's arguable that actually committing to a branch is
> more convenient, for author and reviewers, than juggling emails - but
> that then leads on to other questions, like what expectations people
> can have of the "master" branch, and why we are using Git like CVS...)
If we want to code review as a policy, I'm fine with that, but let's do
it in a structured way then.
Guido van Rossum wrote a webtool for code review, which should be fairly easy
to use and setup
http://code.google.com/appengine/articles/rietveld.html
and I heard people have recently added Git support to it.
How about using this for Guile? (We need to figure out how to set it up, and
contributors need to have gmail addresses, but that should not be a problem,
is it?)
--
Han-Wen Nienhuys - address@hidden - http://www.xs4all.nl/~hanwen
- Re: [PATCH] Avoid `SCM_VALIDATE_LIST ()', (continued)
Re: [PATCH] Avoid `SCM_VALIDATE_LIST ()', Andy Wingo, 2008/09/04
- Re: [PATCH] Avoid `SCM_VALIDATE_LIST ()', Han-Wen Nienhuys, 2008/09/04
- Re: [PATCH] Avoid `SCM_VALIDATE_LIST ()', Andy Wingo, 2008/09/04
- Re: [PATCH] Avoid `SCM_VALIDATE_LIST ()', Neil Jerram, 2008/09/06
- Re: [PATCH] Avoid `SCM_VALIDATE_LIST ()', Han-Wen Nienhuys, 2008/09/06
- Re: [PATCH] Avoid `SCM_VALIDATE_LIST ()', Neil Jerram, 2008/09/07
- Re: [PATCH] Avoid `SCM_VALIDATE_LIST ()', Han-Wen Nienhuys, 2008/09/07
- Re: [PATCH] Avoid `SCM_VALIDATE_LIST ()', Neil Jerram, 2008/09/07
- Re: [PATCH] Avoid `SCM_VALIDATE_LIST ()',
Han-Wen Nienhuys <=
Re: [PATCH] Avoid `SCM_VALIDATE_LIST ()', Andy Wingo, 2008/09/07
Re: development goals (was: [PATCH] Avoid `SCM_VALIDATE_LIST ()'), Han-Wen Nienhuys, 2008/09/07
Re: development goals (was: [PATCH] Avoid `SCM_VALIDATE_LIST ()'), Neil Jerram, 2008/09/07
Re: development goals, Han-Wen Nienhuys, 2008/09/08
Re: development goals, Ludovic Courtès, 2008/09/08
Re: development goals, Han-Wen Nienhuys, 2008/09/08
Re: development goals, Andy Wingo, 2008/09/09
Re: development goals, Ludovic Courtès, 2008/09/08
Re: [PATCH] Avoid `SCM_VALIDATE_LIST ()', Neil Jerram, 2008/09/06