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

tests: posix: common: separate posix xopen streams to standalone test #81445

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Pancakem
Copy link

Fixes #79454
posix.common contains testsuites that can be separated into smaller groups of tests. This change moves stropts into a singular testsuite at tests/posix/xopen_streams app directory.

@zephyrbot zephyrbot added the area: POSIX POSIX API Library label Nov 15, 2024
Copy link
Member

@cfriedt cfriedt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please use xsi_streams as the option group name / directory name.


cmake_minimum_required(VERSION 3.20.0)
find_package(Zephyr REQUIRED HINTS $ENV{ZEPHYR_BASE})
project(posix_xopen_streams)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be xsi_streams to match the Open Group name which should presumably be XSI_STREAMS although the spec is not explicitly clear about that (the feature test macro is _XOPEN_STREAMS).

https://pubs.opengroup.org/onlinepubs/009695399/basedefs/xbd_chap02.html

So please adjust the testsuite to use xsi_streams instead of posix_xopen_streams, including the directory name too.

Thanks!

posix.common contains testsuites that can be separated into smaller
groups of tests. This change moves stropts into a singular
testsuite at tests/posix/xsi_streams app directory.

Signed-off-by: Marvin Ouma <pancakesdeath@protonmail.com>
@Pancakem Pancakem force-pushed the separate_posix_xopen_streams_to_standalone_test branch from 1287e47 to 26a057d Compare November 16, 2024 06:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: POSIX POSIX API Library
Projects
None yet
Development

Successfully merging this pull request may close these issues.

tests: posix: common: separate into smaller testsuites
3 participants