Skip to content

Comments

docs: improve guest creation documentation#1254

Open
jsturtevant wants to merge 2 commits intohyperlight-dev:mainfrom
jsturtevant:rally/1191-make-it-easier-to-create-guests
Open

docs: improve guest creation documentation#1254
jsturtevant wants to merge 2 commits intohyperlight-dev:mainfrom
jsturtevant:rally/1191-make-it-easier-to-create-guests

Conversation

@jsturtevant
Copy link
Contributor

Add a few more directions to readme and add some trouble shooting.

addresses #1191

Add a few more directions to readme and add some trouble shooting.

Signed-off-by: James Sturtevant <jsturtevant@gmail.com>
#![no_std]
#![no_main]
extern crate alloc;
extern crate hyperlight_guest_bin;
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this really needed? The simpleguest doesn't do this, it does it for hyperlight_guest though 🤔

Copy link
Contributor

Choose a reason for hiding this comment

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

I think it's necessary when the crate doesn't use any other hyperlight_guest_bin features

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yea i think that is what we found out here: #1175 (comment)

Copy link
Contributor

@ludfjig ludfjig left a comment

Choose a reason for hiding this comment

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

LGTM. What do you think about adding a minimal (weekly?) test that tries to compile a minimal guest like the one I posted in #1175 using cargo-hyperlight? Maybe we can also link to it from the docs to show what a minimal guest looks like. I think it could be useful, and by making it weekly, we don't slow down existing stuff like just guests

@jsturtevant
Copy link
Contributor Author

LGTM. What do you think about adding a minimal (weekly?) test that tries to compile a minimal guest like the one I posted in #1175 using cargo-hyperlight?

Yea, maybe using the code from the readme would be a good thing since it isn't currently tested as is

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants