Hi, I will update my patch with a fix, but I also like Darran's solution: just delete the class and/or use InetAddress
On Tue, Oct 29, 2013 at 6:42 PM, Kelven Yang <[email protected]> wrote: > Windows ipconfig is not supported, a fix is welcome. > > To run a CS management server cluster on Windows. manual configuration > is required. cluster.node.ip needs to configured correctly in db.properties > > Kelven > > From: Alex Huang <[email protected]> > Date: Tuesday, October 29, 2013 at 8:43 AM > To: Laszlo Hornyak <[email protected]>, "[email protected]" > <[email protected]> > Cc: Kelven Yang <[email protected]> > Subject: RE: ifconfig and MacAddress > > I can confirm the code’s only there because Java didn’t have it before.* > *** > > ** ** > > --Alex**** > > ** ** > > *From:* Laszlo Hornyak > [mailto:[email protected]<[email protected]>] > > *Sent:* Monday, October 28, 2013 11:50 PM > *To:* [email protected] > *Cc:* Alex Huang; Kelven Yang > *Subject:* Re: ifconfig and MacAddress**** > > ** ** > > Hi, > > I noticed other problem with the MacAddress, it appears that it does not > handle the Windows ipconfig output.**** > > (see related patch) https://reviews.apache.org/r/14514/ I tested it with > win8, 7, and XP outputs**** > > As far as I remember this may be a problem when running CS cluster on > windows, since each node determines the default as id.**** > > ** ** > > On Sun, Oct 27, 2013 at 1:11 AM, Darren Shepherd < > [email protected]> wrote:**** > > Is there a specific reason that in MacAddress it uses "ifconfig" or > other utilities to grab the Mac Address as opposed to the java API > java.net.NetworkInterface? There's a comment in that code that says > that code was copied from some public domain utility. So I'm guessing > its there just because that's what somebody wrote back before Java 6 > introduced NetworkInteface.getHardwareAddress(). I'd like to rewrite > this code to use the standard API, grab the Mac deterministically and > also not use ifconfig which was deprecated 3 years ago. > > Darren**** > > > > > -- > > EOF **** > -- EOF
