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 #1171

Closed
wants to merge 1 commit into from
Closed

Conversation

@JohnnyWalkerDesign
Copy link

JohnnyWalkerDesign commented Apr 27, 2018

Added necessary switches otherwise you get a curl not found error

Added necessary switches otherwise you get a curl not found error
@criezy
Copy link
Member

criezy commented Apr 27, 2018

The change is correct.

We do however have more up to date instructions on our wiki at http://wiki.scummvm.org/index.php/Compiling_ScummVM/iPhone with a link to a more recent libraries package that allows to not disable jpeg, and for example play Myst (which was actually the main reason I updated the package and wrote those instructions on the wiki 5 months ago). I never noticed we have that README.md file as well.

We could either update the content of the file to be in sync with those newer instructions, but it might fall out of sync again. So maybe we should replace them with a link to the wiki.

@JohnnyWalkerDesign
Copy link
Author

JohnnyWalkerDesign commented Apr 27, 2018

Ah. Yes, I've been struggling with those instructions for an hour or so. If there's better ones elsewhere, best to just link to them I think!

@JohnnyWalkerDesign
Copy link
Author

JohnnyWalkerDesign commented Apr 27, 2018

FYI: With those other instructions I had to run sudo xcodebuild for it to work, otherwise I got errors.

@criezy
Copy link
Member

criezy commented Apr 29, 2018

I have now updated the instructions on the wiki with additional information and changed the README.md file to point to those.

Thank you for pointing out the issues in the build instructions we had in the README.md file and wiki.

@criezy criezy closed this Apr 29, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.