code review 128110043: go.sys/unix: delete nacl

go.sys/unix: delete nacl
It's a peculiar environment that probably doesn't belong here.
We can bring it back easily if we need it.

[r, 2014-08-12 23:04:43]

Hello rsc (cc: golang-codereviews@googlegroups.com),

I'd like you to review this change to
https://code.google.com/p/go.sys

[dave_cheney.net, 2014-08-12 23:33:04]

LGTM. I agree with rsc that there isn't a strong case to handle nacl in
this package.
Reviewers: rsc,

Message:
Hello rsc (cc: golang-codereviews@googlegroups.com),

I'd like you to review this change to
https://code.google.com/p/go.sys


Description:
go.sys/unix: delete nacl
It's a peculiar environment that probably doesn't belong here.
We can bring it back easily if we need it.

Please review this at https://codereview.appspot.com/128110043/

Affected files (+1, -4678 lines):
  R unix/asm_nacl_386.s
  R unix/asm_nacl_amd64p32.s
  R unix/asm_nacl_arm.s
  M unix/env_unix.go
  R unix/fd_nacl.go
  R unix/fs_nacl.go
  M unix/mkall.sh
  M unix/mksyscall.pl
  R unix/net_nacl.go
  R unix/srpc_nacl.go
  R unix/syscall_nacl.go
  R unix/syscall_nacl_386.go
  R unix/syscall_nacl_amd64p32.go
  R unix/syscall_nacl_arm.go
  R unix/tables_nacl.go
  R unix/time_nacl_386.s
  R unix/time_nacl_amd64p32.s
  R unix/time_nacl_arm.s
  R unix/unzip_nacl.go
  R unix/zsyscall_nacl_386.go
  R unix/zsyscall_nacl_amd64p32.go
  R unix/zsyscall_nacl_arm.go


-- 
You received this message because you are subscribed to the Google Groups
"golang-codereviews" group.
To unsubscribe from this group and stop receiving emails from it, send an
email to golang-codereviews+unsubscribe@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

[rsc, 2014-08-12 23:48:54]

LGTM

[r, 2014-08-12 23:58:22]

*** Submitted as
https://code.google.com/p/go/source/detail?r=42f74880d132&repo=sys ***

go.sys/unix: delete nacl
It's a peculiar environment that probably doesn't belong here.
We can bring it back easily if we need it.

LGTM=dave, rsc
R=rsc, dave
CC=golang-codereviews
https://codereview.appspot.com/128110043

[minux, 2014-08-13 00:46:13]

what about the IRT support for nacl?
i think we will need go.sys/nacl package then?

[rsc, 2014-08-13 01:18:43]

we don't need it now.