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

README: Mention about git submodule #144

Merged
merged 1 commit into from
Jan 8, 2025
Merged

Conversation

ZhanYF
Copy link
Contributor

@ZhanYF ZhanYF commented Jan 7, 2025

Before libvmm can be built, the git submodule for sddf must be initialized first, this PR mentions about it in the README.

Most people using this repository probably know enough about git submodule to do it when make fail in the first run but I think it's still a good idea to document it.

README.md Outdated Show resolved Hide resolved
@Ivan-Velickovic
Copy link
Collaborator

@ZhanYF you'll need to add a sign-off commit for the DCO check.

You can do it with:

git commit --amend -s
git push -f

assuming you have an email setup with your git config.

@ZhanYF
Copy link
Contributor Author

ZhanYF commented Jan 7, 2025

Done, thanks for the hints.

Comment on lines +74 to +75
cd libvmm
git submodule update --init
Copy link
Collaborator

Choose a reason for hiding this comment

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

I messed up the suggestion sorry, there should still be three back ticks at the end and then a new line.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Got it, should be good now.

Co-authored-by: Ivan Velickovic <[email protected]>
Signed-off-by: Yifei Zhan <[email protected]>
@Ivan-Velickovic Ivan-Velickovic merged commit 79c2df1 into au-ts:main Jan 8, 2025
5 of 8 checks passed
@Ivan-Velickovic
Copy link
Collaborator

Cool, thanks!

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.

2 participants