Skip to content

Conversation

@oxzi
Copy link
Contributor

@oxzi oxzi commented Sep 15, 2025

Fix build of check_swap for both CHECK_SWAP_SWAPCTL_SVR4 and CHECK_SWAP_SWAPCTL_BSD build guards.

$ make check_swap
CC check_swap.d/swap.o
check_swap.d/swap.c:58:32: error: too few arguments to function call, single argument 'config' was not specified
58 | return getSwapFromSwapctl_BSD();
| ~~~~~~~~~~~~~~~~~~~~~~ ^
././check_swap.d/check_swap.h:48:13: note: 'getSwapFromSwapctl_BSD' declared here
48 | swap_result getSwapFromSwapctl_BSD(swap_config config);
| ^ ~~~~~~~~~~~~~~~~~~
1 error generated.
*** Error 1 in [...]/monitoring-plugins/plugins (Makefile:2869 'check_swap.d/swap.o': @echo " CC ...)

@oxzi oxzi force-pushed the check_swap-fix-bsd-svr4 branch from d6a9dd8 to 55d904f Compare September 15, 2025 22:18
Fix build of check_swap for both CHECK_SWAP_SWAPCTL_SVR4 and
CHECK_SWAP_SWAPCTL_BSD build guards.

> $ make check_swap
>   CC       check_swap.d/swap.o
> check_swap.d/swap.c:58:32: error: too few arguments to function call, single argument 'config' was not specified
>    58 |         return getSwapFromSwapctl_BSD();
>       |                ~~~~~~~~~~~~~~~~~~~~~~ ^
> ././check_swap.d/check_swap.h:48:13: note: 'getSwapFromSwapctl_BSD' declared here
>    48 | swap_result getSwapFromSwapctl_BSD(swap_config config);
>       |             ^                      ~~~~~~~~~~~~~~~~~~
> 1 error generated.
> *** Error 1 in [...]/monitoring-plugins/plugins (Makefile:2869 'check_swap.d/swap.o': @echo "  CC   ...)
@oxzi oxzi force-pushed the check_swap-fix-bsd-svr4 branch from 55d904f to 8803642 Compare September 15, 2025 22:19
@waja waja requested a review from RincewindsHat September 16, 2025 05:27
Copy link
Member

@RincewindsHat RincewindsHat left a comment

Choose a reason for hiding this comment

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

Oversight on my part ...

@RincewindsHat RincewindsHat merged commit c1f0f11 into monitoring-plugins:master Sep 16, 2025
7 checks passed
@RincewindsHat
Copy link
Member

Thank you @oxzi :-)

@oxzi oxzi deleted the check_swap-fix-bsd-svr4 branch September 16, 2025 11:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants