[geomoose-psc] First draft of RFC4 - GeoMoose commit Management

Fri Nov 30 10:07:21 PST 2012

I've been able to change this to be appropriate for sphinx with minimal
changes (attached).

On Fri, Nov 30, 2012 at 8:52 AM, Basques, Bob (CI-StPaul) <
bob.basques at ci.stpaul.mn.us> wrote:

>  All,****
>
> ** **
>
> I started a draft of RFC4 – GeoMoose Commit Management.  This borrowed
> heavily from Mapserver RFC 7.1.****
>
> ** **
>
> The main reason for moving this along was the legal piece at the bottom of
> the RFC related to Code contributions and related Licensing management.
> This chunk is a Incubation required Item.****
>
> ** **
>
> Please review, edit and/or sign off on as you see fit.
>

I would not mind this being somewhat shorter and less detailed in some
aspects

> ****
>
> ** **
>
> There are a couple of outstanding pieces that I either didn’t know right
> off, or didn’t know how to grab or point at.  You can do a search on “??”
> in the txt to find these locations about where clarification is needed.
>

I think a lot of these areas can be removed.  Before it gets committed to
svn, I think it should have all the MS specific areas removed (configure.in,
Makefile.in, Makefile.vc).

> ****
>
> ** **
>
> NOTES: ****
>
> **·          **One item that may be a extra work item, is the need to
> include the COMMITTERS.txt file in the GeoMoose source tree.
>
Can you explain how
http://trac.osgeo.org/geomoose/browser/geomoose2/trunk/AUTHORS.rst does not
already cover this?  See also:
http://lists.osgeo.org/pipermail/geomoose-psc/2012-October/000034.html  and
http://trac.osgeo.org/geomoose/changeset/917

> ****
>
> **·          **Probably already in place, but there will need to be a
> designated Commit Administrator.****
>
> ** **
>
> I can start TRAC tickets if someone want to send me back titles for them.
>

I'm not sure that there is anything here that yet warrants a ticket.

Thanks for moving this along.

Eli

****
>
> ** **
>
> Thanks****
>
> ** **
>
> See attached.****
>
> ** **
>
> ** **
>
> _______________________________________________
> geomoose-psc mailing list
> geomoose-psc at lists.osgeo.org
> http://lists.osgeo.org/cgi-bin/mailman/listinfo/geomoose-psc
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osgeo.org/pipermail/geomoose-psc/attachments/20121130/6a9d3a7f/attachment-0001.html>
-------------- next part --------------
.. _rfc4:

RFC 4: GeoMOOSE Commit Management
====================================================================

:Author:  Bob Basques,  ( Heavy inspiration, and borrowing from `MapServer RFC 7.1 <http://mapserver.org/development/rfc/ms-rfc-7.1.html>`_)
:Contact: bob DOT basques at ci DOT stpaul DOT mn DOT us
:Status: development
:Last Updated: 11/30/2012

Purpose
-----------

To formalize commit access, and specify guidelines for committers.

Election to Commit Access
-------------------------

Permission for commit access shall be provided to new developers only if accepted by the GeoMoose Project Steering Committee. A proposal should be written to the PSC for new committers and voted on normally. It is not necessary to write an RFC document for these votes ... email to geomoose-psc is sufficient.

Removal of commit access should be handled by the same process.

The new committer should have demonstrated commitment to GeoMoose and knowledge of the GeoMoose source code and processes to the committee's satisfaction, usually by reporting tickets, submitting patches, and/or actively participating in the various GeoMoose forums.

The new committer should also be prepared to support any new feature or changes that he/she commits to the GeoMoose source tree in future releases, or to find someone to which to delegate responsibility for them if he/she stops being available to support the portions of code that he/she is responsible for.

All committers should also be a member of geomoose-psc mailing list so they can stay informed on policies, technical developments and release preparation.

Committer Tracking
-------------------------

A list of all project committers will be kept in the main GeoMoose directory (called COMMITTERS) with a listing for each committer:

    * Userid: the id that will appear in the logs for this person.
    * Full name: the users actual name.
    * Email address: A current email address at which the committer can be reached. It may be altered in normal ways to make it harder to auto-harvest.
    * A brief indication of areas of responsibility.

Commit Administrator
-------------------------

One member of the Project Steering Committee will be designated the Administrator. That person will be responsible for giving commit access to folks, updating the COMMITTERS file, and other related management. Initially ????? will be the Administrator.

Commit Practices
-------------------------

The following are considered good commit practices for the GeoMoose project.

    * Use meaningful descriptions for commit log entries.
    * Add a ticket reference like "(#1232)" at the end of commit log entries when committing changes related to a ticket in Trac.
    * Include changeset revision numbers like "r7622" in tickets when discussing relevant changes to the codebase.
    * Include an entry in the HISTORY file for any significant change or fix committed in the main GeoMoose source tree. Make sure it is placed under the correct version heading and include ticket numbers in these messages too.
    * Changes should not be committed in stable branches without a corresponding ticket and HISTORY entry. Any change worth pushing into the stable version is worth a Trac ticket and good HISTORY notations.
    * Never commit new features to a stable branch: only critical fixes. New features can only go in the main development trunk.
    * Only ticket defects should be committed to the code during pre-release code freeze.
    * Significant changes to the main development version should be discussed on the -pcs list before you make them, and larger changes will require an RFC approved by the PSC.
    * Do not create new branches without the approval of the PSC. Release managers are assumed to have permission to create a branch.
    * All source code should be in Unix text format as opposed to DOS text mode.
    * When committing new features or significant changes to existing source code, the committer should take reasonable measures to insure that the source code continues to build and work on the most commonly supported platforms (currently Linux and Windows), either by testing on those platforms directly, or by getting help from other developers working on those platforms. If new files or library dependencies are added, then the ??configure.in, ??Makefile.in, ??Makefile.vc and related documentations should be kept up to date.

Legal
-------------------------

Commiters are the front line gatekeepers to keep the code base clear of improperly contributed code. It is important to the GeoMoose users, developers and the OSGeo foundation to avoid contributing any code to the project without it being clearly licensed under the project license.

Generally speaking the key issues are that those providing code to be included in the repository understand that the code will be released under the GeoMoose License, and that the person providing the code has the right to contribute the code. For the committer themselves understanding about the license is hopefully clear. For other contributors, the committer should verify the understanding unless the committer is very comfortable that the contributor understands the license (for instance frequent contributors).

If the contribution was developed on behalf of an employer (on work time, as part of a work project, etc) then it is important that an appropriate representative of the employer understand that the code will be contributed under the GeoMoose License. The arrangement should be cleared with an authorized supervisor/manager, etc.

The code should be developed by the contributor, or the code should be from a source which can be rightfully contributed such as from the public domain, or from an open source project under a compatible license.

All unusual situations need to be discussed and/or documented.

Committers should adhere to the following guidelines, and may be personally legally liable for improperly contributing code to the source repository:

    * Make sure the contributor (and possibly employer) is aware of the contribution terms.
    * Code coming from a source other than the contributor (such as adapted from another project) should be clearly marked as to the original source, copyright holders, license terms and so forth. This information can be in the file headers, but should also be added to the project licensing file if not exactly matching normal project licensing (GeoMoose/LICENSE.txt).
    * Existing copyright headers and license text should never be stripped from a file. If a copyright holder wishes to give up copyright they must do so in writing to the foundation before copyright messages are removed. If license terms are changed it has to be by agreement (written in email is ok) of the copyright holders.
    * When substantial contributions are added to a file (such as substantial patches) the author/contributor should be added to the list of copyright holders for the file.
    * If there is uncertainty about whether a change it proper to contribute to the code base, please seek more information from the project steering committee, or the foundation legal counsel.

Voting History
-------------------------