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 installation docs skip-ace Mac #1394

Merged
merged 4 commits into from
Apr 24, 2023
Merged

Fix installation docs skip-ace Mac #1394

merged 4 commits into from
Apr 24, 2023

Conversation

DanielePucci
Copy link
Contributor

Because of #916, one should use the option SKIP-ACE in the Yarp CMAKE. Also, one should not have ace in the mamba dependencies. This PR fixes the installation documentation in those respects

@traversaro
Copy link
Member

traversaro commented Apr 23, 2023

Thanks @DanielePucci, just a clarification: The modification you mention are necessary if you are using a macOS with recent ARM-based processor, while they are not necessary if you using macOS with an Intel-based processor. If you like I can do the necessary modifications for the PR.

@DanielePucci
Copy link
Contributor Author

Good point @traversaro, I tried to correct it with b0e5a8e and 748005c

Take a look at them, in the case do not hesitate to integrate them

Copy link
Member

@traversaro traversaro left a comment

Choose a reason for hiding this comment

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

Thanks!

@traversaro traversaro merged commit 11c51b2 into master Apr 24, 2023
@traversaro traversaro deleted the fix-docs-skip-ace-mac branch April 24, 2023 08:51
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