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
Open

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

dave-fl opened this issue Mar 13, 2018 · 1 comment
Labels
compiler/runtime Issues related to the Go compiler and/or runtime. help wanted NeedsFix The path to resolution is known, but the work has not been done. OS-Solaris
Milestone

Comments

@dave-fl
Copy link

dave-fl 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
@ianlancetaylor ianlancetaylor added help wanted NeedsFix The path to resolution is known, but the work has not been done. labels Mar 13, 2018
@gopherbot
Copy link

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
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>
@gopherbot gopherbot added the compiler/runtime Issues related to the Go compiler and/or runtime. label Jul 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
compiler/runtime Issues related to the Go compiler and/or runtime. help wanted NeedsFix The path to resolution is known, but the work has not been done. OS-Solaris
Projects
Status: Triage Backlog
Development

No branches or pull requests

4 participants