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

ledmon: change returns and exits for macro #212

Merged
merged 1 commit into from
Mar 26, 2024

Conversation

mku514k
Copy link
Contributor

@mku514k mku514k commented Mar 25, 2024

Commit f08dd2c ("Fix build with Musl libc (#139)") introduced a new way of handling exits. For ledmon statuses to display properly EXIT() macro must be called instead of exit() and return in main.

Switch exit() and (main) returns in ledmon to EXIT().

Commit f08dd2c ("Fix build with Musl libc (intel#139)") introduced a new
way of handling exits. For ledmon statuses to display properly EXIT()
macro must be called instead of exit() and return in main.

Switch exit() and (main) returns in ledmon to EXIT().

Signed-off-by: Mateusz Kusiak <mateusz.kusiak@intel.com>
@mku514k mku514k mentioned this pull request Mar 25, 2024
@mtkaczyk
Copy link
Contributor

please remove change-id tag :)

@mku514k
Copy link
Contributor Author

mku514k commented Mar 26, 2024

please remove change-id tag :)

There is no change-id. Checkpatch fails because it can't find a commit with hash that is referenced, but we know it's valid since github redirects to it properly. I suppose it is due to being run in a container.

@mtkaczyk mtkaczyk merged commit c527fe7 into intel:master Mar 26, 2024
10 of 11 checks passed
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.

None yet

5 participants