<br><div class="gmail_quote">2012/10/5 Charles Lepple <span dir="ltr"><<a href="mailto:clepple@gmail.com" target="_blank">clepple@gmail.com</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On Oct 4, 2012, at 6:50 PM, Arnaud Quette wrote:<br>
<br>
> +/* IPMI auth defines, simply using FreeIPMI defines */<br>
> +#ifndef IPMI_AUTHENTICATION_TYPE_NONE<br>
> +  #define IPMI_AUTHENTICATION_TYPE_NONE                  0x00<br>
> +  #define IPMI_AUTHENTICATION_TYPE_MD2                   0x01<br>
> +  #define IPMI_AUTHENTICATION_TYPE_MD5                   0x02<br>
> +  #define IPMI_AUTHENTICATION_TYPE_STRAIGHT_PASSWORD_KEY 0x04<br>
> +  #define IPMI_AUTHENTICATION_TYPE_OEM_PROP              0x05<br>
> +  #define IPMI_AUTHENTICATION_TYPE_RMCPPLUS              0x06<br>
> +#endif /* IPMI_AUTHENTICATION_TYPE_NONE */<br>
<br>
</div>Hmm. Shouldn't some of the FreeIPMI stuff be moved out of the nut-scanner main() function, possibly into another file? Then it would be easier to do conditional compilation in case it is not installed.<br>
<br>
Lots of Buildbot failures when FreeIPMI is not available, e.g. <a href="http://buildbot.networkupstools.org/public/nut/builders/Ubuntu-natty-x86/builds/230" target="_blank">http://buildbot.networkupstools.org/public/nut/builders/Ubuntu-natty-x86/builds/230</a><span class="HOEnZb"></span><br>
</blockquote></div><br>gone to late and too quickly to bed, with a last mod (0 => IPMI_PRIVILEGE_LEVEL_ADMIN) without re-checking compilation is FreeIPMI is not available! I already hit that one in a previous commit tentative. tyro error, sorry.<br>
<br>these default IPMI settings have to happen before getopt processing.<br clear="all">as I didn't wanted to abuse ifdefs again here: there are yet too many, that code already needs simplification.<br><br>I've fixed this compilation error in 3741. I'm monitoring buildslaves, before returning to real life.<br>
<br>cheers<br>Arno<br><br>