gwsystems / composite

A component-based OS
composite.seas.gwu.edu
185 stars 70 forks source link

Rumpkernel #281

Closed RobertGiff closed 7 years ago

RobertGiff commented 7 years ago

Summary of this Pull Request (PR)

Update Composite installation guide with necessary Linux packages.

Intent for your PR

Choose one (Mandatory):

Reviewers (Mandatory):

@gparmer @maloneya @Others

Code Quality

As part of this pull request, I've considered the following:

Style:

Code Craftsmanship:

Testing

N/A

Others commented 7 years ago

@RobertGiff for the Rust instructions, you can remove the temporary file and just pipe it to sh (you only need the temporary script if you aren't willing to answer shell prompts)

gparmer commented 7 years ago

FYI, some information about the PR template:

The "- [ x]" should be "- [x]" https://github.com/blog/1825-task-lists-in-all-markdown-documents

Under testing, you normally include which of the tests (e.g. micro_booter) you ran. That doesn't apply here as you're adding documentation, so "N/A" is fine.

phanikishoreg commented 7 years ago

@robertgiff we also need grub and iso utilities to make a composite bootable iso image, right?

RobertGiff commented 7 years ago

@phanikishoreg making a composite bootable iso I believe just requires package grub-common to run command grub-mkrescue. I'll post more if I find more to be required.

phanikishoreg commented 7 years ago

grub-mkrescue depends on xorriso and that is not part of grub-common.

RobertGiff commented 7 years ago

Interesting, when I run dpkg -S grub-mkrescue to get the command's package I get grub-common. I assume xorriso also installs it?

phanikishoreg commented 7 years ago

Perhaps xoriso maintainers are different, could be why grub-common doesn't include it. All I know is grub-mkrescue requires xorriso and we need to install them separately. ;-)

gparmer commented 7 years ago

Thanks @RobertGiff !