Home | Docs | Issue Tracker | FAQ | Download | |
Date: | 2012/10/04 |
---|---|
Author: | Frank Warmerdam, Tom Kralidis and Thomas Bonfort |
Contact: | warmerdam at pobox.com |
Contact: | tomkralidis at hotmail.com |
Contact: | tbonfort at gmail.com |
Last Edited: | 2012/10/04 |
Status: | Draft |
Note
This RFC obsoletes MS RFC 7.1: MapServer SVN Commit Management.
To formalize Git push access, and specify guidelines for Git committers. More information on working with Git in MapServer can be found at https://github.com/mapserver/mapserver/wiki/WorkingWithGit.
Permission for Git push access shall be provided to new developers only if accepted by the MapServer 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 mapserver-dev is sufficient.
Removal of Git commit access should be handled by the same process.
The new committer should have demonstrated commitment to MapServer and knowledge of the MapServer source code and processes to the committee’s satisfaction, usually by reporting issues , submitting pull requests, and/or actively participating in the various MapServer forums.
The new committer should also be prepared to support any new feature or changes that he/she commits to the MapServer 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 mapserver-dev mailing list so they can stay informed on policies, technical developments and release preparation.
A list of all project committers will be managed in https://github.com/mapserver/mapserver/blob/master/COMMITERS with the following format:
One member of the Project Steering Committee will be designated the Git Administrator. That person will be responsible for giving Git commit access to folks, updating the COMMITERS file, and other Git related management.
Thomas Bonfort, Steve Lime and Daniel Morissette will be the Git Administrators.
The following are considered good Git commit practices for the MapServer project:
Committers are the front line gatekeepers to keep the code base clear of improperly contributed code. It is important to the MapServer 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 MapServer 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 MapServer 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:
TBD