-
Notifications
You must be signed in to change notification settings - Fork 89
Setup a basic run of cmake-smoke-test in CI #480
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
Conversation
a24e15b
to
bd2ec17
Compare
@@ -166,29 +168,18 @@ enum OS { | |||
} | |||
|
|||
extension Process { | |||
func run() async throws { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Process execution is hanging the CI action, haven't fully debugged it yet
3d6a592
to
b95a058
Compare
479b177
to
142181b
Compare
@swift-ci test |
|
||
var mostSignals = sigset_t() | ||
sigemptyset(&mostSignals) | ||
for i in 1 ..< SIGSYS { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
..<SIGRTMAX
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think anything is using the realtime signals, and there could be a gap between SIGSYS and SIGRTMIN on some implementations
Add test coverage of the Cmake build