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

Fix cmake example #105

Merged
merged 2 commits into from
Mar 30, 2015
Merged

Conversation

mikepurvis
Copy link
Contributor

This PR is my suggested replacement for #102.

I have retained the -rdynamic removal, but avoided adding some of the other variables— I feel these are unnecessary and add unwanted indirection.

The build works correctly with the version number request simply left blank, so I have switched the example to use that approach.

@punkkeks Please review.

- Remove specific version requirement for CMake example.
- Make default build against soft float library.
@hellow554
Copy link
Contributor

For the compiler output see https://gist.github.com/punkkeks/7ead6f92f72a9613dcd1
I like your pull request, ship it ;)

andysworkshop added a commit that referenced this pull request Mar 30, 2015
@andysworkshop andysworkshop merged commit d8d9a57 into andysworkshop:master Mar 30, 2015
@andysworkshop
Copy link
Owner

👍

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