c59e5eed68
A bug in JGit's footer match method caused "Sample-Bug-Id: 42" to be treated identical to the footer line "Signed-off-by: 42". Since Signed-off-by tags are converted into reviewers, we were adding account number 42 as a reviewer anytime we saw "Sample-Bug-Id: 42". This happened for any footer line key, so long as its length matched the length of "Signed-off-by". This was fixed in a newer version of JGit, so upgrade. Also prevent "Signed-off-by: 42" from being treated as an account. Within a commit message, we only want to match to a reviewer by their full name or their email address. Matching on their internal database id number isn't useful. Bug: issue 553 Change-Id: Ie26974240f26d29fb55cd95dc6b6c72c89903f37 Signed-off-by: Shawn O. Pearce <sop@google.com>
60 lines
2.0 KiB
Plaintext
60 lines
2.0 KiB
Plaintext
Release notes for Gerrit 2.1.2.4
|
|
================================
|
|
|
|
Gerrit 2.1.2.4 is now available in the usual location:
|
|
|
|
link:http://code.google.com/p/gerrit/downloads/list[http://code.google.com/p/gerrit/downloads/list]
|
|
|
|
New Features
|
|
------------
|
|
|
|
* Add 'checkout' download command to patch sets
|
|
+
|
|
The Download area of a patch set now offers a command line to fetch
|
|
and checkout the patch set on a detached HEAD. This is more suitable
|
|
for building and testing the change locally.
|
|
|
|
Bug Fixes
|
|
---------
|
|
|
|
* issue 545 Fallback to ISO-8859-1 if charset isn't supported
|
|
+
|
|
Some input files are misrecognized by the jchardet library that is
|
|
used to automatically guess a character set. A guessed charset
|
|
might not even be supported by the local JRE. In such cases the
|
|
ISO-8859-1 character set is used as a fallback, so the file content
|
|
is still visible.
|
|
|
|
* issue 553 Bugs sometimes added as change reviewers
|
|
+
|
|
Bug references were sometimes added as an 'Anonymous Coward' change
|
|
reviewer when the line used to mention the bug in the commit message
|
|
was the same length as 'Signed-off-by'. Fixed.
|
|
|
|
* Update JGit to 0.7.1.46-gdd63f5c to fix empty tree bug
|
|
+
|
|
Repositories which contained an empty tree object (very uncommon, its
|
|
technically a bug to produce a repository like this) wouldn't clone
|
|
properly from the embedded Gerrit SSH or HTTP daemon. Fixed upstream
|
|
in JGit 0.7.0, but we never picked up the bug fix release.
|
|
|
|
* Allow LDAP to unset the user name
|
|
+
|
|
If the user name is configured to be set only by the LDAP directory,
|
|
and an account has a user name, but the name is no longer present
|
|
in the directory, Gerrit crashed during sign-in while trying to
|
|
clear out the user name. Fixed.
|
|
|
|
Documentation Corrections
|
|
~~~~~~~~~~~~~~~~~~~~~~~~~
|
|
|
|
* documentation: Elaborate on branch level Owner
|
|
+
|
|
Documentation didn't describe that the Owner permission within a
|
|
project can be used to delegate control over a branch namespace to
|
|
another group.
|
|
|
|
* documentation: Document Read Access +2 aka Upload Access
|
|
+
|
|
The documentation didn't describe what Read +2 means.
|