Hello All, We have linked peer review discussions on git@xxxxxxxxxxxxxxx to their respective commits within the main git.git repository. You can view the linked reviews from 2012 until present in the GitHub repo at: https://github.com/mmukadam/git/tree/review If you want to search for the reviews related to a commit, you can type in the full hash code here: http://users.encs.concordia.ca/~m_mukada/git-review-tracker.html or append the commit hash to the following url: https://github.com/mmukadam/git/tree/review/<commit_hash> Note, if you get a '404 page not found' from GitHub it means that we couldn't find a review to link the commit to. It's a bit hard to view the reviews on GitHub because the 'review' branch was designed to be viewed on the command line using a set of simple scripts that sit on top of the standard git commands. For example: to view all reviews Hamano is involved in you can do: git reviewed --log-reviewer hamano to view the reviews of particular commit: git reviewed --log 1a1832eb085e5bca198735e5d0e766a3cb61b8fc You can also create new reviews on a commit and mail them as well as import reviews from email. Note the command is git-reivewED because git-review is associated with Gerrit. The tool is beta, but it just creates a detached review branch, so it won't affect master. To remove it all you do is delete the scripts and the 'review' branch. More info available here: The online man-page is here: http://users.encs.concordia.ca/~m_mukada/git-review.html In order to install git-review, you need to clone the repository: https://github.com/mmukadam/git-reviewed.git The online tutorial is available here: http://users.encs.concordia.ca/~m_mukada/git-review-tutorial.html Sample output: Let's say you want to see the reviews related to commit: --------------------------------------------------------------------------------- git reviewed --log 9ce415d972a04df4bfceaad0fab8eaea9a66997e Review: 74d203accc341993dbed507fceceb2841670ce37 Commit Reviewed: 9ce415d972a04df4bfceaad0fab8eaea9a66997e Author: Johannes Sixt <j6t@xxxxxxxx> AuthorDate: Fri Jun 7 22:53:27 2013 +0200 Reviewer: Junio C Hamano gitster@xxxxxxxxx <gitster@xxxxxxxxx> ReviewDate: Thu Feb 13 11:35:13 2014 -0800 Re: [PATCH] tests: turn on network daemon tests by default Review: e80ebaa1fdd132a13bfbc6f87cdd4ecec7f2b715 Commit Reviewed: 9ce415d972a04df4bfceaad0fab8eaea9a66997e Author: Johannes Sixt <j6t@xxxxxxxx> AuthorDate: Fri Jun 7 22:53:27 2013 +0200 Reviewer: Junio C Hamano gitster@xxxxxxxxx <gitster@xxxxxxxxx> ReviewDate: Wed Feb 12 14:34:52 2014 -0800 Re: [PATCH] tests: turn on network daemon tests by default Review: 8da03766afcd60e432ffdd2d6f549e2ed02a4bc9 Commit Reviewed: 9ce415d972a04df4bfceaad0fab8eaea9a66997e Author: Johannes Sixt <j6t@xxxxxxxx> AuthorDate: Fri Jun 7 22:53:27 2013 +0200 Reviewer: Jeff King peff@xxxxxxxx <peff@xxxxxxxx> ReviewDate: Mon Feb 10 16:29:37 2014 -0500 [PATCH] tests: turn on network daemon tests by default To show the email review discussion for a single review: --------------------------------------------------------------------------------- git reviewed --view 74d203accc341993dbed507fceceb2841670ce37 Commit Reviewed: 9ce415d972a04df4bfceaad0fab8eaea9a66997e Reviewer: Junio C Hamano <gitster@xxxxxxxxx> Reviewer Date: Thu, 13 Feb 2014 11:35:13 -0800 --------------------------------------------------------------- >From : Junio C Hamano <gitster@xxxxxxxxx> Message-Id : <xmqqfvnmlsb2.fsf@xxxxxxxxxxxxxxxxxxxxxxxxxxx> Date : Thu, 13 Feb 2014 11:35:13 -0800 Subject: Re: [PATCH] tests: turn on network daemon tests by default References: <20140210191521.GA3112@xxxxxxxxxxxxxxxxxxxxx> <20140210212931.GA16154@xxxxxxxxxxxxxxxxxxxxx> <xmqqa9dxpgw9.fsf@xxxxxxxxxxxxxxxxxxxxxxxxxxx> <20140211200445.GA27946@xxxxxxxxxxxxxxxxxxxxx> <xmqqzjlxnqvw.fsf@xxxxxxxxxxxxxxxxxxxxxxxxxxx> <20140212214753.GA6799@xxxxxxxxxxxxxxxxxxxxx> <xmqq38jom037.fsf@xxxxxxxxxxxxxxxxxxxxxxxxxxx> In-Reply-To: <xmqq38jom037.fsf@xxxxxxxxxxxxxxxxxxxxxxxxxxx> (Junio C. Hamano's message of "Wed, 12 Feb 2014 14:34:52 -0800") Cc: Duy Nguyen <pclouds@xxxxxxxxx>, Git Mailing List <git@xxxxxxxxxxxxxxx>, "Shawn O. Pearce" <spearce@xxxxxxxxxxx> commit: 9ce415d972a04df4bfceaad0fab8eaea9a66997e Junio C Hamano <gitster@xxxxxxxxx> writes: > Jeff King <peff@xxxxxxxx> writes: > >> test_normalize_tristate GIT_TEST_DAEMON > > Heh, great minds think alike. This is what I am playing with, > without committing (because I do like your "ask config if this is a > kind of various boolean 'false' representations, which I haven't > managed to add to it). [snip] --- t/lib-git-daemon.sh | 8 ++++--- t/lib-httpd.sh | 22 +++++++++---------- t/test-lib-functions.sh | 58 +++++++++++++++++++++++++++++++++++++++++++++++++ 3 files changed, 74 insertions(+), 14 deletions(-) diff --git a/t/lib-git-daemon.sh b/t/lib-git-daemon.sh index 394b06b..615bf5d 100644 --- a/t/lib-git-daemon.sh +++ b/t/lib-git-daemon.sh @@ -16,9 +16,10 @@ # stop_git_daemon # test_done -if test -z "$GIT_TEST_GIT_DAEMON" +test_tristate GIT_TEST_GIT_DAEMON +if test "$GIT_TEST_GIT_DAEMON" = false then - skip_all="git-daemon testing disabled (define GIT_TEST_GIT_DAEMON to enable)" + skip_all="git-daemon testing disabled (unset GIT_TEST_GIT_DAEMON to enable)" test_done fi [snip] # The following mingw_* functions obey POSIX shell syntax, but are actually # bash scripts, and are meant to be used only with bash on Windows. -- 1.9.0-rc3-260-g4cf525c To see all the reviews anyone is involved in: ------------------------------------------------------------------- git reviewed --log reviewer hamano Review: ecb6f98955700169ed6f88d929babd7d0f770116 Commit Reviewed: 9c8ce7397bac108f83d77dfd96786edb28937511 Author: Michael J Gruber <git@xxxxxxxxxxxxxxxxxxxx> AuthorDate: Thu Feb 13 11:35:08 2014 +0100 Reviewer: Junio C Hamano gitster@xxxxxxxxx <gitster@xxxxxxxxx> ReviewDate: Thu Feb 13 13:25:23 2014 -0800 Re: [PATCH] release notes: typo fixes Review: 74d203accc341993dbed507fceceb2841670ce37 Commit Reviewed: 9ce415d972a04df4bfceaad0fab8eaea9a66997e Author: Johannes Sixt <j6t@xxxxxxxx> AuthorDate: Fri Jun 7 22:53:27 2013 +0200 Reviewer: Junio C Hamano gitster@xxxxxxxxx <gitster@xxxxxxxxx> ReviewDate: Thu Feb 13 11:35:13 2014 -0800 Re: [PATCH] tests: turn on network daemon tests by default Review: e80ebaa1fdd132a13bfbc6f87cdd4ecec7f2b715 Commit Reviewed: 9ce415d972a04df4bfceaad0fab8eaea9a66997e Author: Johannes Sixt <j6t@xxxxxxxx> AuthorDate: Fri Jun 7 22:53:27 2013 +0200 Reviewer: Junio C Hamano gitster@xxxxxxxxx <gitster@xxxxxxxxx> ReviewDate: Wed Feb 12 14:34:52 2014 -0800 Re: [PATCH] tests: turn on network daemon tests by default Please let us know what you think about this tool as it will help us in improving it. If you'd like us to extract the reviews related to a different repository, please let us know. This is part of my Msc thesis, so any feedback is appreciated. We thank you for you time. Cheers, Murtuza -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html