Difference between revisions of "Library submissions"

From HaskellWiki
Jump to: navigation, search
(updating maintainer of record for vector and primitive and random)
 
(59 intermediate revisions by 12 users not shown)
Line 1: Line 1:
As the Haskell community has grown, and emphasis on development has
 
  +
This page describes the process followed by the [http://www.haskell.org/haskellwiki/Core_Libraries_Committee Core Libraries Committee], or CLC, for maintaining the libraries it oversees. Many of these libraries are '''core libraries''', defined as libraries which ship with GHC itself, which is the source of the name for this committee. However, over the years, other libraries have been subsumed under CLC maintainership.
moved from language to libraries, the need for a formal process
 
for making changes to core libraries has emerged.
 
   
The concern behind this process is:
 
  +
Generally speaking, all libraries maintained by the CLC are "important" for some reason or another. The core libraries themselves are particularly important since they ship with GHC, and are generally used by a large subset of the community (e.g., base is used by virtually all Haskell code).
   
* to ensure on-going improvement to critical, shared code;
 
  +
= Goals =
* with appropriate levels of oversight and assurance.
 
   
This page documents the mechanism by which new code for libraries maintained by the community may be proposed. This mechanism has been in place since October 2006.
 
  +
Our goals for maintenance of these libraries are:
   
== Creating a proposal ==
 
  +
* Bugs are resolved
  +
* Feature requests can be implemented
  +
* Patches can be merged
  +
* All of this is done in such a way as to avoid creating problems for users of these libraries
   
In order to ensure we have something concrete to discuss, please follow
 
  +
The last point is vital. These libraries must have the highest quality
the following guidelines:
 
  +
standards of any libraries in the Haskell ecosystem. In order to ensure timely
  +
response to issues (the first three points) while maintaining the quality
  +
standards of the fourth point, we have a dual maintainership system in place.
   
* ''Currency''. Make your changes against a copy of the HEAD branch of the [http://hackage.haskell.org/trac/ghc/wiki/DarcsRepositories relevant library], and make sure it compiles.
 
  +
Each library has an individual maintainer for routine maintenance of the
* ''Portability''. Code should be portable. If it is not portable, reasons should be given. At the very least ensure the code runs in Hugs and GHC, and on Windows and Linux.
 
  +
package. In addition, the
* ''Style''. Follow the conventions in the library you are modifying.
 
  +
[http://www.haskell.org/haskellwiki/Core_Libraries_Committee Core Libraries Committee], or CLC, has oversight on the maintenance of these libraries.
* ''Documentation''. It must include valid [http://haskell.org/haddock Haddock] documentation.
 
* ''Tests''. Code should ideally also come with suitable tests for the testsuite. There's currently some disagreement about what this means. Discussion of where we may want to head is in the [[library tests]] page.
 
   
== Submitting the proposal ==
 
  +
The individual maintainer is responsible for dealing with each individual issue
  +
against that library. That individual is fully empowered to make decisions on
  +
that library. However, the following restrictions apply:
   
* ''Patch''. Create a [http://darcs.net darcs] patch of your change using <tt>darcs record</tt>, including a rationale for the change. Save the patch to a file, using <tt>darcs send --output</tt>.
 
  +
* For "controversial" decisions (definition discussed below), the maintainer should not make a unilateral decision, but rather discuss with the CLC as a whole (and possibly a wider audience)
  +
* If the CLC is dissatisfied with the job being performed by the maintainer, it may remove the maintainer from his/her role (via a majority vote)
   
* ''Tracking''. [http://hackage.haskell.org/trac/ghc/newticket?type=proposal&component=libraries/base&milestone=Not+GHC Add a Trac ticket] of type ''proposal'' for the appropriate library component, with a timescale for consideration (to focus the community's attention for a limited period), adding the patch file as an attachment.
 
  +
'''NOTE''': The oversight role of the CLC only applies to libraries which have explicitly been placed under the CLC's maintainership. Other libraries are encouraged to follow this practice, but are not required to do so. Currently, the libraries which are not maintained by the CLC are containers, parallel, pretty, time, and xhtml.
   
* ''Submission''. Send an email containing the Trac ticket number and description to libraries@haskell.org, which you need to [http://www.haskell.org/mailman/listinfo/libraries subscribe] before posting. (In the future this should happen when you record the ticket, but we haven't set it up yet.) You may wish to include a pointer to updated Haddock documentation, if relevant.
 
  +
The list of all libraries is provided below.
   
If someone has done all this, they are entitled to expect feedback;
 
  +
While this page only covers maintenance of CLC-maintained libraries, the policies
silence during the discussion period can be interpreted as consent.
 
  +
described may be useful for other libraries, especially for packages
  +
in the Haskell Platform.
   
Here are the [http://hackage.haskell.org/trac/ghc/query?status=new&status=assigned&status=reopened&group=component&type=proposal&order=priority proposals currently under consideration].
 
  +
= The Libraries =
   
== At the end of the discussion period ==
 
  +
Our definition of "core library" is a library that ships with GHC. This can happen for one of three reasons:
   
* Add to the ticket a summary of the relevant parts of the discussion. (The summary is needed for anyone wondering about the change later: it's not reasonable to point people at a 50-message thread (but please do include a link to the thread in the list archives too, so that people can review it if they wish)).
 
  +
# It is part of the Haskell language standard
* If consensus was achieved, revise the patch as necessary, including the Trac ticket number, and commit it (or set the ticket state to "patch" so that someone will commit it for you).
 
  +
# It is tightly coupled to GHC’s implementation of some feature. These libraries cannot be changed independently of GHC. You could think of them as part of GHC, part that happens to be implemented in library code.
* Close the ticket (usually as ''fixed'' or ''wontfix'').
 
  +
# It is used to build GHC itself. We have to ship these libraries with GHC because ghc itself is a package, and so we must have installed packages for anything the ghc package depends on.
   
A deeply held disagreement at this point may require some form of governance (voting, dictatorship, etc). This should be a rare event.
 
  +
The following table lists the libraries maintained by the CLC, their primary maintainers, issue tracker for filing bugs and feature requests, and the reason the library is a member of the core libraries (one of the three reasons above, or blank if the library is a non-core library). Ideally, all libraries listed will have a maintainer as well, though we have not filled all roles yet.
   
Here are the [http://hackage.haskell.org/trac/ghc/query?status=closed&group=component&type=proposal&order=priority archived past proposals].
 
  +
{| border="1"
  +
! Package !! Maintainer !! Issue Tracker !! Reason
  +
|-
  +
| [http://hackage.haskell.org/package/array array] || Dan Doel || [https://ghc.haskell.org/trac/ghc/newticket?component=libraries%20%28other%29&keywords=array GHC Trac] || 1, 3
  +
|-
  +
| [http://hackage.haskell.org/package/base base] || Edward Kmett and Ryan Scott || [https://ghc.haskell.org/trac/ghc/newticket?component=libraries%20%28base GHC Trac] || 1, and partly 2
  +
|-
  +
| [http://hackage.haskell.org/package/deepseq deepseq] || Herbert Valerio Riedel || [https://github.com/haskell/deepseq/issues Github] || 3
  +
|-
  +
| [http://hackage.haskell.org/package/directory directory] || Phil Ruffwind and Elliot Robinson|| [https://github.com/haskell/directory/issues Github] || 3
  +
|-
  +
| [http://hackage.haskell.org/package/filepath filepath] || Neil Mitchell || [https://github.com/haskell/filepath/issues Github] || 3
  +
|-
  +
| [http://hackage.haskell.org/package/ghc-prim ghc-prim] || GHC HQ || [http://ghc.haskell.org/trac/ghc/newticket?component=libraries%20%28other%29&keywords=ghc-prim GHC Trac] || 2
  +
|-
  +
| [http://hackage.haskell.org/package/hpc hpc] || GHC HQ || [http://ghc.haskell.org/trac/ghc/newticket?component=Code%20Coverage GHC Trac] || 2
  +
|-
  +
| [http://hackage.haskell.org/package/mtl mtl] || Edward Kmett || [https://github.com/ekmett/mtl/issues Github] || non-core
  +
|-
  +
| [http://hackage.haskell.org/package/primitive primitive] || Carter Schonwald || [https://github.com/haskell/primitive/issues Github] || non-core
  +
|-
  +
| [http://hackage.haskell.org/package/process process] || Michael Snoyman || [https://github.com/haskell/process/issues Github] || 3
  +
|-
  +
| [http://hackage.haskell.org/package/random random] || Carter Schonwald || [https://github.com/haskell/random/issues Github] || non-core
  +
|-
  +
| [http://hackage.haskell.org/package/stm stm] || GHC HQ || [https://ghc.haskell.org/trac/ghc/newticket?component=libraries%20%28other%29&keywords=stm GHC Trac] || 2
  +
|-
  +
| [http://hackage.haskell.org/package/template-haskell template-haskell] || Edward Kmett || [http://ghc.haskell.org/trac/ghc/newticket?component=Template%20Haskell GHC Trac] || 2
  +
|-
  +
| [http://hackage.haskell.org/package/unix unix] || Herbert Valerio Riedel and Elliot Robinson || [https://github.com/haskell/unix/issues Github] || 3
  +
|-
  +
| [http://hackage.haskell.org/package/vector vector] || Carter Schonwald || [https://github.com/haskell/vector/issues Github] || non-core
  +
|-
  +
| [http://hackage.haskell.org/package/old-locale old-locale] || Herbert Valerio Riedel || [https://github.com/haskell/old-locale/issues Github] || non-core
  +
|-
  +
| [http://hackage.haskell.org/package/old-time old-time] || Herbert Valerio Riedel || [https://github.com/haskell/old-time/issues Github] || non-core
  +
|-
  +
| [http://hackage.haskell.org/package/Win32 Win32] || [https://ghc.haskell.org/trac/ghc/wiki/WindowsTaskForce Windows Task Force] || [https://github.com/haskell/win32/issues Github] || 3
  +
|-
  +
|}
   
Here is an example of how to [http://hackage.haskell.org/trac/ghc/ticket/966 summarise a successful submission].
 
  +
The maintainer "GHC HQ" means Simon Marlow, Simon Peyton Jones, Herbert Valerio Riedel and Ben Gamari. Daniel Fischer has taken responsibility for numeric stuff. Email ghc-devs@haskell.org
   
== See also ==
 
  +
The [http://www.haskell.org/haskellwiki/Core_Libraries_Committee Core Libraries Committee] was formed to provide a maintainer of many of the packages previously managed by GHC HQ and can be reached by emailing core-libraries-committee@haskell.org or publicly through libraries@haskell.org.
  +
  +
There is no requirement that a library maintainer be him/herself a member of the CLC.
  +
  +
If you believe an additional library should be part of the CLC library set, or
  +
wish to take over as maintainer on one of the libraries above, please send an
  +
email to the libraries@ mailing list with the request.
  +
  +
The following packages match the appropriate language standard, and as such cannot change independently. The code is maintained by the GHC team.
  +
  +
{| border="1"
  +
|-
  +
| [http://hackage.haskell.org/package/haskell2010 haskell2010] || GHC HQ
  +
|-
  +
| [http://hackage.haskell.org/package/haskell98 haskell98] || GHC HQ
  +
|}
  +
  +
The following packages are core libraries in that they ship with GHC, but are not maintained by the core libraries committee.
  +
  +
{| border="1"
  +
! Package !! Maintainer !! Issue Tracker !! Reason
  +
|-
  +
| [http://hackage.haskell.org/package/containers containers] || David Feuer || [https://github.com/haskell/containers/issues Github] || 3
  +
|-
  +
| [http://hackage.haskell.org/package/hoopl hoopl] || Ning Wang || [https://github.com/haskell/hoopl/issues Github] || 3
  +
|-
  +
| [http://hackage.haskell.org/package/parallel parallel] || Simon Marlow || [https://github.com/haskell/parallel/issues Github] || non-core ('''NOTE''': Should we remove this library from the table?)
  +
|-
  +
| [http://hackage.haskell.org/package/pretty pretty] || David Terei || [https://github.com/haskell/pretty/issues Github] || 3
  +
|-
  +
| [http://hackage.haskell.org/package/time time] || Ashley Yakeley || [https://github.com/haskell/time/issues Github] || 3
  +
|-
  +
| [http://hackage.haskell.org/package/xhtml xhtml] || Chris Dornan || [https://github.com/haskell/xhtml/issues Github] || 3
  +
|-
  +
|}
  +
  +
= General workflow =
  +
  +
# Issues should be filed against the appropriate issue tracker for that library.
  +
#* Note: In the past, core library issues were all made against GHC Trac. That is no longer the case, and issues opened in such a way will likely be closed with a request to reopen in the appropriate place.
  +
# If any member of the committee feels particularly interested or well qualified in an individual issue, and the official maintainer has not started working on the issue yet, he/she should feel free to self-assign the issue. Similarly, if the official maintainer feels unqualified to handle an issue, he/she should assign to another individual (whether part of the CLC or not).
  +
# There must be a clear status for issues at all time, and this status should be understandable from the Trac discussion itself. Side channels of communication must be recorded in Trac (or at least referenced from there). The community should be able to read a discussion, understand current status, and also have a clear idea of how to move the ticket forward towards resolution. (Note that this requirement is very similar to [https://github.com/commercialhaskell/commercialhaskell/blob/master/guide/maintainer-communication.md the commercial SIG maintainer communication guide].)
  +
# Any individual should feel empowered to assign unassigned or misassigned issues to the appropriate maintainer.
  +
# Ultimately, it is the responsibility of the committee chair to ensure that this process is continuing to run, and that issues are being addressed and resolved quickly and professionally. The chair may of course employ the assistance of others in this endeavor.
  +
  +
Since this has been a difficult point in the past, is bears repeating: '''an individual maintainer is fully empowered to make decisions, and should do so!''' Only controversial decisions should involve the rest of the CLC.
  +
  +
= Responsiveness =
  +
  +
* Third parties submitting proposals to the maintainer of a library can expect a timely and thoughtful response.
  +
* The more effort the proposer invests (for example, by constructing a patch rather than making an off-the-cuff suggestion) the more consideration s/he can reasonably expect.
  +
* Ideally, no issues will be left without a response for more than one week, with two weeks being a maximum. If a maintainer will be on vacation or otherwise unavailable, he/she should notify the rest of the committee beforehand.
  +
* If there is more than a week's delay on an issue from the maintainer's side, it is acceptable to ping him/her.
  +
* If there is more than a week's delay on an issue from the issue creator's side, the maintainer should feel free to close the issue if desired, with a comment explaining why it was closed.
  +
* Proposals that have widespread support, and are accompanied by patches (preferably with tests and documentation), should normally be accepted by the maintainer.
  +
* It is up to the maintainer to decide what "widespread" means; in particular, it does not always mean "a majority of those who responded". The majority-responder story is vulnerable to selection bias; e.g. 7 people (out of a client base of hundreds) say "add this function" but the maintainer thinks it will make the interface incrementally more complicated without sufficient benefit.
  +
* Where there is significant work involved in implementing a proposal, it is reasonable for a maintainer to ask for a patch. The principle is that maintainers are not obliged to do the work of implementing a proposal, even if it does enjoy wide support. For more substantial changes, it makes sense to develop the implementation in dialogue with the maintainer.
  +
  +
= Mistakes =
  +
  +
People are fallible. The process above will allow some mistakes to be made.
  +
That's a conscious trade-off we're accepting now. If we had unlimited time and
  +
unlimited manpower, we may decide to discuss in detail each and every issue.
  +
However, that's not our reality: everyone involved has other responsibilities
  +
and does not have time or inclination to discuss in depth each issue.
  +
  +
Said another way: let's make progress with 95% accuracy rather than get bogged
  +
down in gridlock with 99% accuracy.
  +
  +
= Controversial decisions =
  +
  +
It's impossible in this document to record all possible forms of a
  +
controversial decision. Here are some basic guidelines:
  +
  +
* Adding a new, useful function under a clear name is probably not controversial
  +
* Breaking the API in any way is almost certainly controversial
  +
* Changing the semantics of an existing function is almost certainly controversial
  +
* Changes affecting Prelude and the Haskell Report are almost certainly controversial
  +
  +
It is important that, at the very least, the CLC receives an email on this
  +
topic. The individual maintainer is still in charge of the issue, but should
  +
give the CLC a chance to express an opinion. If after a reasonable amount of
  +
time (let's say one week) no objections are raised, the maintainer should feel
  +
free to move ahead with the change.
  +
  +
The CLC should repeat this decision making process again with more
  +
controversial issues. The prime example of this was the "burning bridges
  +
proposal": a change which drastically impacts <code>Prelude</code> should
  +
almost certainly be elevated to the libraries@ mailing list. Anything going to
  +
libraries@ should have at least a two week deadline on it.
  +
  +
At the end of the day, the CLC is still the body with decision making power.
  +
Explicitly: no single person should ever have veto power on a change. That
  +
includes the library maintainer; if the CLC votes against the library
  +
maintainer, that decision will stand. The CLC may itself decide to request
  +
support from others to make a decision, but is never required to do so.
  +
  +
= Guide to proposers =
  +
  +
Previously (before February 2015), the initiative remained with the proposer
  +
for quite a while on the proposal process. The process described here is
  +
significantly different, so please note the change.
  +
  +
All library proposals should start on the relevant issue tracker. This may be a
  +
bug report, feature request, or even a complete patch/pull request. At this
  +
point, the library maintainer is responsible for taking next steps. Some
  +
example next steps would be to accept the patch, implement a bug fix, reject a
  +
feature request, or decide that this is a controversial decision that must be
  +
discussed with the CLC.
  +
  +
If the CLC decides that the discussion must be discussed with the libraries@
  +
mailing list, the original proposer may be asked to moderate the libraries@
  +
mailing list discussion. The general format for this is:
  +
  +
* Send your proposal by email to the libraries@haskell.org mailing list (which you need to [http://www.haskell.org/mailman/listinfo/libraries subscribe] to before posting).
  +
  +
* Set a deadline for discussion (no less than two weeks), and act as chair/moderator for the discussion.
  +
  +
* Following discussion, it's the responsibility of the core libraries committee to determine if the proposal should be accepted. Send an email to the core libraries committee requesting a decision, optionally including your own summary of the mailing list discussion.
  +
  +
* From that point forward, the CLC and maintainer will again be responsible for making a decision on how to proceed, and will inform the proposer and the mailing list. Possible decisions include acceptance, rejection, and requesting a patch be provided.
  +
  +
Here are desirable properties for a proposal and its implementation. The more of these properties your proposal or patch has, the more likely it is that the maintainer will adopt your idea. After all, to adopt it the maintainter will have to do whatever tasks you didn't do.
  +
  +
* '''Description'''. A good proposal says clearly ''what'' you propose, ''why'' it is a good idea, and what its consequences would be.
  +
* '''Patch'''. Use <tt>darcs record</tt> or <tt>git commit</tt> (depending on what sort of repo the library lives in) to create it. Save the patch to a file, using <tt>darcs send --output</tt> or <tt>git format-patch</tt>. Make your changes against a copy of the master branch of the relevant library, and make sure it compiles.
  +
* '''Portability'''. Good code is portable. In particular, try to ensure the code runs in Hugs and GHC, and on Windows and Linux.
  +
* '''Style'''. Good code follows the conventions in the library you are modifying.
  +
* '''Documentation'''. Good code includes valid [http://haskell.org/haddock Haddock] documentation.
  +
* '''Tests'''. Good patches have suitable tests for the library's testsuite.
  +
  +
= Guidance for maintainers =
  +
  +
The principle is that we '''trust the maintainer''' to behave sensibly. The guidelines below are just that: guidelines, not rules. Still, the core libraries are used by many, many people, so maintainers should make every effort not to mess them up by accident.
  +
  +
* '''API changes should be discussed''' on the libraries mailing list prior to making the change, even if the maintainer is the proposer. The maintainer still has ultimate say in what changes are made, but the community should have the opportunity to comment on changes. However, unanimity (or even a majority) is not required.
  +
  +
* '''Every API change should be described precisely in the commit log.''' The commit logs should be sent to a public mailing list, or otherwise made easily available (e.g. via github), so that the community can keep an eye on changes and comment.
  +
  +
* '''Backwards compatibility''' is important to many users. API changes are expected to retain backwards compatibility wherever possible. However, from time to time we may decide to have major revisions which are explicitly not backwards compatible; in these cases we may try to make the previous version of the package available concurrently, as in the base-3/base-4 switchover.
  +
  +
* You don't need to consult the community for '''purely internal changes'''; i.e. changes that do not affect the library's clients.
  +
  +
* Changes that simply '''widen the API by adding new functions''' are a bit of a grey area. It's better to consult the community, because there may be useful feedback about (say) the order of arguments, or the name of the function, or whatnot. On the other hand few clients will actually break if you add a new function to the API. Use your judgment.
  +
  +
Libraries maintained by the GHC team are subject to the GHC validation policy - patches will be tested for validation before committing ([http://hackage.haskell.org/trac/ghc/wiki/TestingPatches TestingPatches]). Those packages not maintained by the GHC team will probably have a GHC lagging mirror repository that is subject to validation.
  +
  +
= See also =
   
 
* [http://hackage.haskell.org/trac/ghc/wiki/WorkingConventions GHC Working Conventions], including guidelines for submitting patches via darcs.
 
* [http://hackage.haskell.org/trac/ghc/wiki/WorkingConventions GHC Working Conventions], including guidelines for submitting patches via darcs.

Latest revision as of 02:42, 4 February 2019

This page describes the process followed by the Core Libraries Committee, or CLC, for maintaining the libraries it oversees. Many of these libraries are core libraries, defined as libraries which ship with GHC itself, which is the source of the name for this committee. However, over the years, other libraries have been subsumed under CLC maintainership.

Generally speaking, all libraries maintained by the CLC are "important" for some reason or another. The core libraries themselves are particularly important since they ship with GHC, and are generally used by a large subset of the community (e.g., base is used by virtually all Haskell code).

Goals

Our goals for maintenance of these libraries are:

  • Bugs are resolved
  • Feature requests can be implemented
  • Patches can be merged
  • All of this is done in such a way as to avoid creating problems for users of these libraries

The last point is vital. These libraries must have the highest quality standards of any libraries in the Haskell ecosystem. In order to ensure timely response to issues (the first three points) while maintaining the quality standards of the fourth point, we have a dual maintainership system in place.

Each library has an individual maintainer for routine maintenance of the package. In addition, the Core Libraries Committee, or CLC, has oversight on the maintenance of these libraries.

The individual maintainer is responsible for dealing with each individual issue against that library. That individual is fully empowered to make decisions on that library. However, the following restrictions apply:

  • For "controversial" decisions (definition discussed below), the maintainer should not make a unilateral decision, but rather discuss with the CLC as a whole (and possibly a wider audience)
  • If the CLC is dissatisfied with the job being performed by the maintainer, it may remove the maintainer from his/her role (via a majority vote)

NOTE: The oversight role of the CLC only applies to libraries which have explicitly been placed under the CLC's maintainership. Other libraries are encouraged to follow this practice, but are not required to do so. Currently, the libraries which are not maintained by the CLC are containers, parallel, pretty, time, and xhtml.

The list of all libraries is provided below.

While this page only covers maintenance of CLC-maintained libraries, the policies described may be useful for other libraries, especially for packages in the Haskell Platform.

The Libraries

Our definition of "core library" is a library that ships with GHC. This can happen for one of three reasons:

  1. It is part of the Haskell language standard
  2. It is tightly coupled to GHC’s implementation of some feature. These libraries cannot be changed independently of GHC. You could think of them as part of GHC, part that happens to be implemented in library code.
  3. It is used to build GHC itself. We have to ship these libraries with GHC because ghc itself is a package, and so we must have installed packages for anything the ghc package depends on.

The following table lists the libraries maintained by the CLC, their primary maintainers, issue tracker for filing bugs and feature requests, and the reason the library is a member of the core libraries (one of the three reasons above, or blank if the library is a non-core library). Ideally, all libraries listed will have a maintainer as well, though we have not filled all roles yet.

Package Maintainer Issue Tracker Reason
array Dan Doel GHC Trac 1, 3
base Edward Kmett and Ryan Scott GHC Trac 1, and partly 2
deepseq Herbert Valerio Riedel Github 3
directory Phil Ruffwind and Elliot Robinson Github 3
filepath Neil Mitchell Github 3
ghc-prim GHC HQ GHC Trac 2
hpc GHC HQ GHC Trac 2
mtl Edward Kmett Github non-core
primitive Carter Schonwald Github non-core
process Michael Snoyman Github 3
random Carter Schonwald Github non-core
stm GHC HQ GHC Trac 2
template-haskell Edward Kmett GHC Trac 2
unix Herbert Valerio Riedel and Elliot Robinson Github 3
vector Carter Schonwald Github non-core
old-locale Herbert Valerio Riedel Github non-core
old-time Herbert Valerio Riedel Github non-core
Win32 Windows Task Force Github 3

The maintainer "GHC HQ" means Simon Marlow, Simon Peyton Jones, Herbert Valerio Riedel and Ben Gamari. Daniel Fischer has taken responsibility for numeric stuff. Email ghc-devs@haskell.org

The Core Libraries Committee was formed to provide a maintainer of many of the packages previously managed by GHC HQ and can be reached by emailing core-libraries-committee@haskell.org or publicly through libraries@haskell.org.

There is no requirement that a library maintainer be him/herself a member of the CLC.

If you believe an additional library should be part of the CLC library set, or wish to take over as maintainer on one of the libraries above, please send an email to the libraries@ mailing list with the request.

The following packages match the appropriate language standard, and as such cannot change independently. The code is maintained by the GHC team.

haskell2010 GHC HQ
haskell98 GHC HQ

The following packages are core libraries in that they ship with GHC, but are not maintained by the core libraries committee.

Package Maintainer Issue Tracker Reason
containers David Feuer Github 3
hoopl Ning Wang Github 3
parallel Simon Marlow Github non-core (NOTE: Should we remove this library from the table?)
pretty David Terei Github 3
time Ashley Yakeley Github 3
xhtml Chris Dornan Github 3

General workflow

  1. Issues should be filed against the appropriate issue tracker for that library.
    • Note: In the past, core library issues were all made against GHC Trac. That is no longer the case, and issues opened in such a way will likely be closed with a request to reopen in the appropriate place.
  2. If any member of the committee feels particularly interested or well qualified in an individual issue, and the official maintainer has not started working on the issue yet, he/she should feel free to self-assign the issue. Similarly, if the official maintainer feels unqualified to handle an issue, he/she should assign to another individual (whether part of the CLC or not).
  3. There must be a clear status for issues at all time, and this status should be understandable from the Trac discussion itself. Side channels of communication must be recorded in Trac (or at least referenced from there). The community should be able to read a discussion, understand current status, and also have a clear idea of how to move the ticket forward towards resolution. (Note that this requirement is very similar to the commercial SIG maintainer communication guide.)
  4. Any individual should feel empowered to assign unassigned or misassigned issues to the appropriate maintainer.
  5. Ultimately, it is the responsibility of the committee chair to ensure that this process is continuing to run, and that issues are being addressed and resolved quickly and professionally. The chair may of course employ the assistance of others in this endeavor.

Since this has been a difficult point in the past, is bears repeating: an individual maintainer is fully empowered to make decisions, and should do so! Only controversial decisions should involve the rest of the CLC.

Responsiveness

  • Third parties submitting proposals to the maintainer of a library can expect a timely and thoughtful response.
  • The more effort the proposer invests (for example, by constructing a patch rather than making an off-the-cuff suggestion) the more consideration s/he can reasonably expect.
  • Ideally, no issues will be left without a response for more than one week, with two weeks being a maximum. If a maintainer will be on vacation or otherwise unavailable, he/she should notify the rest of the committee beforehand.
  • If there is more than a week's delay on an issue from the maintainer's side, it is acceptable to ping him/her.
  • If there is more than a week's delay on an issue from the issue creator's side, the maintainer should feel free to close the issue if desired, with a comment explaining why it was closed.
  • Proposals that have widespread support, and are accompanied by patches (preferably with tests and documentation), should normally be accepted by the maintainer.
  • It is up to the maintainer to decide what "widespread" means; in particular, it does not always mean "a majority of those who responded". The majority-responder story is vulnerable to selection bias; e.g. 7 people (out of a client base of hundreds) say "add this function" but the maintainer thinks it will make the interface incrementally more complicated without sufficient benefit.
  • Where there is significant work involved in implementing a proposal, it is reasonable for a maintainer to ask for a patch. The principle is that maintainers are not obliged to do the work of implementing a proposal, even if it does enjoy wide support. For more substantial changes, it makes sense to develop the implementation in dialogue with the maintainer.

Mistakes

People are fallible. The process above will allow some mistakes to be made. That's a conscious trade-off we're accepting now. If we had unlimited time and unlimited manpower, we may decide to discuss in detail each and every issue. However, that's not our reality: everyone involved has other responsibilities and does not have time or inclination to discuss in depth each issue.

Said another way: let's make progress with 95% accuracy rather than get bogged down in gridlock with 99% accuracy.

Controversial decisions

It's impossible in this document to record all possible forms of a controversial decision. Here are some basic guidelines:

  • Adding a new, useful function under a clear name is probably not controversial
  • Breaking the API in any way is almost certainly controversial
  • Changing the semantics of an existing function is almost certainly controversial
  • Changes affecting Prelude and the Haskell Report are almost certainly controversial

It is important that, at the very least, the CLC receives an email on this topic. The individual maintainer is still in charge of the issue, but should give the CLC a chance to express an opinion. If after a reasonable amount of time (let's say one week) no objections are raised, the maintainer should feel free to move ahead with the change.

The CLC should repeat this decision making process again with more controversial issues. The prime example of this was the "burning bridges proposal": a change which drastically impacts Prelude should almost certainly be elevated to the libraries@ mailing list. Anything going to libraries@ should have at least a two week deadline on it.

At the end of the day, the CLC is still the body with decision making power. Explicitly: no single person should ever have veto power on a change. That includes the library maintainer; if the CLC votes against the library maintainer, that decision will stand. The CLC may itself decide to request support from others to make a decision, but is never required to do so.

Guide to proposers

Previously (before February 2015), the initiative remained with the proposer for quite a while on the proposal process. The process described here is significantly different, so please note the change.

All library proposals should start on the relevant issue tracker. This may be a bug report, feature request, or even a complete patch/pull request. At this point, the library maintainer is responsible for taking next steps. Some example next steps would be to accept the patch, implement a bug fix, reject a feature request, or decide that this is a controversial decision that must be discussed with the CLC.

If the CLC decides that the discussion must be discussed with the libraries@ mailing list, the original proposer may be asked to moderate the libraries@ mailing list discussion. The general format for this is:

  • Send your proposal by email to the libraries@haskell.org mailing list (which you need to subscribe to before posting).
  • Set a deadline for discussion (no less than two weeks), and act as chair/moderator for the discussion.
  • Following discussion, it's the responsibility of the core libraries committee to determine if the proposal should be accepted. Send an email to the core libraries committee requesting a decision, optionally including your own summary of the mailing list discussion.
  • From that point forward, the CLC and maintainer will again be responsible for making a decision on how to proceed, and will inform the proposer and the mailing list. Possible decisions include acceptance, rejection, and requesting a patch be provided.

Here are desirable properties for a proposal and its implementation. The more of these properties your proposal or patch has, the more likely it is that the maintainer will adopt your idea. After all, to adopt it the maintainter will have to do whatever tasks you didn't do.

  • Description. A good proposal says clearly what you propose, why it is a good idea, and what its consequences would be.
  • Patch. Use darcs record or git commit (depending on what sort of repo the library lives in) to create it. Save the patch to a file, using darcs send --output or git format-patch. Make your changes against a copy of the master branch of the relevant library, and make sure it compiles.
  • Portability. Good code is portable. In particular, try to ensure the code runs in Hugs and GHC, and on Windows and Linux.
  • Style. Good code follows the conventions in the library you are modifying.
  • Documentation. Good code includes valid Haddock documentation.
  • Tests. Good patches have suitable tests for the library's testsuite.

Guidance for maintainers

The principle is that we trust the maintainer to behave sensibly. The guidelines below are just that: guidelines, not rules. Still, the core libraries are used by many, many people, so maintainers should make every effort not to mess them up by accident.

  • API changes should be discussed on the libraries mailing list prior to making the change, even if the maintainer is the proposer. The maintainer still has ultimate say in what changes are made, but the community should have the opportunity to comment on changes. However, unanimity (or even a majority) is not required.
  • Every API change should be described precisely in the commit log. The commit logs should be sent to a public mailing list, or otherwise made easily available (e.g. via github), so that the community can keep an eye on changes and comment.
  • Backwards compatibility is important to many users. API changes are expected to retain backwards compatibility wherever possible. However, from time to time we may decide to have major revisions which are explicitly not backwards compatible; in these cases we may try to make the previous version of the package available concurrently, as in the base-3/base-4 switchover.
  • You don't need to consult the community for purely internal changes; i.e. changes that do not affect the library's clients.
  • Changes that simply widen the API by adding new functions are a bit of a grey area. It's better to consult the community, because there may be useful feedback about (say) the order of arguments, or the name of the function, or whatnot. On the other hand few clients will actually break if you add a new function to the API. Use your judgment.

Libraries maintained by the GHC team are subject to the GHC validation policy - patches will be tested for validation before committing (TestingPatches). Those packages not maintained by the GHC team will probably have a GHC lagging mirror repository that is subject to validation.

See also