diff options
author | Shawn O. Pearce <sop@google.com> | 2010-08-23 08:34:03 -0700 |
---|---|---|
committer | Shawn O. Pearce <sop@google.com> | 2010-08-23 08:34:03 -0700 |
commit | baf6e3a52e22fd43ae68043b6a96c05e9abb2b31 (patch) | |
tree | a56283b6f89d9272fc1720b93a4c51affad1f05e | |
parent | bd870e515b7a43b10d00fb4760f5c3b918e3aeec (diff) |
Draft release notes for 2.1.5v2.1.5-rc0
Change-Id: I587e9e5693aee85e805127dc1c4b50ffaaa7e643
Signed-off-by: Shawn O. Pearce <sop@google.com>
-rw-r--r-- | ReleaseNotes/ReleaseNotes-2.1.5.txt | 169 | ||||
-rw-r--r-- | ReleaseNotes/index.txt | 1 |
2 files changed, 170 insertions, 0 deletions
diff --git a/ReleaseNotes/ReleaseNotes-2.1.5.txt b/ReleaseNotes/ReleaseNotes-2.1.5.txt new file mode 100644 index 0000000000..6985f6b961 --- /dev/null +++ b/ReleaseNotes/ReleaseNotes-2.1.5.txt @@ -0,0 +1,169 @@ +Release notes for Gerrit 2.1.5 +============================== + +Gerrit 2.1.5 is now available: + +link:http://code.google.com/p/gerrit/downloads/detail?name=gerrit-2.1.5.war[http://code.google.com/p/gerrit/downloads/detail?name=gerrit-2.1.5.war] + +This is primarly a bug fix release to 2.1.4, but some additional +new features were included so its named 2.1.5 rather than 2.1.4.1. + +Schema Change +------------- + +There is no change since 2.1.4 (however there is a change since +2.1.3 and earlier). + +New Features +------------ + +Web UI +~~~~~~ +* issue 361 Enable commenting on commit messages ++ +The commit message of a change can now be commented on inline, and +even compared between patch sets, just like any other file contents. +The message is presented as a magical file called 'Commit Message', +in the first row of every change. + +* issue 312 Implement 'Restore Change' to undo 'Abandon Change' ++ +Any user who can abandon a change (the change owner, project owner, +or any site administrator) can now restore the change from Abandoned +status back to Review in Progress. + +* issue 583 Enable/disable download protocols ++ +The new download section in `gerrit.config` controls how the patch +set download links are presented in the web UI. Administrators +can use this section to enable `repo download`, `git://`, or to +disable `http://` style URLs. This section replaces the older +repo.showDownloadCommand. + +* issue 499 Display the size of a patch (lines added/removed) ++ +A 'diffstat' is shown for each file, summarizing the size of the +change on that file in terms of number of lines added or deleted. + +Email Notifications +~~~~~~~~~~~~~~~~~~~ +* issue 452 Include a quick summary of the size of a change in email ++ +After the file listing, a summary totaling the number of files +changed, lines added, and lines removed is displayed. This may +help reviewers to get a quick estimation on the time required for +them to review the change. + +Bug Fixes +--------- + +Web UI +~~~~~~ +* issue 639 Fix keyboard shortcuts under Chrome/Safari ++ +Keyboard shortcuts didn't work properly on modern WebKit browsers +like Chrome and Safari. We kept trying to blame this on the browser, +but it was Gerrit Code Review at fault. The UI was using the wrong +listener type to receive keyboard events in comment editors. Fixed. + +* Make 'u' go up to the last change listing ++ +Previously the 'u' key on a change page was hardcoded to take +the user to their own dashboard. However, if they arrived at the +change through a query such as `is:starred status:open`, this was +quite annoying, as the query had to be started over again to move +to the next matching change. Now the 'u' key goes back to the +query results. + +* issue 671 Honor user's syntax coloring preference in unified view ++ +The user's syntax coloring preference was always ignored in the +unified view, even though the side-by-side view honored it. Fixed. + +* issue 651 Display stars in dependency tables ++ +The 'Depends On' and 'Needed By' tables on a change page did not +show the current user's star settings, even though the star icon +is present and will toggle the user's starred flag for that change. +Fixed. + +Access Control +~~~~~~~~~~~~~~ +* issue 672 Fix branch owner adding exclusive ACL ++ +Branch owners could not add exclusive ACLs within their branch +namespace. This was caused by the server trying to match the leading +`-` entered by the branch administrator against patterns that did +not contain `-`, and therefore always failed. Fixed by removing +the magical `-` from the proposed new specification before testing +the access rights. + +* '@' in ref specs shouldn't be magical. ++ +The dk.brics.automaton package that is used to handle regular +expressions on branch access patterns supports '@' to mean +"any string". We don't want that behavior. Fixed by disabling +the optional features of dk.brics.automaton, thereby making '@' +mean a literal '@' sign as expected. + +* issue 668 Fix inherited Read Access +2 not inheriting ++ +Upload access (aka Read +2) did not inherit properly from the parent +project (e.g. '\-- All Projects \--') if there was any branch level +Read access control within the local project. This was a coding +bug which failed to consider the project inheritance if any branch +(not just the one being uploaded to) denied upload access. + +Misc. +~~~~~ +* issue 641 Don't pass null arguments to hooks ++ +Some hooks crashed inside of the server during invocation because the +`gerrit.canonicalWebUrl` variable wasn't configured, and the hook +was started out of an SSH or background thread context, so the URL +couldn't be assumed from the current request. The bug was worked +around by not passing the `\--change-url` flag in these cases. +Administrators whose hooks always need the flag should configure +`gerrit.canonicalWebUrl`. + +* issue 652 Fix NPE during merge failure on new branch ++ +Submitting a change with a missing dependency to a new branch +resulted in a NullPointerException in the server, because the server +tried to create the branch anyway, even though there was no commit +ready because one or more dependencies were missing. Fixed. + +* Fix NPE while matching `file:^` pattern on deleted files ++ +Sending email notifications crashed with NullPointerException if the +change contained a deleted file and one or more users had a project +watch on that project using a `file:^` pattern in their filter. +Fixed. + +* issue 658 Allow to use refspec shortcuts for push replication ++ +A push refspec of `refs/heads/\*` in replication.config is now +supported as a shorthand notation for `refs/heads/\*:refs/heads/\*`. + +* issue 676 Fix clearing of topic during replace ++ +The topic was cleared if a replacement patch set was uploaded without +the topic name. The topic is now left as-is during replacement +if no new topic was supplied. If a new topic is supplied, it is +changed to match the new topic given. + +* Allow ; and & to seperate parameters in gitweb ++ +gitweb.cgi accepts either ';' or '&' between parameters, but +Gerrit Code Review was only accepting the ';' syntax. Fixed +to support both. + +Documentation +~~~~~~~~~~~~~ +* Fixed example for gerrit create-account. +* gerrit.sh: Correct /etc/default path in error message + +Version +------- + +2765ff9e5f821100e9ca671f4d502b5c938457a5 diff --git a/ReleaseNotes/index.txt b/ReleaseNotes/index.txt index bd98667f57..985c0dd742 100644 --- a/ReleaseNotes/index.txt +++ b/ReleaseNotes/index.txt @@ -4,6 +4,7 @@ Gerrit Code Review - Release Notes [[2_1]] Version 2.1.x ------------- +* link:ReleaseNotes-2.1.5.html[2.1.5] * link:ReleaseNotes-2.1.4.html[2.1.4] * link:ReleaseNotes-2.1.3.html[2.1.3] * link:ReleaseNotes-2.1.2.5.html[2.1.2.5] |