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

Fix build when MUNIT_NO_BUFFER is defined and MUNIT_NO_FORK is not #103

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

Conversation

Syeberman
Copy link

@Syeberman Syeberman commented Apr 14, 2024

munit_replace_stderr is used here, surrounded by #if !defined(MUNIT_NO_BUFFER):

munit/munit.c

Line 1426 in fbbdf14

const volatile int orig_stderr = munit_replace_stderr(stderr_buf);

It is also used here, surrounded by #if !defined(MUNIT_NO_FORK):

munit/munit.c

Line 1355 in fbbdf14

orig_stderr = munit_replace_stderr(stderr_buf);

This PR ensures that munit_replace_stderr is available when MUNIT_NO_BUFFER is defined but MUNIT_NO_FORK is not.

@Syeberman Syeberman changed the title Fix build when both MUNIT_NO_BUFFER and MUNIT_NO_FORK are defined Fix build when MUNIT_NO_BUFFER is defined and MUNIT_NO_FORK is not Nov 30, 2024
@Syeberman Syeberman force-pushed the fix-build-no-buffer-no-fork branch from 1911289 to d21bed3 Compare November 30, 2024 22:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant