code review 138840044: undo CL 107150043 / caa2646daa63

undo CL 107150043 / caa2646daa63

preparing for the syscall package freeze.
the change for issue 8218 is only applied to go.sys/unix.

««« original CL description
syscall: implement setresuid(2) and setresgid(2) on OpenBSD/FreeBSD/DragonflyBSD

Fixes issue 8218.

LGTM=iant
R=golang-codereviews, iant, minux
CC=golang-codereviews
https://codereview.appspot.com/107150043

Committer: Ian Lance Taylor <iant@golang.org>
»»»

[mikio, 2014-08-29 22:33:39]

Hello r@golang.org, iant@golang.org, golang-codereviews@googlegroups.com (cc:
golang-codereviews@googlegroups.com),

I'd like you to review this change to
https://go.googlecode.com/hg/

[r, 2014-08-29 22:40:09]

LGTM

[mikio, 2014-08-29 22:52:50]

*** Submitted as https://code.google.com/p/go/source/detail?r=12d4c082a491 ***

undo CL 107150043 / caa2646daa63

preparing for the syscall package freeze.
the change for issue 8218 is only applied to go.sys/unix.

««« original CL description
syscall: implement setresuid(2) and setresgid(2) on OpenBSD/FreeBSD/DragonflyBSD

Fixes issue 8218.

LGTM=iant
R=golang-codereviews, iant, minux
CC=golang-codereviews
https://codereview.appspot.com/107150043

Committer: Ian Lance Taylor <iant@golang.org>
»»»

LGTM=r
R=r, iant, golang-codereviews
CC=golang-codereviews
https://codereview.appspot.com/138840044

[gobot, 2014-08-29 22:53:59]

This CL appears to have broken the windows-amd64-perf builder.
See http://build.golang.org/log/321f25cb21aaa1bb122fdfa08423075ebc56f55a

[worr, 2014-09-12 07:00:57]

On 2014/08/29 22:52:50, mikio wrote:
> *** Submitted as https://code.google.com/p/go/source/detail?r=12d4c082a491 ***
> 
> undo CL 107150043 / caa2646daa63
> 
> preparing for the syscall package freeze.
> the change for issue 8218 is only applied to go.sys/unix.
> 
> ««« original CL description
> syscall: implement setresuid(2) and setresgid(2) on
OpenBSD/FreeBSD/DragonflyBSD
> 
> Fixes issue 8218.
> 
> LGTM=iant
> R=golang-codereviews, iant, minux
> CC=golang-codereviews
> https://codereview.appspot.com/107150043
> 
> Committer: Ian Lance Taylor <mailto:iant@golang.org>
> »»»
> 
> LGTM=r
> R=r, iant, golang-codereviews
> CC=golang-codereviews
> https://codereview.appspot.com/138840044

Will this change be reapplied? I submitted the patch that this reverts because I
use these syscalls on OpenBSD.

[mikio, 2014-09-12 07:20:21]

On Fri, Sep 12, 2014 at 4:00 PM,  <ay1244@gmail.com> wrote:

> Will this change be reapplied?I submitted the patch that this reverts
> because I use these syscalls on OpenBSD.

that's unfortunate, see https://golang.org/s/go1.4-syscall
i also did revert a few patches i submitted, urgh. looks like
go.sys/unix has not been curated yet, hm.

[worr, 2014-09-12 22:08:39]

On 2014/09/12 07:20:21, mikio wrote:
> On Fri, Sep 12, 2014 at 4:00 PM,  <mailto:ay1244@gmail.com> wrote:
> 
> > Will this change be reapplied?I submitted the patch that this reverts
> > because I use these syscalls on OpenBSD.
> 
> that's unfortunate, see https://golang.org/s/go1.4-syscall
> i also did revert a few patches i submitted, urgh. looks like
> go.sys/unix has not been curated yet, hm.

That is unfortunate.

Should I draw up a patch for go.sys?

[r, 2014-09-16 19:46:40]

Yes please.

On Fri, Sep 12, 2014 at 3:08 PM,  <ay1244@gmail.com> wrote:
> On 2014/09/12 07:20:21, mikio wrote:
>
>> On Fri, Sep 12, 2014 at 4:00 PM,  <mailto:ay1244@gmail.com> wrote:
>
>
>> > Will this change be reapplied?I submitted the patch that this
>
> reverts
>>
>> > because I use these syscalls on OpenBSD.
>
>
>> that's unfortunate, see https://golang.org/s/go1.4-syscall
>> i also did revert a few patches i submitted, urgh. looks like
>> go.sys/unix has not been curated yet, hm.
>
>
> That is unfortunate.
>
> Should I draw up a patch for go.sys?
>
> https://codereview.appspot.com/138840044/