-
Notifications
You must be signed in to change notification settings - Fork 64
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
test solaris #265
test solaris #265
Conversation
a2f6e14
to
7ac516d
Compare
This change is weird. It introduces an int function without return?
|
Right, should be a void. The sole point is that there is any symbol at all in the lib, else Solaris build fails. It is never used. |
I see. Could this be wrapped pre-compiler conditionals for Solaris only? |
which triggerred build errors/warnings on some platforms We could correct this with detection for Solaris, but for that we would need to change our autoconf configuration, for what I currently have no time. So I just correct the invalid type. This causes no harm except for maybe some bytes of unnecessary object code. Comment now describes the use case. Thanks to Andreas Stieger for bringing this up. see also rsyslog#265 (comment)
I have now just corrected the invalid type -- digging out the right OS detection is time-consuming for me. It's better to have this solution now. The commit comment says it all. |
@andreasstieger the change is merged, please let me know if this works for you (at least as a temporary solution). Do you need a public release (or would prefer it)? |
#273 0a044b1 works, although we went for a different solution. There is no need for a new release just for build fixes, proper distributions are happy to patch the source with upstream fixes or their own. |
Thanks for the reply. I hope you go without the patch in the future. As time permits, I'll also see that we do a check for Solaris, but I am wading through a very long backlog. |
which triggerred build errors/warnings on some platforms We could correct this with detection for Solaris, but for that we would need to change our autoconf configuration, for what I currently have no time. So I just correct the invalid type. This causes no harm except for maybe some bytes of unnecessary object code. Comment now describes the use case. Thanks to Andreas Stieger for bringing this up. see also rsyslog#265 (comment)
No description provided.