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

Release 2022.06.02 does not work with the latest naemon version 1.4.2 #159

Open
savv3 opened this issue Jan 16, 2024 · 4 comments
Open

Release 2022.06.02 does not work with the latest naemon version 1.4.2 #159

savv3 opened this issue Jan 16, 2024 · 4 comments

Comments

@savv3
Copy link

savv3 commented Jan 16, 2024

Getting the following error:
Error: Could not load module '/usr/lib64/merlin/merlin.so' -> /usr/lib64/merlin/merlin.so: undefined symbol: comment_list

Naemon release 1.4.2 has the following changes:
https://github.com/naemon/naemon-core/releases/tag/v1.4.2

I've tried compiling from source, using naemon 1.4.2 but getting errors

module/module.c:681:21: error: 'comment' has no member named 'nexthash'
     next_cmnt = cmnt->nexthash;
                     ^
module/module.c:690:16: error: 'comment_list' undeclared (first use in this function)
    for (cmnt = comment_list; cmnt; cmnt = next_cmnt) {

Commenting out those rows will allow compile to work. And naemon will start after that.

@eschoeller
Copy link

I have run into the same problem. But I was not able to comment out line 681 and the entire for() at line 690. This yielded another error and I didn't dig into it any further. For now I have held back naemon to version 1.4.1. I hope the fix is easy and a new release can be made, which should also address issue #158

@eschoeller
Copy link

Are there any updates on this?

willhaines added a commit to UCBoulder/oit-sepe-naemon-build that referenced this issue Feb 12, 2024
@dirtyren
Copy link

dirtyren commented Jun 3, 2024

Hey guys, I have patched merlin to compile for naemon 1.4.2 and I'd appreciate if someone would try it to see if everything is working fine.

https://github.com/dirtyren/monitor-merlin/tree/naemon-1.4.2

If all is ok I will open a PR.

@eschoeller
Copy link

Great news! I will give this a try in the next week or so.

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

No branches or pull requests

3 participants