Process for integrating outside patches into ilisp

"Clementson, Bill" <[email protected]>
Newsgroups gmane.lisp.ilisp.devel
Message-ID <[email protected]>
I've been on vacation the past few weeks and have been slowly catching
up on my backlog of emails. One thing that I've noticed is that, over
the past month, Alain Picard, Thomas Burdick (in ilisp-help), and Adam
Warner have submitted patches to ilisp; however, they don't appear to
have received any response from the ilisp developer list nor have their
patches been incorporated into cvs. Since I'm a relatively new
participant to open source projects, could someone tell me what the
"standard" process is for incorporating patches from programmers who
aren't registered as project developers? One of the advantages
(potentially) of open source efforts is that many eyes are looking at
the source code and patches can come from individuals who aren't
directly associated with the project - that advantage is negated if
there is no process for incorporating patches from others.

If there is no "standard" process in place for ilisp, I would like to
propose the following:

1. Let's nominate a "gatekeeper" who will be responsible for routing
patches to registered developers and providing an initial response to
the patch submitter (official response should be on ilisp-devel so
people know something is happening with the patch but routing & initial
comments could be done by email to avoid cluttering ilisp-devel).
2. Volunteers will be responsible for specific "types" of reviews (e.g.
-- I work with ilisp/clisp on Windows - I would be willing to review
generic ilisp functionality patches, clisp-specific patches or
windows-specific patches; however, I could not verify cmucl or linux
patches)
3. Some degree of patch testing will be done by the reviewer (we all
have day jobs so this may not be very comprehensive; however, at least
some testing should be done) and the patch (if approved) will be
submitted to cvs. If the patch is inadequate or addresses something that
the reviewer thinks is out-of-scope for inclusion, the patch will be
rejected.
4. Notice of the acceptance/rejection of the patch will be posted on
ilisp-devel

How does this sound? If there is some alternative procedure, I am quite
happy to use that; however, I would hate to have patches fall through
the cracks just because we have no procedure in place at all.

I would be willing to act as "gatekeeper" and would volunteer to review
some generic ilisp patches as well as any clisp-specific patches or
windows-specific patches. Who could review cmucl, LW, acl and linux
specific patches?

--
Bill Clementson


-------------------------------------------------------
This SF.Net email is sponsored by: INetU
Attention Web Developers & Consultants: Become An INetU Hosting Partner.
Refer Dedicated Servers. We Manage Them. You Get 10% Monthly Commission!
INetU Dedicated Managed Hosting http://www.inetu.net/partner/index.php
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.