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

os/signal: SIGSYS documented as both handled and unhandled #67729

Closed
hdonnay opened this issue May 30, 2024 · 3 comments
Closed

os/signal: SIGSYS documented as both handled and unhandled #67729

hdonnay opened this issue May 30, 2024 · 3 comments
Labels
compiler/runtime Issues related to the Go compiler and/or runtime. Documentation NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one.
Milestone

Comments

@hdonnay
Copy link

hdonnay commented May 30, 2024

Go version

go 1.22.3

Output of go env in your module/workspace:

N/A

What did you do?

Visited https://pkg.go.dev/os/signal

What did you see happen?

The documentation claims both:

A SIGQUIT, SIGILL, SIGTRAP, SIGABRT, SIGSTKFLT, SIGEMT, or SIGSYS signal causes the program to exit with a stack dump.

and

It also applies to some signals that otherwise cause no action: SIGUSR1, SIGUSR2, SIGPIPE, SIGALRM, SIGCHLD, SIGCONT, SIGURG, SIGXCPU, SIGXFSZ, SIGVTALRM, SIGWINCH, SIGIO, SIGPWR, SIGSYS, SIGINFO, SIGTHR, SIGWAITING, SIGLWP, SIGFREEZE, SIGTHAW, SIGLOST, SIGXRES, SIGJVM1, SIGJVM2, and any real time signals used on the system.

SIGSYS should be removed from whichever list is incorrect.

What did you expect to see?

SIGSYS listed as exactly one of handled or unhandled.

@gopherbot gopherbot added compiler/runtime Issues related to the Go compiler and/or runtime. Documentation labels May 30, 2024
@mknyszek mknyszek added NeedsFix The path to resolution is known, but the work has not been done. NeedsDecision Feedback is required from experts, contributors, and/or the community before a change can be made. NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. labels Jun 3, 2024
@gopherbot gopherbot removed NeedsFix The path to resolution is known, but the work has not been done. NeedsDecision Feedback is required from experts, contributors, and/or the community before a change can be made. labels Jun 3, 2024
@mknyszek
Copy link
Contributor

mknyszek commented Jun 3, 2024

CC @golang/runtime

@mknyszek mknyszek added this to the Backlog milestone Jun 3, 2024
@ianlancetaylor
Copy link
Contributor

Thanks for noticing. Sending a fix.

@gopherbot
Copy link
Contributor

Change https://go.dev/cl/590915 mentions this issue: os/signal: remove SIGSYS from list of signals that take no action

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. Documentation NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one.
Projects
Development

No branches or pull requests

4 participants