Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updated microkernel to build with current Alpine packages. #35

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

jtilander
Copy link

Fixed various problems with the image to build:

  • Certificates for gems out of date, disabled verification for now
  • ipmitool has fallen out of alpine distro, used https://github.com/urzds/ipmitool-docker to build the pkg/ipmitool.tgz file
  • Source repro for lshw changed from testing to community (presumably due to the same circumstances as for ipmitool)
  • Gems install version changed to 2.3.0

@tjmcs
Copy link
Collaborator

tjmcs commented Jan 3, 2017

I think this looks good, @jtilander, except for inclusion of the tgz file (we generally try to avoid including binaries directly in the repository); is there any way that you can rebase the branch used to submit this pull request so that this file is not included as part of the PR?

@jtilander
Copy link
Author

I guess I could remove the .tgz but the project won't build anymore then. Which is a problem, for anyone new trying to actually get a microkernel going it's a bit of a pain to discover and find the different pieces. The .tgz is rather small, and could probably be replaced with a script that generates it based on checking out the dependent git repository.

That being said, I really have no skin in this game anymore since Hanlon didn't seem to actually work in my experiments and I had a couple of similar experiences to building the microkernel after this with little docs and stuff that simply didn't work at all. I've since moved onto evaluating http://rackhd.readthedocs.io instead.

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.

None yet

2 participants