Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

x/sys/unix: add Solaris/Illumos ACL and extended attribute support #24381

Open
dave-fl opened this issue Mar 13, 2018 · 1 comment

Comments

@dave-fl
Copy link

commented Mar 13, 2018

Would it be possible to get these calls added as part of Solaris/Illumos. Must use cgo to interface with file attributes otherwise. Some of these might exist, most don't. Most implementations seem to handle these via a syscall which we don't have in Solaris.

fgetattr,fsetattr,getattrat,setattrat
acl_get,facl_get,acl_set,facl_set, acl_totext, acl_fromtext,unlinkat
openat,fchownat,fstatat,futimesat,renameat, attropen, fdopendir, readdir, read, write

http://illumos.org/man/3lib/libc
http://illumos.org/man/3lib/libsec

What version of Go are you using (go version)?

1.10

Does this issue reproduce with the latest release?

Yes

What operating system and processor architecture are you using (go env)?

amd64/solaris

What did you do?

If possible, provide a recipe for reproducing the error.
A complete runnable program is good.
A link on play.golang.org is best.

What did you expect to see?

Ability to change acls and extended attributes from syscall or os package.

What did you see instead?

Need to use cgo.

@dave-fl dave-fl changed the title Adding Solaris/Illumos ACL Support Adding Solaris/Illumos ACL and extended attribute support Mar 13, 2018

@ianlancetaylor ianlancetaylor changed the title Adding Solaris/Illumos ACL and extended attribute support x/sys/unix: add Solaris/Illumos ACL and extended attribute support Mar 13, 2018

@gopherbot gopherbot added this to the Unreleased milestone Mar 13, 2018

@tklauser tklauser added the OS-Solaris label Mar 14, 2018

@gopherbot

This comment has been minimized.

Copy link

commented Mar 14, 2018

Change https://golang.org/cl/100555 mentions this issue: unix: add Fstatat on Solaris

gopherbot pushed a commit to golang/sys that referenced this issue Mar 14, 2018
unix: add Fstatat on Solaris
Updates golang/go#24381

Change-Id: I9bd15bb7961f02af12a45b12ac62e07d059c744e
Reviewed-on: https://go-review.googlesource.com/100555
Run-TryBot: Tobias Klauser <tobias.klauser@gmail.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Ian Lance Taylor <iant@golang.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.