Differences between revisions 91 and 92
Revision 91 as of 2014-05-01 01:38:45
Size: 15892
Comment:
Revision 92 as of 2014-08-04 00:33:21
Size: 15893
Comment: use plain link for request-assign.changes
Deletions are marked like this. Additions are marked like this.
Line 166: Line 166:
  * [[http://git.savannah.gnu.org/cgit/gnulib.git/tree/doc/Copyright/request-assign.changes|request-assign.changes]]   * [[http://git.savannah.gnu.org/cgit/gnulib.git/plain/doc/Copyright/request-assign.changes|request-assign.changes]]

Contribution Checklist

A contribution to the glibc project should always be sent to the appropriate mailing list for the contribution scope:

  • libc-alpha (Core glibc)
  • libc-locale (Locale changes)
  • libc-help (Need help)

A high-quality contribution includes the following things:

1. Bugzilla entry

A bugzilla entry for a bug should be created and *should* contain everything a high-quality mailing list submission would contain.

If your contribution fixes a user-visible issue we strongly suggest it should have a bugzilla entry.

Please follow the bugzilla procedures.

2. Contribution Email Subject Line

A high-quality email subject line for a contribution contains the following tags:

For a Single patch contribution:

  • [PATCH] <Subject>

For Multi-part patch set contributions where [PATCH 0/2] is used to described the forth coming patches:

  • [PATCH 0/2] <Optional unique prefix e.g. "String cleanup:"><Subject related to the patch set e.g. Cleanup all broken string routines.>

  • [PATCH 1/2] <Optional unique prefix e.g. "String cleanup:"><Distinct subject line for this patch e.g. Cleanup strstr.>

  • [PATCH 2/2] <Optional unique prefix e.g. "String cleanup:"><Another distinct subject line for this patch e.g. Cleanup strncmp.>

    Notes: http://sourceware.org/ml/libc-ports/2011-11/msg00025.html

For a request for comments that doesn't have to be a patch at all:

  • [RFC] <Subject>

For a request for comments on a patch or patch set:

  • [RFC][PATCH] <Subject>

For a patch that fixes a bug filed in bugzilla:

  • [PATCH][BZ <bugzilla number>] <Subject>

For a patch that affects a particular architecture only:

  • [PATCH] PowerPC - <Subject>

For a patch that is the Nth version of an earlier patch:

  • [PATCH][vN] <Subject>

For a patch carried on a non-canonical namespace branch, but not necessary in-line in the patch body:

  • [PATCH namespace/branch] <Subject>, e.g., [PATCH roland/arm-memcpy] ARM: Make multiarch memcpy always use NEON when compiler does

For a patch that is already committed:

  • [COMMITTED] ...

3. Properly Identified Contribution Scope

  • Before you generate a patch or edit a ChangeLog file run find . -name ChangeLog to help identify the scope of your contribution, e.g.,

    • ./localedata/ChangeLog
      ./libidn/ChangeLog
      ./ChangeLog

    If you've touched code in any of the directories these ChangeLogs are found you can use the directory(s) to identify the scope(s) of the patch. Please note that stable branches may have one or more legacy ChangeLogs still in use so please be aware of that when submitting patches to stable branches.

  • If you have modified code in more than one than one scope you need to generate separate patch sets and separate ChangeLog entries for each scope.

  • If possible keep a patch to a single scope.

3.1. Base

  • Any modification to code not in the Add-ons scope should be considered part of glibc base scope.
  • Send base scope patches to the libc-alpha mailing list.
  • The project toplevel ChangeLog is for anything in the base scope that doesn't have it's own ChangeLog.

  • All entries for the toplevel ChangeLog file should be relative to the base directory and all patches under this scope should apply with patch -p1.

3.2. Add-Ons (NPTL, localedata, posix, libidn, et al)

  • Patches to add-on projects (like NPTL) should be sent to the libc-alpha mailing list.
  • Some Add-ons have their own ChangeLog files so any changes to the add-on directory should be indicated in <add-on>/ChangeLog.

  • All entries for the <add-on>/ChangeLog file should be relative to the <add-on>/ directory

  • All Add-on patches should apply with patch -p0 from the base glibc directory.

  • For locale changes to the Localedata add-on please see the Qualifications section in the Locales page.

3.3. Ports

  • The ports/ directory is no longer used as of glibc 2.20. All code has been merged into the core project. If you have patches for ports/ please rebase them against the appropriate new directories.

  • For patches to stable branches that still use ports/ please send ports patches to the libc-alpha mailing list.

  • On old stable branches the ports/ project has its own ChangeLog file so any changes to a ports platform should be indicated in the ports/ChangeLog.

  • All entries for the ports/ChangeLog file should be relative to the ports/ directory and all patches should apply with patch -p1.

3.4. Out-of-Scope (Automatically Generated Files)

Changes in automatically regenerated files (configure, for example) should not be included in the submitted patch.

4. Detailed Explanation of the Patch

4.1. General Patch Requirements

  • Explain the steps required to reproduce the problem.
    • Extra points are awarded for a test-case which demonstrates the problem and can be used in the test-suite to demonstrate success of a fix.
  • Explain the observed versus expected behaviour.
  • Quote relevant standards, whenever possible give section or URL reference.
  • All patches for consideration must have ChangeLog entry updates (see below) of they will be rejected during mailing list moderation.

  • If the code in question is a Request For Comment (RFC) please indicate as such in the email subject. Explicit RFC emails aren't constrained by the ChangeLog rules but they won't be considered for inclusion until they comply.

4.2. Documentation

If your patch adds a new interface then that interface should be documented in the glibc manual.

Where possible please additionally contribute a manual page to the Linux man-pages project: http://www.kernel.org/doc/man-pages/

5. Check For New Warnings

Check the make log for new warnings. Any new warnings generated should have the cause corrected before submission of the patch.

6. Testing

  • Explain any regressions in the test suite.

  • Explain for which targets the test suite was run.
  • Adding a test-case to the test suite may increase the likelihood of acceptance of your patch. In some cases it may be necessary.

6.1. Testing Locales

Please see the Testing Locales section in the Locales page.

Specific functionality in glibc is often implemented separately for different configurations, such as different operating systems or processor architectures. When identifying and fixing a problem in one of these implementations, the same reasoning might also apply to the others, and the same or a similar fix might also be needed there. You should try to identify these cases, and draw attention to these in your submission (»similar changes to [this] are probably needed [here], too«), or if the case is simple enough try to fix all occurrences at the same time and propose a possibly untested patch, too. Especially for nontrivial issues, it is of course fine to defer to the respective maintainers for help -- we don't expect you to fix SPARC assembler code when you're fixing an issue in an x86 assembler implementation, and neither do we expect you to be able to test a patch for GNU/Hurd when you're fixing an issue in glibc's GNU/Linux-specific code.

  • Explain your current FSF copyright assignment status.
  • In the event that you have not previously completed an FSF copyright assignment, a libc-alpha moderator or glibc maintainer will direct you to a FSF copyright assignment request form that's best suited to your contribution under the guidelines on this gnu.org webpage.

  • Copyright assignment is not required for localedata changes.

  • When non-trivial, i.e. legally significant, changes are made to a file the copyright date must be updated in the file. This GNU Copyright-Notices webpage indicates the proper format for the copyright notice and how to update the dates:

  • A copyright notice takes the following form:

     Copyright (C) year1, year5 copyright-holder
  • To add 'year6' following 'year5' simply add ', year6' between 'year5' and the copyright-holder, e.g.,:

     Copyright (C) year1, year5, year6 copyright-holder
  • For glibc, spanning of consecutive years IS allowed. Therefore, when adding a third (or more) consecutive year to the copyright statement (like 'year7'), you may replace individual consecutive years with a year range, e.g.,

     Copyright (C) year1, year5-year7 copyright-holder
  • For glibc, collapsing of years between 1991 and the present IS allowed (See this thread on libc-alpha). Therefore, when adding a second (or more) years to the copyright statement (like 'year7'), you may collapse years with a year range given that the range starts at or after 1991, e.g.,

     Copyright (C) year1-year7 copyright-holder
  • Do NOT abbreviate dates into two digit year format, i.e., don't abbreviate '2008' as '08'. As indicated in this thread on libc-alpha, the maintainer does not like the two-digit year format.

  • The copyright notice of some files indicates a copyright holder other than the FSF (such files have been copied from another project). In those cases, leave the the non-FSF copyright notice as is, and add a new line for the FSF copyright. If there's already an FSF-copyright line, simply update it, adding it a year as usual. If the two copyright notices disagree for other reasons, e.g., licensing information, keep the notices separate. E.g.,

     Copyright (C) 2012-2013 Free Software Foundation, Inc.
     ... rest of FSF copyright notice ...

     Copyright (C) 2010 other-copyright-holder
     ... rest of other copyright notice ...

10. Attribution

  • "Contributed by" statements are no longer required or desired in glibc source files (though existing statements will remain). Attribution is now recorded in git commit logs as the commit "Author". Please do not use "Contributed by" statements in your submissions.

11. Properly Formatted GNU ChangeLog

  • Your ChangeLog must be in the GNU ChangeLog format. The ChangeLog format is described here and here.

  • This should not be part of the change diff; put it right before the diff. (The git-show-gnu script can be used to produce the standard format for posting a patch once you've done a git commit on a local branch.)

  • Files that were regenerated should be identified and the change should say "Regenerated."
  • The GDB folks have a tool called mklog to help generate ChangeLogs.

  • Per this comment, every change line in a ChangeLog should start with a capitalized word and end with a period.

  • The ChangeLog should indicate what was changed in the file. For instance, if you change the size of a variable you need to indicate as much in the ChangeLog.

  • ChangeLog standard tab-width is 8.

  • ChangeLog standard max-line-length is 80.

  • Make sure your mailer doesn't turn TABS into SPACES, i.e., turn on preformat.

  • An example for a trivial change follows: Note: words in arrow brackets shouldn't appear in the ChangeLog, they indicate whitespace.

  • When multiple entries contain the same ChangeLog information as previous entries the glibc approved shorthand is: * file: Likewise.

YYYY-MM-DD<2 spaces>John Doe<2 spaces><johndoe@some.email.address>
<blank-line>
<tab--->[BZ #<number>]<use this if there is a bugzilla associated with the patch>
<tab--->* login/utmp_file.c (pututline_file): Replace call to 'dup()'<line wrap at or before column 79>
<tab--->with libc internal symbol '__dup()' to avoid access through<line wrap at or before column 79>
<tab--->the PLT.
<tab--->* manual/install.texi: Minimum version updated.
<tab--->* INSTALL: Regenerated.

2008-12-12  John Doe  <johndoe@some.email.address>

        [BZ #9999]
        * login/utmp_file.c (pututline_file): Replace call to 'dup()'
        with libc internal symbol '__dup()' to avoid access through
        the PLT.
        * manual/install.texi: Minimum version updated.
        * INSTALL: Regenerated.

12. Properly Formatted Changes

Please see Style_and_Conventions for a full description of coding guidelines.

13. Proper Formatted Unified diff of the Changes

  • Only unified diff (-uNr) format is acceptable. Patches which are context diffs will not be reviewed.

  • Unified diff must be in a format that can be applied with patch -p1.
  • The strong preference is that you include the patch as part of the text of the email. However, if your mailer unavoidably wraps long lines or otherwise alters patch contents, or if the diff might cause problems due to binary data or mixed encodings, attach the patch to the email. The attachment should normally employ a Content-Type of text/x-diff or text/x-patch (or at least text/plain) and a charset of UTF-8 or US-ASCII; if it contains binary data or mixed encoding it may need to specify a Content-Type of application/octet-stream instead. Please use an attachment file name that ends in ".diff" or ".patch" (or at least ".txt").

If you have any questions regarding these criteria please email libc-help@sourceware.org .

14. Ping and keep pinging

If your patch is not reviewed it may just mean that the reviewers are busy. Please ping and keep pinging the patch on a weekly basis.

None: Contribution checklist (last edited 2014-08-04 00:33:21 by MikeFrysinger)