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

Update README.md #949

Closed
wants to merge 1 commit into from
Closed

Update README.md #949

wants to merge 1 commit into from

Conversation

dsbrown
Copy link
Contributor

@dsbrown dsbrown commented Feb 27, 2024

The instructions for git clone is missing one step: sudo make install. This required me to examine the makefile targets and figure it out for myself. Not so bad for me, but others may be confused about what to do.

The instructions for git clone is missing one step: sudo make install. This required me to examine the makefile targets and figure it out for myself. Not so bad for me, but others may be confused about what to do.
@lj3954
Copy link
Member

lj3954 commented Feb 29, 2024

The makefile is not in this repository. You running sudo make install after cloning this repository will just throw an error. Anyways, one could just run quickemu directly out of whichever directory they cloned it into, either by pointing directly to it or adding it to their PATH.

@philclifford
Copy link
Contributor

The Makefile is in the docs subdirectory. It is provided as a courtesy/convenience not as a recommended installation method.
If it is to be used the instruction should be to install all the documented pre-requisites (plus pandoc if the man pages are to be regenerated) then

cd docs
make
sudo make install

@zen0bit zen0bit mentioned this pull request Apr 11, 2024
@flexiondotorg
Copy link
Member

Closed via #986

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.

4 participants