-
Notifications
You must be signed in to change notification settings - Fork 7k
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
tests: posix: common: split XSI_REALTIME to standalone test #81859
tests: posix: common: split XSI_REALTIME to standalone test #81859
Conversation
This is a tricky one.
At some point, it would be nice if the standards body made it a proper Option Group. Maybe it's best to keep it in common for now? What do you think @ycsin ? |
Aha - found it here.
Aside from Most of these options (including So I woulud say that we need to define at least one additional Option Group. Based on existing naming conventions, for the So @Pancakem - can you please
There may be other existing tests that should be moved, but I have to get to some other things atm. @ycsin - if you have time at some point, can you please create an issue to create the |
ea46918
to
a623aa3
Compare
We should wait to merge this until 82004 is done |
72b63e5
to
c5fa3a2
Compare
44311c6
to
ea4b831
Compare
@Pancakem - can you please move tests related to the following, into a new
|
Alright on it. |
@cfriedt should |
@Pancakem - no, just move the fsync and fdatasync tests. We want to organize suites by Option Group, and POSIX_FILE_SYSTEM tests should remain here. |
ea4b831
to
438836e
Compare
438836e
to
faede97
Compare
faede97
to
2f120d5
Compare
52bac14
to
9d95f26
Compare
bc7ebe3
to
4dad8c5
Compare
posix.common contains testsuites that can be separated into smaller groups of tests. This change moves mqueue into a singular testsuite at tests/posix/message_passing app directory. Signed-off-by: Marvin Ouma <[email protected]>
4dad8c5
to
a036840
Compare
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.
Preemptive +1
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.
LGTM
Fixes #83634
posix.common contains testsuites that can be separated into smaller groups of tests.
This change moves XSI_REALTIME tests into a singular testsuite at tests/posix/xsi_realtime app directory.