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

Issue 107150043: code review 107150043: syscall: implement setresuid(2) and setresgid(2) on Ope... (Closed)

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

Description

syscall: implement setresuid(2) and setresgid(2) on OpenBSD/FreeBSD/DragonflyBSD Fixes issue 8218.

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

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

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

Unified diffs Side-by-side diffs Delta from patch set Stats (+146 lines, -2 lines) Patch
M src/pkg/syscall/syscall_dragonfly.go View 1 1 chunk +2 lines, -0 lines 0 comments Download
M src/pkg/syscall/syscall_freebsd.go View 1 1 chunk +2 lines, -0 lines 0 comments Download
M src/pkg/syscall/syscall_openbsd.go View 2 chunks +2 lines, -2 lines 0 comments Download
M src/pkg/syscall/zsyscall_dragonfly_386.go View 1 1 chunk +20 lines, -0 lines 0 comments Download
M src/pkg/syscall/zsyscall_dragonfly_amd64.go View 1 1 chunk +20 lines, -0 lines 0 comments Download
M src/pkg/syscall/zsyscall_freebsd_386.go View 1 1 chunk +20 lines, -0 lines 0 comments Download
M src/pkg/syscall/zsyscall_freebsd_amd64.go View 1 1 chunk +20 lines, -0 lines 0 comments Download
M src/pkg/syscall/zsyscall_freebsd_arm.go View 1 2 1 chunk +20 lines, -0 lines 0 comments Download
M src/pkg/syscall/zsyscall_openbsd_386.go View 1 1 chunk +20 lines, -0 lines 0 comments Download
M src/pkg/syscall/zsyscall_openbsd_amd64.go View 1 chunk +20 lines, -0 lines 0 comments Download

Messages

Total messages: 11
worr
syscall: implement setresuid(2) and setresgid(2) on OpenBSD Fixes issue 8218.
10 years, 9 months ago (2014-06-16 13:47:52 UTC) #1
iant
Can you also regenerate zsyscall_openbsd_386.go?
10 years, 9 months ago (2014-06-16 16:36:07 UTC) #2
minux
I've updated the summary of the issue, please replace "Fixes issue 8218." with this: Update ...
10 years, 9 months ago (2014-06-16 19:11:52 UTC) #3
worr
On 2014/06/16 19:11:52, minux wrote: > I've updated the summary of the issue, please replace ...
10 years, 9 months ago (2014-06-16 19:26:28 UTC) #4
worr
Hello golang-dev@googlegroups.com, iant@golang.org, minux@golang.org (cc: golang-codereviews@googlegroups.com), I'd like you to review this change to https://code.google.com/p/go/
10 years, 9 months ago (2014-06-17 01:27:01 UTC) #5
gobot
Replacing golang-dev with golang-codereviews. To the author of this CL: If you are using 'hg ...
10 years, 9 months ago (2014-06-17 01:30:14 UTC) #6
minux
you will also need to generate zsyscall for freebsd/arm.
10 years, 9 months ago (2014-06-17 01:34:12 UTC) #7
worr
Hello golang-codereviews@googlegroups.com, iant@golang.org, minux@golang.org (cc: golang-codereviews@googlegroups.com), Please take another look.
10 years, 9 months ago (2014-06-17 01:38:10 UTC) #8
iant
LGTM
10 years, 8 months ago (2014-06-24 20:29:59 UTC) #9
iant
*** Submitted as https://code.google.com/p/go/source/detail?r=caa2646daa63 *** syscall: implement setresuid(2) and setresgid(2) on OpenBSD/FreeBSD/DragonflyBSD Fixes issue 8218. ...
10 years, 8 months ago (2014-06-24 20:30:33 UTC) #10
worr
10 years, 5 months ago (2014-09-20 04:41:39 UTC) #11
*** Abandoned ***
Sign in to reply to this message.

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