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.

Sign up for GitHub

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

posix: signal: include_next to get the libc signal() decl #60948

Closed
wants to merge 4 commits into from
Closed

posix: signal: include_next to get the libc signal() decl #60948

wants to merge 4 commits into from

Commits on Aug 10, 2023

  1. libc: minimal: declare sig_atomic_t and signal() 

    We don't actually implement signal() in Zephyr, but some users
require the declaration.

Additionally, typedef sig_atomic_t since it is part of ISO.

Signed-off-by: Christopher Friedt <cfriedt@meta.com>
    @cfriedt
    cfriedt committed Aug 10, 2023
    Configuration menu
    Copy the full SHA
    35f11b3 View commit details
    Browse the repository at this point in the history

  2. posix: signal: some declarations were not visible 

    Some declarations were not visible when including the POSIX
version of `<time.h>` or `<signal.h>`.

Signed-off-by: Christopher Friedt <cfriedt@meta.com>
    @cfriedt
    cfriedt committed Aug 10, 2023
    Configuration menu
    Copy the full SHA
    f3df0c2 View commit details
    Browse the repository at this point in the history

  3. tests: c_lib: test for declaration of signal() 

    We don't actually implement signal() in Zephyr, but some users
require the declaration.

Signed-off-by: Christopher Friedt <cfriedt@meta.com>
    @cfriedt
    cfriedt committed Aug 10, 2023
    Configuration menu
    Copy the full SHA
    d585d09 View commit details
    Browse the repository at this point in the history

  4. tests: posix: signal: account for ANSI / ISO C signal bits 

    The `signal()` function was part of ANSI-C and predates
POSIX, so it should be visible (even if it completely
unsupported in Zephyr currently).

The `sig_atomic_t` type is part of ISO C, regardless of
whether POSIX is enabled.

Signed-off-by: Christopher Friedt <cfriedt@meta.com>
    @cfriedt
    cfriedt committed Aug 10, 2023
    Configuration menu
    Copy the full SHA
    d566e1f View commit details
    Browse the repository at this point in the history