Skip to content

Conversation

kbrock
Copy link
Member

@kbrock kbrock commented Jan 16, 2016

@Fryguy @carbonin I want to run appliance_console on my mac.

While I made the ip and gateway methods work, it really isn't necessary. I just want to be able to run the various menu options.

/cc @jrafanie @gtanzillo FYI

@kbrock kbrock force-pushed the mac_support branch 2 times, most recently from 29a3dc7 to f075581 Compare January 16, 2016 18:24
@kbrock
Copy link
Member Author

kbrock commented Jan 16, 2016

Looks like ip is supported on a mac with:

brew install iproute2mac

So I guess this is not needed - but not standard.

Would you prefer me to continue on this PR, or we'll just go the iproute2mac roue?

@miq-bot
Copy link
Member

miq-bot commented Jan 16, 2016

Checked commits kbrock/linux_admin@5f9d2b9~...61f243c with ruby 2.2.3, rubocop 0.34.2, and haml-lint 0.13.0
7 files checked, 0 offenses detected
Everything looks good. 👍

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe this can all be:

case(Distros.local)
when Distros.rhel, Distros.fedora
  NetworkInterfaceRH
when Distros.mac
  NetworkInterfaceMac
else
  NetworkInterfaceGeneric
end

I think it's nicer than the array include followed by ==

@miq-bot
Copy link
Member

miq-bot commented Feb 11, 2016

<pr_mergeability_checker />This pull request is not mergeable. Please rebase and repush.

@kbrock kbrock closed this Feb 11, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants