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

Issue 6681044: code review 6681044: log/syslog: build on Plan 9 and Windows (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years, 6 months ago by ality
Modified:
11 years, 3 months ago
Reviewers:
minux1, rsc, brainman, mikio
CC:
golang-dev
Visibility:
Public.

Description

log/syslog: build on Plan 9 and Windows This allows logging errors to a local or remote syslog daemon over any protocol except Unix domain sockets.

Patch Set 1 #

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

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

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

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+40 lines, -16 lines) Patch
M src/pkg/go/build/deps_test.go View 1 2 3 1 chunk +2 lines, -6 lines 0 comments Download
M src/pkg/log/syslog/syslog.go View 1 1 chunk +0 lines, -2 lines 0 comments Download
A src/pkg/log/syslog/syslog_notunix.go View 1 1 chunk +25 lines, -0 lines 0 comments Download
M src/pkg/log/syslog/syslog_test.go View 1 4 chunks +13 lines, -0 lines 1 comment Download
R src/pkg/log/syslog/syslog_windows.go View 1 1 chunk +0 lines, -8 lines 1 comment Download

Messages

Total messages: 8
ality
Hello alex.brainman@gmail.com (cc: golang-dev@googlegroups.com), I'd like you to review this change to https://code.google.com/p/go/
11 years, 6 months ago (2012-10-13 03:20:29 UTC) #1
ality
Hello alex.brainman@gmail.com (cc: golang-dev@googlegroups.com), Please take another look.
11 years, 6 months ago (2012-10-13 04:00:30 UTC) #2
minux1
https://codereview.appspot.com/6681044/diff/6001/src/pkg/log/syslog/syslog_test.go File src/pkg/log/syslog/syslog_test.go (right): https://codereview.appspot.com/6681044/diff/6001/src/pkg/log/syslog/syslog_test.go#newcode5 src/pkg/log/syslog/syslog_test.go:5: // +build !windows,!plan9 perhaps change the build tags here? ...
11 years, 6 months ago (2012-10-13 07:16:03 UTC) #3
mikio
s/syslog_notunix.go/syslog_stub.go/ or split syslog_notunix.go into syslog_windows.go and syslog_plan9.go.
11 years, 6 months ago (2012-10-13 16:26:07 UTC) #4
brainman
I do not have problem with this change, but Russ might. We had this discussion ...
11 years, 6 months ago (2012-10-15 03:52:00 UTC) #5
ality
On 2012/10/15 03:52:00, brainman wrote: > I do not have problem with this change, but ...
11 years, 5 months ago (2012-11-08 03:53:37 UTC) #6
rsc
I don't believe this is right. I would like log/syslog to talk to the standard ...
11 years, 5 months ago (2012-11-26 19:14:13 UTC) #7
ality
11 years, 3 months ago (2013-01-07 23:10:39 UTC) #8
*** Abandoned ***
Sign in to reply to this message.

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