Re: [Ipmitool-devel] [Dell] [Patch 1/6] Dell Extension commands and Bug Fixes

2012-02-09 Thread Jan Safranek
On 08/11/2011 02:36 PM, srinivas_g_go...@dell.com wrote: Hi , This is the patch that contains bug fixes for Non-Dell components Hello, I can see all other patches in this patch set were accepted into upstream CVS except this one. Is there any reason? The patch does not contain any description

Re: [Ipmitool-devel] ipmitool fix to display percentage units correctly

2012-02-09 Thread Jim Mank
All, After reading and replying to Al's comments about how percent should be handled, I coded up the unit string conversion function as follows. Note: I did not make any changes to deal with the cases where the textual representation of based and/or modifier is unspecified in the two

Re: [Ipmitool-devel] -P switch missing

2012-02-09 Thread Zdenek Styblik
On Thu, Feb 9, 2012 at 11:06 PM, carl brunning ca...@cblinux.co.uk wrote: I just want to know if the –P for Password is missing or broken Hello. What version of ipmitool? What's the source aka where did it come from aka where did you get it? Is it specific build eg. without LAN/LAN+

Re: [Ipmitool-devel] ipmitool fix to display percentage units correctly

2012-02-09 Thread Zdenek Styblik
Hi, it looks fine by me. One thing though. Since it returns pointer to static char, should it be: static char * ipmi_sdr_get_unit_string(uint8_t pct, uint8_t type, uint8_t base, uint8_t modifier) { ... } as well? Regards, Z. On Thu, Feb 9, 2012 at 10:01 PM, Jim Mank jm...@hp.com wrote: [...]

Re: [Ipmitool-devel] -P switch missing

2012-02-09 Thread carl brunning
Hi I downloaded ipmitool-1.8.11.tar.gz and compile on opensolairs 10. I did the normal Configure make make install does not work but am not worry as Solairs has some directory that are different