Navigation Menu

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

Jump to bottom

x/sys/unix: zero-sized field may be removed by go1.5's change. #14440

Closed
hirochachacha opened this issue Feb 21, 2016 · 4 comments
Closed

x/sys/unix: zero-sized field may be removed by go1.5's change. #14440

hirochachacha opened this issue Feb 21, 2016 · 4 comments
Labels
compiler/runtime Issues related to the Go compiler and/or runtime. FrozenDueToAge
Milestone
Unreleased

Comments

@hirochachacha
Copy link
Contributor

Currently source code seems to be generated by go <= 1.4.

https://godoc.org/golang.org/x/sys/unix#InotifyEvent

Because of go 1.5's change, we may want to remove zero-size field.

https://golang.org/doc/go1.5#cgo

I don't know this is intentional. If not we need to updated code.
I also want to know which OS version are targeted for x/sys/unix.
Clarifying it somewhere is reasonable for me.

Thank you.
@ianlancetaylor ianlancetaylor added this to the Unreleased milestone Feb 21, 2016
@ianlancetaylor
Copy link
Contributor

Thanks for the bug report.

There is no clear documentation for which operating system versions are supported by x/sys/unix. That would be desirable, but it seems an unrelated issue. Note that the set of supported versions is distinct from the question of which version is used to generate the files.

@hirochachacha
Copy link
Contributor Author

it seems an unrelated issue.

Indeed.

Honestly, I want to post more general question to golang-dev:
"which version of OS and Go is preferable for code generation?"
I tried to post it, but it failed silently. So I filed a relevant issue instead. Sorry for that.

@hirochachacha
Copy link
Contributor Author

Oops. It seems to that google group has delayed. sorry.

@dominikh dominikh mentioned this issue Mar 3, 2019
Closed
@gopherbot gopherbot added the compiler/runtime Issues related to the Go compiler and/or runtime. label Jul 7, 2022
@prattmic
Copy link
Member

Done in https://go.dev/cl/37943.

@golang golang locked and limited conversation to collaborators Jul 29, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
compiler/runtime Issues related to the Go compiler and/or runtime. FrozenDueToAge
Projects
None yet
Milestone
Unreleased
Development

No branches or pull requests
4 participants
@hirochachacha @prattmic @ianlancetaylor @gopherbot