Merge branch 'ab/fsck-skiplist'
authorJunio C Hamano <gitster@pobox.com>
Wed, 10 Oct 2018 03:37:16 +0000 (12:37 +0900)
committerJunio C Hamano <gitster@pobox.com>
Wed, 10 Oct 2018 03:37:16 +0000 (12:37 +0900)
Update fsck.skipList implementation and documentation.

* ab/fsck-skiplist:
  fsck: support comments & empty lines in skipList
  fsck: use oidset instead of oid_array for skipList
  fsck: use strbuf_getline() to read skiplist file
  fsck: add a performance test for skipList
  fsck: add a performance test
  fsck: document that skipList input must be unabbreviated
  fsck: document and test commented & empty line skipList input
  fsck: document and test sorted skipList input
  fsck tests: add a test for no skipList input
  fsck tests: setup of bogus commit object

1  2 
Documentation/config.txt
fsck.c

diff --combined Documentation/config.txt
@@@ -832,6 -832,12 +832,6 @@@ core.packedRefsTimeout:
        all; -1 means to try indefinitely. Default is 1000 (i.e.,
        retry for 1 second).
  
 -sequence.editor::
 -      Text editor used by `git rebase -i` for editing the rebase instruction file.
 -      The value is meant to be interpreted by the shell when it is used.
 -      It can be overridden by the `GIT_SEQUENCE_EDITOR` environment variable.
 -      When not configured the default commit message editor is used instead.
 -
  core.pager::
        Text viewer for use by Git commands (e.g., 'less').  The value
        is meant to be interpreted by the shell.  The order of preference
@@@ -931,11 -937,6 +931,11 @@@ core.useReplaceRefs:
        option was given on the command line. See linkgit:git[1] and
        linkgit:git-replace[1] for more information.
  
 +core.multiPackIndex::
 +      Use the multi-pack-index file to track multiple packfiles using a
 +      single index. See link:technical/multi-pack-index.html[the
 +      multi-pack-index design document].
 +
  core.sparseCheckout::
        Enable "sparse checkout" feature. See section "Sparse checkout" in
        linkgit:git-read-tree[1] for more information.
@@@ -1153,14 -1154,6 +1153,14 @@@ and by linkgit:git-worktree[1] when 'gi
  remote branch. This setting might be used for other checkout-like
  commands or functionality in the future.
  
 +checkout.optimizeNewBranch::
 +      Optimizes the performance of "git checkout -b <new_branch>" when
 +      using sparse-checkout.  When set to true, git will not update the
 +      repo based on the current sparse-checkout settings.  This means it
 +      will not update the skip-worktree bit in the index nor add/remove
 +      files in the working directory to reflect the current sparse checkout
 +      settings nor will it show the local changes.
 +
  clean.requireForce::
        A boolean to make git-clean do nothing unless given -f,
        -i or -n.   Defaults to true.
@@@ -1514,9 -1507,159 +1514,9 @@@ fastimport.unpackLimit:
        operation complete faster, especially on slow filesystems.  If
        not set, the value of `transfer.unpackLimit` is used instead.
  
 -fetch.recurseSubmodules::
 -      This option can be either set to a boolean value or to 'on-demand'.
 -      Setting it to a boolean changes the behavior of fetch and pull to
 -      unconditionally recurse into submodules when set to true or to not
 -      recurse at all when set to false. When set to 'on-demand' (the default
 -      value), fetch and pull will only recurse into a populated submodule
 -      when its superproject retrieves a commit that updates the submodule's
 -      reference.
 -
 -fetch.fsckObjects::
 -      If it is set to true, git-fetch-pack will check all fetched
 -      objects. See `transfer.fsckObjects` for what's
 -      checked. Defaults to false. If not set, the value of
 -      `transfer.fsckObjects` is used instead.
 -
 -fetch.fsck.<msg-id>::
 -      Acts like `fsck.<msg-id>`, but is used by
 -      linkgit:git-fetch-pack[1] instead of linkgit:git-fsck[1]. See
 -      the `fsck.<msg-id>` documentation for details.
 -
 -fetch.fsck.skipList::
 -      Acts like `fsck.skipList`, but is used by
 -      linkgit:git-fetch-pack[1] instead of linkgit:git-fsck[1]. See
 -      the `fsck.skipList` documentation for details.
 -
 -fetch.unpackLimit::
 -      If the number of objects fetched over the Git native
 -      transfer is below this
 -      limit, then the objects will be unpacked into loose object
 -      files. However if the number of received objects equals or
 -      exceeds this limit then the received pack will be stored as
 -      a pack, after adding any missing delta bases.  Storing the
 -      pack from a push can make the push operation complete faster,
 -      especially on slow filesystems.  If not set, the value of
 -      `transfer.unpackLimit` is used instead.
 -
 -fetch.prune::
 -      If true, fetch will automatically behave as if the `--prune`
 -      option was given on the command line.  See also `remote.<name>.prune`
 -      and the PRUNING section of linkgit:git-fetch[1].
 -
 -fetch.pruneTags::
 -      If true, fetch will automatically behave as if the
 -      `refs/tags/*:refs/tags/*` refspec was provided when pruning,
 -      if not set already. This allows for setting both this option
 -      and `fetch.prune` to maintain a 1=1 mapping to upstream
 -      refs. See also `remote.<name>.pruneTags` and the PRUNING
 -      section of linkgit:git-fetch[1].
 -
 -fetch.output::
 -      Control how ref update status is printed. Valid values are
 -      `full` and `compact`. Default value is `full`. See section
 -      OUTPUT in linkgit:git-fetch[1] for detail.
 -
 -fetch.negotiationAlgorithm::
 -      Control how information about the commits in the local repository is
 -      sent when negotiating the contents of the packfile to be sent by the
 -      server. Set to "skipping" to use an algorithm that skips commits in an
 -      effort to converge faster, but may result in a larger-than-necessary
 -      packfile; The default is "default" which instructs Git to use the default algorithm
 -      that never skips commits (unless the server has acknowledged it or one
 -      of its descendants).
 -      Unknown values will cause 'git fetch' to error out.
 -+
 -See also the `--negotiation-tip` option for linkgit:git-fetch[1].
 -
 -format.attach::
 -      Enable multipart/mixed attachments as the default for
 -      'format-patch'.  The value can also be a double quoted string
 -      which will enable attachments as the default and set the
 -      value as the boundary.  See the --attach option in
 -      linkgit:git-format-patch[1].
 -
 -format.from::
 -      Provides the default value for the `--from` option to format-patch.
 -      Accepts a boolean value, or a name and email address.  If false,
 -      format-patch defaults to `--no-from`, using commit authors directly in
 -      the "From:" field of patch mails.  If true, format-patch defaults to
 -      `--from`, using your committer identity in the "From:" field of patch
 -      mails and including a "From:" field in the body of the patch mail if
 -      different.  If set to a non-boolean value, format-patch uses that
 -      value instead of your committer identity.  Defaults to false.
 -
 -format.numbered::
 -      A boolean which can enable or disable sequence numbers in patch
 -      subjects.  It defaults to "auto" which enables it only if there
 -      is more than one patch.  It can be enabled or disabled for all
 -      messages by setting it to "true" or "false".  See --numbered
 -      option in linkgit:git-format-patch[1].
 -
 -format.headers::
 -      Additional email headers to include in a patch to be submitted
 -      by mail.  See linkgit:git-format-patch[1].
 -
 -format.to::
 -format.cc::
 -      Additional recipients to include in a patch to be submitted
 -      by mail.  See the --to and --cc options in
 -      linkgit:git-format-patch[1].
 -
 -format.subjectPrefix::
 -      The default for format-patch is to output files with the '[PATCH]'
 -      subject prefix. Use this variable to change that prefix.
 -
 -format.signature::
 -      The default for format-patch is to output a signature containing
 -      the Git version number. Use this variable to change that default.
 -      Set this variable to the empty string ("") to suppress
 -      signature generation.
 -
 -format.signatureFile::
 -      Works just like format.signature except the contents of the
 -      file specified by this variable will be used as the signature.
 -
 -format.suffix::
 -      The default for format-patch is to output files with the suffix
 -      `.patch`. Use this variable to change that suffix (make sure to
 -      include the dot if you want it).
 -
 -format.pretty::
 -      The default pretty format for log/show/whatchanged command,
 -      See linkgit:git-log[1], linkgit:git-show[1],
 -      linkgit:git-whatchanged[1].
 -
 -format.thread::
 -      The default threading style for 'git format-patch'.  Can be
 -      a boolean value, or `shallow` or `deep`.  `shallow` threading
 -      makes every mail a reply to the head of the series,
 -      where the head is chosen from the cover letter, the
 -      `--in-reply-to`, and the first patch mail, in this order.
 -      `deep` threading makes every mail a reply to the previous one.
 -      A true boolean value is the same as `shallow`, and a false
 -      value disables threading.
 -
 -format.signOff::
 -      A boolean value which lets you enable the `-s/--signoff` option of
 -      format-patch by default. *Note:* Adding the Signed-off-by: line to a
 -      patch should be a conscious act and means that you certify you have
 -      the rights to submit this work under the same open source license.
 -      Please see the 'SubmittingPatches' document for further discussion.
 -
 -format.coverLetter::
 -      A boolean that controls whether to generate a cover-letter when
 -      format-patch is invoked, but in addition can be set to "auto", to
 -      generate a cover-letter only when there's more than one patch.
 -
 -format.outputDirectory::
 -      Set a custom directory to store the resulting files instead of the
 -      current working directory.
 -
 -format.useAutoBase::
 -      A boolean value which lets you enable the `--base=auto` option of
 -      format-patch by default.
 +include::fetch-config.txt[]
 +
 +include::format-config.txt[]
  
  filter.<driver>.clean::
        The command which is used to convert the content of a worktree
@@@ -1567,12 -1710,16 +1567,16 @@@ doing the same for `receive.fsck.<msg-i
  will only cause git to warn.
  
  fsck.skipList::
-       The path to a sorted list of object names (i.e. one SHA-1 per
+       The path to a list of object names (i.e. one unabbreviated SHA-1 per
        line) that are known to be broken in a non-fatal way and should
-       be ignored. This feature is useful when an established project
-       should be accepted despite early commits containing errors that
-       can be safely ignored such as invalid committer email addresses.
-       Note: corrupt objects cannot be skipped with this setting.
+       be ignored. On versions of Git 2.20 and later comments ('#'), empty
+       lines, and any leading and trailing whitespace is ignored. Everything
+       but a SHA-1 per line will error out on older versions.
+ +
+ This feature is useful when an established project should be accepted
+ despite early commits containing errors that can be safely ignored
+ such as invalid committer email addresses.  Note: corrupt objects
+ cannot be skipped with this setting.
  +
  Like `fsck.<msg-id>` this variable has corresponding
  `receive.fsck.skipList` and `fetch.fsck.skipList` variants.
@@@ -1582,6 -1729,15 +1586,15 @@@ Unlike variables like `color.ui` and `c
  fall back on the `fsck.skipList` configuration if they aren't set. To
  uniformly configure the same fsck settings in different circumstances
  all three of them they must all set to the same values.
+ +
+ Older versions of Git (before 2.20) documented that the object names
+ list should be sorted. This was never a requirement, the object names
+ could appear in any order, but when reading the list we tracked whether
+ the list was sorted for the purposes of an internal binary search
+ implementation, which could save itself some work with an already sorted
+ list. Unless you had a humongous list there was no reason to go out of
+ your way to pre-sort the list. After Git version 2.20 a hash implementation
+ is used instead, so there's now no reason to pre-sort the list.
  
  gc.aggressiveDepth::
        The depth parameter used in the delta compression
@@@ -1691,7 -1847,73 +1704,7 @@@ gc.rerereUnresolved:
        You can also use more human-readable "1.month.ago", etc.
        The default is 15 days.  See linkgit:git-rerere[1].
  
 -gitcvs.commitMsgAnnotation::
 -      Append this string to each commit message. Set to empty string
 -      to disable this feature. Defaults to "via git-CVS emulator".
 -
 -gitcvs.enabled::
 -      Whether the CVS server interface is enabled for this repository.
 -      See linkgit:git-cvsserver[1].
 -
 -gitcvs.logFile::
 -      Path to a log file where the CVS server interface well... logs
 -      various stuff. See linkgit:git-cvsserver[1].
 -
 -gitcvs.usecrlfattr::
 -      If true, the server will look up the end-of-line conversion
 -      attributes for files to determine the `-k` modes to use. If
 -      the attributes force Git to treat a file as text,
 -      the `-k` mode will be left blank so CVS clients will
 -      treat it as text. If they suppress text conversion, the file
 -      will be set with '-kb' mode, which suppresses any newline munging
 -      the client might otherwise do. If the attributes do not allow
 -      the file type to be determined, then `gitcvs.allBinary` is
 -      used. See linkgit:gitattributes[5].
 -
 -gitcvs.allBinary::
 -      This is used if `gitcvs.usecrlfattr` does not resolve
 -      the correct '-kb' mode to use. If true, all
 -      unresolved files are sent to the client in
 -      mode '-kb'. This causes the client to treat them
 -      as binary files, which suppresses any newline munging it
 -      otherwise might do. Alternatively, if it is set to "guess",
 -      then the contents of the file are examined to decide if
 -      it is binary, similar to `core.autocrlf`.
 -
 -gitcvs.dbName::
 -      Database used by git-cvsserver to cache revision information
 -      derived from the Git repository. The exact meaning depends on the
 -      used database driver, for SQLite (which is the default driver) this
 -      is a filename. Supports variable substitution (see
 -      linkgit:git-cvsserver[1] for details). May not contain semicolons (`;`).
 -      Default: '%Ggitcvs.%m.sqlite'
 -
 -gitcvs.dbDriver::
 -      Used Perl DBI driver. You can specify any available driver
 -      for this here, but it might not work. git-cvsserver is tested
 -      with 'DBD::SQLite', reported to work with 'DBD::Pg', and
 -      reported *not* to work with 'DBD::mysql'. Experimental feature.
 -      May not contain double colons (`:`). Default: 'SQLite'.
 -      See linkgit:git-cvsserver[1].
 -
 -gitcvs.dbUser, gitcvs.dbPass::
 -      Database user and password. Only useful if setting `gitcvs.dbDriver`,
 -      since SQLite has no concept of database users and/or passwords.
 -      'gitcvs.dbUser' supports variable substitution (see
 -      linkgit:git-cvsserver[1] for details).
 -
 -gitcvs.dbTableNamePrefix::
 -      Database table name prefix.  Prepended to the names of any
 -      database tables used, allowing a single database to be used
 -      for several repositories.  Supports variable substitution (see
 -      linkgit:git-cvsserver[1] for details).  Any non-alphabetic
 -      characters will be replaced with underscores.
 -
 -All gitcvs variables except for `gitcvs.usecrlfattr` and
 -`gitcvs.allBinary` can also be specified as
 -'gitcvs.<access_method>.<varname>' (where 'access_method'
 -is one of "ext" and "pserver") to make them apply only for the given
 -access method.
 +include::gitcvs-config.txt[]
  
  gitweb.category::
  gitweb.description::
@@@ -1756,7 -1978,63 +1769,7 @@@ gpg.<format>.program:
        be used as a legacy synonym for `gpg.openpgp.program`. The default
        value for `gpg.x509.program` is "gpgsm".
  
 -gui.commitMsgWidth::
 -      Defines how wide the commit message window is in the
 -      linkgit:git-gui[1]. "75" is the default.
 -
 -gui.diffContext::
 -      Specifies how many context lines should be used in calls to diff
 -      made by the linkgit:git-gui[1]. The default is "5".
 -
 -gui.displayUntracked::
 -      Determines if linkgit:git-gui[1] shows untracked files
 -      in the file list. The default is "true".
 -
 -gui.encoding::
 -      Specifies the default encoding to use for displaying of
 -      file contents in linkgit:git-gui[1] and linkgit:gitk[1].
 -      It can be overridden by setting the 'encoding' attribute
 -      for relevant files (see linkgit:gitattributes[5]).
 -      If this option is not set, the tools default to the
 -      locale encoding.
 -
 -gui.matchTrackingBranch::
 -      Determines if new branches created with linkgit:git-gui[1] should
 -      default to tracking remote branches with matching names or
 -      not. Default: "false".
 -
 -gui.newBranchTemplate::
 -      Is used as suggested name when creating new branches using the
 -      linkgit:git-gui[1].
 -
 -gui.pruneDuringFetch::
 -      "true" if linkgit:git-gui[1] should prune remote-tracking branches when
 -      performing a fetch. The default value is "false".
 -
 -gui.trustmtime::
 -      Determines if linkgit:git-gui[1] should trust the file modification
 -      timestamp or not. By default the timestamps are not trusted.
 -
 -gui.spellingDictionary::
 -      Specifies the dictionary used for spell checking commit messages in
 -      the linkgit:git-gui[1]. When set to "none" spell checking is turned
 -      off.
 -
 -gui.fastCopyBlame::
 -      If true, 'git gui blame' uses `-C` instead of `-C -C` for original
 -      location detection. It makes blame significantly faster on huge
 -      repositories at the expense of less thorough copy detection.
 -
 -gui.copyBlameThreshold::
 -      Specifies the threshold to use in 'git gui blame' original location
 -      detection, measured in alphanumeric characters. See the
 -      linkgit:git-blame[1] manual for more information on copy detection.
 -
 -gui.blamehistoryctx::
 -      Specifies the radius of history context in days to show in
 -      linkgit:gitk[1] for the selected commit, when the `Show History
 -      Context` menu item is invoked from 'git gui blame'. If this
 -      variable is set to zero, the whole history is shown.
 +include::gui-config.txt[]
  
  guitool.<name>.cmd::
        Specifies the shell command line to execute when the corresponding item
@@@ -2406,21 -2684,6 +2419,21 @@@ Note that changing the compression leve
  all existing objects. You can force recompression by passing the -F option
  to linkgit:git-repack[1].
  
 +pack.island::
 +      An extended regular expression configuring a set of delta
 +      islands. See "DELTA ISLANDS" in linkgit:git-pack-objects[1]
 +      for details.
 +
 +pack.islandCore::
 +      Specify an island name which gets to have its objects be
 +      packed first. This creates a kind of pseudo-pack at the front
 +      of one pack, so that the objects from the specified island are
 +      hopefully faster to copy into any pack that should be served
 +      to a user requesting these objects. In practice this means
 +      that the island specified should likely correspond to what is
 +      the most commonly cloned in the repo. See also "DELTA ISLANDS"
 +      in linkgit:git-pack-objects[1].
 +
  pack.deltaCacheSize::
        The maximum memory in bytes used for caching deltas in
        linkgit:git-pack-objects[1] before writing them out to a pack.
@@@ -2578,17 -2841,284 +2591,17 @@@ protocol.version:
  * `1` - the original wire protocol with the addition of a version string
    in the initial response from the server.
  
 ---
 +* `2` - link:technical/protocol-v2.html[wire protocol version 2].
  
 -pull.ff::
 -      By default, Git does not create an extra merge commit when merging
 -      a commit that is a descendant of the current commit. Instead, the
 -      tip of the current branch is fast-forwarded. When set to `false`,
 -      this variable tells Git to create an extra merge commit in such
 -      a case (equivalent to giving the `--no-ff` option from the command
 -      line). When set to `only`, only such fast-forward merges are
 -      allowed (equivalent to giving the `--ff-only` option from the
 -      command line). This setting overrides `merge.ff` when pulling.
 -
 -pull.rebase::
 -      When true, rebase branches on top of the fetched branch, instead
 -      of merging the default branch from the default remote when "git
 -      pull" is run. See "branch.<name>.rebase" for setting this on a
 -      per-branch basis.
 -+
 -When `merges`, pass the `--rebase-merges` option to 'git rebase'
 -so that the local merge commits are included in the rebase (see
 -linkgit:git-rebase[1] for details).
 -+
 -When preserve, also pass `--preserve-merges` along to 'git rebase'
 -so that locally committed merge commits will not be flattened
 -by running 'git pull'.
 -+
 -When the value is `interactive`, the rebase is run in interactive mode.
 -+
 -*NOTE*: this is a possibly dangerous operation; do *not* use
 -it unless you understand the implications (see linkgit:git-rebase[1]
 -for details).
 -
 -pull.octopus::
 -      The default merge strategy to use when pulling multiple branches
 -      at once.
 -
 -pull.twohead::
 -      The default merge strategy to use when pulling a single branch.
 -
 -push.default::
 -      Defines the action `git push` should take if no refspec is
 -      explicitly given.  Different values are well-suited for
 -      specific workflows; for instance, in a purely central workflow
 -      (i.e. the fetch source is equal to the push destination),
 -      `upstream` is probably what you want.  Possible values are:
 -+
  --
  
 -* `nothing` - do not push anything (error out) unless a refspec is
 -  explicitly given. This is primarily meant for people who want to
 -  avoid mistakes by always being explicit.
 +include::pull-config.txt[]
  
 -* `current` - push the current branch to update a branch with the same
 -  name on the receiving end.  Works in both central and non-central
 -  workflows.
 -
 -* `upstream` - push the current branch back to the branch whose
 -  changes are usually integrated into the current branch (which is
 -  called `@{upstream}`).  This mode only makes sense if you are
 -  pushing to the same repository you would normally pull from
 -  (i.e. central workflow).
 -
 -* `tracking` - This is a deprecated synonym for `upstream`.
 -
 -* `simple` - in centralized workflow, work like `upstream` with an
 -  added safety to refuse to push if the upstream branch's name is
 -  different from the local one.
 -+
 -When pushing to a remote that is different from the remote you normally
 -pull from, work as `current`.  This is the safest option and is suited
 -for beginners.
 -+
 -This mode has become the default in Git 2.0.
 -
 -* `matching` - push all branches having the same name on both ends.
 -  This makes the repository you are pushing to remember the set of
 -  branches that will be pushed out (e.g. if you always push 'maint'
 -  and 'master' there and no other branches, the repository you push
 -  to will have these two branches, and your local 'maint' and
 -  'master' will be pushed there).
 -+
 -To use this mode effectively, you have to make sure _all_ the
 -branches you would push out are ready to be pushed out before
 -running 'git push', as the whole point of this mode is to allow you
 -to push all of the branches in one go.  If you usually finish work
 -on only one branch and push out the result, while other branches are
 -unfinished, this mode is not for you.  Also this mode is not
 -suitable for pushing into a shared central repository, as other
 -people may add new branches there, or update the tip of existing
 -branches outside your control.
 -+
 -This used to be the default, but not since Git 2.0 (`simple` is the
 -new default).
 -
 ---
 -
 -push.followTags::
 -      If set to true enable `--follow-tags` option by default.  You
 -      may override this configuration at time of push by specifying
 -      `--no-follow-tags`.
 -
 -push.gpgSign::
 -      May be set to a boolean value, or the string 'if-asked'. A true
 -      value causes all pushes to be GPG signed, as if `--signed` is
 -      passed to linkgit:git-push[1]. The string 'if-asked' causes
 -      pushes to be signed if the server supports it, as if
 -      `--signed=if-asked` is passed to 'git push'. A false value may
 -      override a value from a lower-priority config file. An explicit
 -      command-line flag always overrides this config option.
 -
 -push.pushOption::
 -      When no `--push-option=<option>` argument is given from the
 -      command line, `git push` behaves as if each <value> of
 -      this variable is given as `--push-option=<value>`.
 -+
 -This is a multi-valued variable, and an empty value can be used in a
 -higher priority configuration file (e.g. `.git/config` in a
 -repository) to clear the values inherited from a lower priority
 -configuration files (e.g. `$HOME/.gitconfig`).
 -+
 ---
 -
 -Example:
 -
 -/etc/gitconfig
 -  push.pushoption = a
 -  push.pushoption = b
 -
 -~/.gitconfig
 -  push.pushoption = c
 -
 -repo/.git/config
 -  push.pushoption =
 -  push.pushoption = b
 -
 -This will result in only b (a and c are cleared).
 -
 ---
 -
 -push.recurseSubmodules::
 -      Make sure all submodule commits used by the revisions to be pushed
 -      are available on a remote-tracking branch. If the value is 'check'
 -      then Git will verify that all submodule commits that changed in the
 -      revisions to be pushed are available on at least one remote of the
 -      submodule. If any commits are missing, the push will be aborted and
 -      exit with non-zero status. If the value is 'on-demand' then all
 -      submodules that changed in the revisions to be pushed will be
 -      pushed. If on-demand was not able to push all necessary revisions
 -      it will also be aborted and exit with non-zero status. If the value
 -      is 'no' then default behavior of ignoring submodules when pushing
 -      is retained. You may override this configuration at time of push by
 -      specifying '--recurse-submodules=check|on-demand|no'.
 +include::push-config.txt[]
  
  include::rebase-config.txt[]
  
 -receive.advertiseAtomic::
 -      By default, git-receive-pack will advertise the atomic push
 -      capability to its clients. If you don't want to advertise this
 -      capability, set this variable to false.
 -
 -receive.advertisePushOptions::
 -      When set to true, git-receive-pack will advertise the push options
 -      capability to its clients. False by default.
 -
 -receive.autogc::
 -      By default, git-receive-pack will run "git-gc --auto" after
 -      receiving data from git-push and updating refs.  You can stop
 -      it by setting this variable to false.
 -
 -receive.certNonceSeed::
 -      By setting this variable to a string, `git receive-pack`
 -      will accept a `git push --signed` and verifies it by using
 -      a "nonce" protected by HMAC using this string as a secret
 -      key.
 -
 -receive.certNonceSlop::
 -      When a `git push --signed` sent a push certificate with a
 -      "nonce" that was issued by a receive-pack serving the same
 -      repository within this many seconds, export the "nonce"
 -      found in the certificate to `GIT_PUSH_CERT_NONCE` to the
 -      hooks (instead of what the receive-pack asked the sending
 -      side to include).  This may allow writing checks in
 -      `pre-receive` and `post-receive` a bit easier.  Instead of
 -      checking `GIT_PUSH_CERT_NONCE_SLOP` environment variable
 -      that records by how many seconds the nonce is stale to
 -      decide if they want to accept the certificate, they only
 -      can check `GIT_PUSH_CERT_NONCE_STATUS` is `OK`.
 -
 -receive.fsckObjects::
 -      If it is set to true, git-receive-pack will check all received
 -      objects. See `transfer.fsckObjects` for what's checked.
 -      Defaults to false. If not set, the value of
 -      `transfer.fsckObjects` is used instead.
 -
 -receive.fsck.<msg-id>::
 -      Acts like `fsck.<msg-id>`, but is used by
 -      linkgit:git-receive-pack[1] instead of
 -      linkgit:git-fsck[1]. See the `fsck.<msg-id>` documentation for
 -      details.
 -
 -receive.fsck.skipList::
 -      Acts like `fsck.skipList`, but is used by
 -      linkgit:git-receive-pack[1] instead of
 -      linkgit:git-fsck[1]. See the `fsck.skipList` documentation for
 -      details.
 -
 -receive.keepAlive::
 -      After receiving the pack from the client, `receive-pack` may
 -      produce no output (if `--quiet` was specified) while processing
 -      the pack, causing some networks to drop the TCP connection.
 -      With this option set, if `receive-pack` does not transmit
 -      any data in this phase for `receive.keepAlive` seconds, it will
 -      send a short keepalive packet.  The default is 5 seconds; set
 -      to 0 to disable keepalives entirely.
 -
 -receive.unpackLimit::
 -      If the number of objects received in a push is below this
 -      limit then the objects will be unpacked into loose object
 -      files. However if the number of received objects equals or
 -      exceeds this limit then the received pack will be stored as
 -      a pack, after adding any missing delta bases.  Storing the
 -      pack from a push can make the push operation complete faster,
 -      especially on slow filesystems.  If not set, the value of
 -      `transfer.unpackLimit` is used instead.
 -
 -receive.maxInputSize::
 -      If the size of the incoming pack stream is larger than this
 -      limit, then git-receive-pack will error out, instead of
 -      accepting the pack file. If not set or set to 0, then the size
 -      is unlimited.
 -
 -receive.denyDeletes::
 -      If set to true, git-receive-pack will deny a ref update that deletes
 -      the ref. Use this to prevent such a ref deletion via a push.
 -
 -receive.denyDeleteCurrent::
 -      If set to true, git-receive-pack will deny a ref update that
 -      deletes the currently checked out branch of a non-bare repository.
 -
 -receive.denyCurrentBranch::
 -      If set to true or "refuse", git-receive-pack will deny a ref update
 -      to the currently checked out branch of a non-bare repository.
 -      Such a push is potentially dangerous because it brings the HEAD
 -      out of sync with the index and working tree. If set to "warn",
 -      print a warning of such a push to stderr, but allow the push to
 -      proceed. If set to false or "ignore", allow such pushes with no
 -      message. Defaults to "refuse".
 -+
 -Another option is "updateInstead" which will update the working
 -tree if pushing into the current branch.  This option is
 -intended for synchronizing working directories when one side is not easily
 -accessible via interactive ssh (e.g. a live web site, hence the requirement
 -that the working directory be clean). This mode also comes in handy when
 -developing inside a VM to test and fix code on different Operating Systems.
 -+
 -By default, "updateInstead" will refuse the push if the working tree or
 -the index have any difference from the HEAD, but the `push-to-checkout`
 -hook can be used to customize this.  See linkgit:githooks[5].
 -
 -receive.denyNonFastForwards::
 -      If set to true, git-receive-pack will deny a ref update which is
 -      not a fast-forward. Use this to prevent such an update via a push,
 -      even if that push is forced. This configuration variable is
 -      set when initializing a shared repository.
 -
 -receive.hideRefs::
 -      This variable is the same as `transfer.hideRefs`, but applies
 -      only to `receive-pack` (and so affects pushes, but not fetches).
 -      An attempt to update or delete a hidden ref by `git push` is
 -      rejected.
 -
 -receive.updateServerInfo::
 -      If set to true, git-receive-pack will run git-update-server-info
 -      after receiving data from git-push and updating refs.
 -
 -receive.shallowUpdate::
 -      If set to true, .git/shallow can be updated when new refs
 -      require new shallow roots. Otherwise those refs are rejected.
 +include::receive-config.txt[]
  
  remote.pushDefault::
        The remote to push to by default.  Overrides
@@@ -2688,10 -3218,6 +2701,10 @@@ repack.packKeptObjects:
        index is being written (either via `--write-bitmap-index` or
        `repack.writeBitmaps`).
  
 +repack.useDeltaIslands::
 +      If set to true, makes `git repack` act as if `--delta-islands`
 +      was passed. Defaults to `false`.
 +
  repack.writeBitmaps::
        When true, git will write a bitmap index when packing all
        objects to disk (e.g., when `git repack -a` is run).  This
@@@ -2714,15 -3240,71 +2727,15 @@@ rerere.enabled:
        `$GIT_DIR`, e.g. if "rerere" was previously used in the
        repository.
  
 -sendemail.identity::
 -      A configuration identity. When given, causes values in the
 -      'sendemail.<identity>' subsection to take precedence over
 -      values in the 'sendemail' section. The default identity is
 -      the value of `sendemail.identity`.
 -
 -sendemail.smtpEncryption::
 -      See linkgit:git-send-email[1] for description.  Note that this
 -      setting is not subject to the 'identity' mechanism.
 -
 -sendemail.smtpssl (deprecated)::
 -      Deprecated alias for 'sendemail.smtpEncryption = ssl'.
 -
 -sendemail.smtpsslcertpath::
 -      Path to ca-certificates (either a directory or a single file).
 -      Set it to an empty string to disable certificate verification.
 -
 -sendemail.<identity>.*::
 -      Identity-specific versions of the 'sendemail.*' parameters
 -      found below, taking precedence over those when this
 -      identity is selected, through either the command-line or
 -      `sendemail.identity`.
 -
 -sendemail.aliasesFile::
 -sendemail.aliasFileType::
 -sendemail.annotate::
 -sendemail.bcc::
 -sendemail.cc::
 -sendemail.ccCmd::
 -sendemail.chainReplyTo::
 -sendemail.confirm::
 -sendemail.envelopeSender::
 -sendemail.from::
 -sendemail.multiEdit::
 -sendemail.signedoffbycc::
 -sendemail.smtpPass::
 -sendemail.suppresscc::
 -sendemail.suppressFrom::
 -sendemail.to::
 -sendemail.tocmd::
 -sendemail.smtpDomain::
 -sendemail.smtpServer::
 -sendemail.smtpServerPort::
 -sendemail.smtpServerOption::
 -sendemail.smtpUser::
 -sendemail.thread::
 -sendemail.transferEncoding::
 -sendemail.validate::
 -sendemail.xmailer::
 -      See linkgit:git-send-email[1] for description.
 -
 -sendemail.signedoffcc (deprecated)::
 -      Deprecated alias for `sendemail.signedoffbycc`.
 -
 -sendemail.smtpBatchSize::
 -      Number of messages to be sent per connection, after that a relogin
 -      will happen.  If the value is 0 or undefined, send all messages in
 -      one connection.
 -      See also the `--batch-size` option of linkgit:git-send-email[1].
 -
 -sendemail.smtpReloginDelay::
 -      Seconds wait before reconnecting to smtp server.
 -      See also the `--relogin-delay` option of linkgit:git-send-email[1].
 -
 -showbranch.default::
 +include::sendemail-config.txt[]
 +
 +sequence.editor::
 +      Text editor used by `git rebase -i` for editing the rebase instruction file.
 +      The value is meant to be interpreted by the shell when it is used.
 +      It can be overridden by the `GIT_SEQUENCE_EDITOR` environment variable.
 +      When not configured the default commit message editor is used instead.
 +
 +showBranch.default::
        The default set of branches for linkgit:git-show-branch[1].
        See linkgit:git-show-branch[1].
  
@@@ -2834,7 -3416,88 +2847,7 @@@ stash.showStat:
        option will show diffstat of the stash entry.  Defaults to true.
        See description of 'show' command in linkgit:git-stash[1].
  
 -submodule.<name>.url::
 -      The URL for a submodule. This variable is copied from the .gitmodules
 -      file to the git config via 'git submodule init'. The user can change
 -      the configured URL before obtaining the submodule via 'git submodule
 -      update'. If neither submodule.<name>.active or submodule.active are
 -      set, the presence of this variable is used as a fallback to indicate
 -      whether the submodule is of interest to git commands.
 -      See linkgit:git-submodule[1] and linkgit:gitmodules[5] for details.
 -
 -submodule.<name>.update::
 -      The method by which a submodule is updated by 'git submodule update',
 -      which is the only affected command, others such as
 -      'git checkout --recurse-submodules' are unaffected. It exists for
 -      historical reasons, when 'git submodule' was the only command to
 -      interact with submodules; settings like `submodule.active`
 -      and `pull.rebase` are more specific. It is populated by
 -      `git submodule init` from the linkgit:gitmodules[5] file.
 -      See description of 'update' command in linkgit:git-submodule[1].
 -
 -submodule.<name>.branch::
 -      The remote branch name for a submodule, used by `git submodule
 -      update --remote`.  Set this option to override the value found in
 -      the `.gitmodules` file.  See linkgit:git-submodule[1] and
 -      linkgit:gitmodules[5] for details.
 -
 -submodule.<name>.fetchRecurseSubmodules::
 -      This option can be used to control recursive fetching of this
 -      submodule. It can be overridden by using the --[no-]recurse-submodules
 -      command-line option to "git fetch" and "git pull".
 -      This setting will override that from in the linkgit:gitmodules[5]
 -      file.
 -
 -submodule.<name>.ignore::
 -      Defines under what circumstances "git status" and the diff family show
 -      a submodule as modified. When set to "all", it will never be considered
 -      modified (but it will nonetheless show up in the output of status and
 -      commit when it has been staged), "dirty" will ignore all changes
 -      to the submodules work tree and
 -      takes only differences between the HEAD of the submodule and the commit
 -      recorded in the superproject into account. "untracked" will additionally
 -      let submodules with modified tracked files in their work tree show up.
 -      Using "none" (the default when this option is not set) also shows
 -      submodules that have untracked files in their work tree as changed.
 -      This setting overrides any setting made in .gitmodules for this submodule,
 -      both settings can be overridden on the command line by using the
 -      "--ignore-submodules" option. The 'git submodule' commands are not
 -      affected by this setting.
 -
 -submodule.<name>.active::
 -      Boolean value indicating if the submodule is of interest to git
 -      commands.  This config option takes precedence over the
 -      submodule.active config option. See linkgit:gitsubmodules[7] for
 -      details.
 -
 -submodule.active::
 -      A repeated field which contains a pathspec used to match against a
 -      submodule's path to determine if the submodule is of interest to git
 -      commands. See linkgit:gitsubmodules[7] for details.
 -
 -submodule.recurse::
 -      Specifies if commands recurse into submodules by default. This
 -      applies to all commands that have a `--recurse-submodules` option,
 -      except `clone`.
 -      Defaults to false.
 -
 -submodule.fetchJobs::
 -      Specifies how many submodules are fetched/cloned at the same time.
 -      A positive integer allows up to that number of submodules fetched
 -      in parallel. A value of 0 will give some reasonable default.
 -      If unset, it defaults to 1.
 -
 -submodule.alternateLocation::
 -      Specifies how the submodules obtain alternates when submodules are
 -      cloned. Possible values are `no`, `superproject`.
 -      By default `no` is assumed, which doesn't add references. When the
 -      value is set to `superproject` the submodule to be cloned computes
 -      its alternates location relative to the superprojects alternate.
 -
 -submodule.alternateErrorStrategy::
 -      Specifies how to treat errors with the alternates for a submodule
 -      as computed via `submodule.alternateLocation`. Possible values are
 -      `ignore`, `info`, `die`. Default is `die`.
 +include::submodule-config.txt[]
  
  tag.forceSignAnnotated::
        A boolean to specify whether annotated tags created should be GPG signed.
diff --combined fsck.c
--- 1/fsck.c
--- 2/fsck.c
+++ b/fsck.c
@@@ -10,7 -10,6 +10,6 @@@
  #include "fsck.h"
  #include "refs.h"
  #include "utf8.h"
- #include "sha1-array.h"
  #include "decorate.h"
  #include "oidset.h"
  #include "packfile.h"
@@@ -67,8 -66,6 +66,8 @@@ static struct oidset gitmodules_done = 
        FUNC(GITMODULES_LARGE, ERROR) \
        FUNC(GITMODULES_NAME, ERROR) \
        FUNC(GITMODULES_SYMLINK, ERROR) \
 +      FUNC(GITMODULES_URL, ERROR) \
 +      FUNC(GITMODULES_PATH, ERROR) \
        /* warnings */ \
        FUNC(BAD_FILEMODE, WARN) \
        FUNC(EMPTY_NAME, WARN) \
@@@ -184,40 -181,37 +183,37 @@@ static int fsck_msg_type(enum fsck_msg_
  
  static void init_skiplist(struct fsck_options *options, const char *path)
  {
-       static struct oid_array skiplist = OID_ARRAY_INIT;
-       int sorted, fd;
-       char buffer[GIT_MAX_HEXSZ + 1];
+       FILE *fp;
+       struct strbuf sb = STRBUF_INIT;
        struct object_id oid;
  
-       if (options->skiplist)
-               sorted = options->skiplist->sorted;
-       else {
-               sorted = 1;
-               options->skiplist = &skiplist;
-       }
-       fd = open(path, O_RDONLY);
-       if (fd < 0)
+       fp = fopen(path, "r");
+       if (!fp)
                die("Could not open skip list: %s", path);
-       for (;;) {
+       while (!strbuf_getline(&sb, fp)) {
                const char *p;
-               int result = read_in_full(fd, buffer, sizeof(buffer));
-               if (result < 0)
-                       die_errno("Could not read '%s'", path);
-               if (!result)
-                       break;
-               if (parse_oid_hex(buffer, &oid, &p) || *p != '\n')
-                       die("Invalid SHA-1: %s", buffer);
-               oid_array_append(&skiplist, &oid);
-               if (sorted && skiplist.nr > 1 &&
-                               oidcmp(&skiplist.oid[skiplist.nr - 2],
-                                      &oid) > 0)
-                       sorted = 0;
-       }
-       close(fd);
+               const char *hash;
+               /*
+                * Allow trailing comments, leading whitespace
+                * (including before commits), and empty or whitespace
+                * only lines.
+                */
+               hash = strchr(sb.buf, '#');
+               if (hash)
+                       strbuf_setlen(&sb, hash - sb.buf);
+               strbuf_trim(&sb);
+               if (!sb.len)
+                       continue;
  
-       if (sorted)
-               skiplist.sorted = 1;
+               if (parse_oid_hex(sb.buf, &oid, &p) || *p != '\0')
+                       die("Invalid SHA-1: %s", sb.buf);
+               oidset_insert(&options->skiplist, &oid);
+       }
+       if (ferror(fp))
+               die_errno("Could not read '%s'", path);
+       fclose(fp);
+       strbuf_release(&sb);
  }
  
  static int parse_msg_type(const char *str)
@@@ -322,9 -316,7 +318,7 @@@ static void append_msg_id(struct strbu
  
  static int object_on_skiplist(struct fsck_options *opts, struct object *obj)
  {
-       if (opts && opts->skiplist && obj)
-               return oid_array_lookup(opts->skiplist, &obj->oid) >= 0;
-       return 0;
+       return opts && obj && oidset_contains(&opts->skiplist, &obj->oid);
  }
  
  __attribute__((format (printf, 4, 5)))
@@@ -994,18 -986,6 +988,18 @@@ static int fsck_gitmodules_fn(const cha
                                    FSCK_MSG_GITMODULES_NAME,
                                    "disallowed submodule name: %s",
                                    name);
 +      if (!strcmp(key, "url") && value &&
 +          looks_like_command_line_option(value))
 +              data->ret |= report(data->options, data->obj,
 +                                  FSCK_MSG_GITMODULES_URL,
 +                                  "disallowed submodule url: %s",
 +                                  value);
 +      if (!strcmp(key, "path") && value &&
 +          looks_like_command_line_option(value))
 +              data->ret |= report(data->options, data->obj,
 +                                  FSCK_MSG_GITMODULES_PATH,
 +                                  "disallowed submodule path: %s",
 +                                  value);
        free(name);
  
        return 0;