Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1408)

Issue 106320043: code review 106320043: codereview: abort 'hg commit' before entering commit me... (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 10 months ago by minux
Modified:
10 years, 10 months ago
Reviewers:
gobot, rsc
CC:
rsc, golang-codereviews
Visibility:
Public.

Description

codereview: abort 'hg commit' before entering commit message. Hook pre-commit runs before 'hg commit' command whereas hook precommit runs after the user has entered commit message.

Patch Set 1 #

Patch Set 2 : diff -r 1fcd603852b0 https://code.google.com/p/go #

Patch Set 3 : diff -r 1fcd603852b0 https://code.google.com/p/go #

Patch Set 4 : diff -r d0e5709db0ea https://code.google.com/p/go #

Patch Set 5 : diff -r d0e5709db0ea https://code.google.com/p/go #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -4 lines) Patch
M lib/codereview/codereview.py View 2 chunks +4 lines, -4 lines 0 comments Download

Messages

Total messages: 6
minux
Hello rsc@golang.org (cc: golang-codereviews@googlegroups.com), I'd like you to review this change to https://code.google.com/p/go
10 years, 10 months ago (2014-06-30 18:29:31 UTC) #1
rsc
And where does uisetup run?
10 years, 10 months ago (2014-06-30 18:53:21 UTC) #2
rsc
LGTM
10 years, 10 months ago (2014-06-30 21:26:56 UTC) #3
minux
*** Submitted as https://code.google.com/p/go/source/detail?r=26ba1fd4913d *** codereview: abort 'hg commit' before entering commit message. Hook pre-commit ...
10 years, 10 months ago (2014-07-19 01:04:59 UTC) #4
gobot
This CL appears to have broken the linux-arm-cheney-imx6 builder. See http://build.golang.org/log/0b43245352ede392671b5e105a303e85881f1398
10 years, 10 months ago (2014-07-19 02:07:06 UTC) #5
minux
10 years, 10 months ago (2014-07-19 02:14:14 UTC) #6
On Fri, Jul 18, 2014 at 10:07 PM, <gobot@golang.org> wrote:

> This CL appears to have broken the linux-arm-cheney-imx6 builder.
> See http://build.golang.org/log/0b43245352ede392671b5e105a303e85881f1398

Looks real, but this change definitely won't cause that.

# GOMAXPROCS=2 runtime -cpu=1,2,4
--- FAIL: TestCgoCrashHandler (0.61s)
crash_test.go:86: output:
  wanted:
main: recovered done
new-thread: recovered done
 second-new-thread: recovered done
main-again: recovered done
FAIL
FAIL runtime 109.351s
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b