On Tue, Jul 23, 2013 at 07:48:29PM +0200, Zdenek Styblik wrote:
> On Mon, Jul 22, 2013 at 10:33 PM, Jeffrey Bastian <[email protected]> wrote:
> > I have gone through Calxeda's commits and extracted only those that
> > touch the ipmi_cxoem.{c,h} files.
>
> Funny, because that's not what you have attached.
I should clarify: the commits that created ipmi_cxoem.{c,h} also
modified other files like ipmi_lanp.{c,h}. For example,
http://sources.calxeda.com/gitweb/?p=ipmitool.git;a=commit;h=671b435
I just excluded the commits that focused solely on other code like fru,
ekanalyzer, sel, etc.
> I wanted to say adding 'cxoem' shouldn't be a big deal, but I see
> couple issues already.
Thanks for the review. I'll pass the comments along back to the
developers at Calxeda.
> > I've also modified get_lan_param_select() from lib/ipmi_lanp.c as I
> > described in my email last Friday.
>
> Your patch is incorrect. I see nothing wrong with code in question.
I'll add more details to the other email thread since that bug is independent
of the cxoem patches.
> You really weren't serious about this one, were you???!!!
No, I'm an idiot. I forgot the -d flag when I ran 'cvs update' so I
was missing the src/plugins/serial directory. Sorry about that...
PEBKAC... (I use mostly git now so my cvs is very rusty.)
Thanks again!
Jeff
------------------------------------------------------------------------------
See everything from the browser to the database with AppDynamics
Get end-to-end visibility with application monitoring from AppDynamics
Isolate bottlenecks and diagnose root cause in seconds.
Start your free trial of AppDynamics Pro today!
http://pubads.g.doubleclick.net/gampad/clk?id=48808831&iu=/4140/ostg.clktrk
_______________________________________________
Ipmitool-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ipmitool-devel
