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

Add MACOS install instructions #252

Merged
merged 2 commits into from
Sep 24, 2021
Merged

Add MACOS install instructions #252

merged 2 commits into from
Sep 24, 2021

Conversation

ahcorde
Copy link
Contributor

@ahcorde ahcorde commented Sep 24, 2021

Signed-off-by: Alejandro Hernández ahcorde@gmail.com

🦟 Bug fix

Summary

Checklist

  • Signed all commits for DCO
  • Added tests
  • Updated documentation (as needed)
  • Updated migration guide (as needed)
  • codecheck passed (See contributing)
  • All tests passed (See test coverage)
  • While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge

Signed-off-by: Alejandro Hernández <ahcorde@gmail.com>
@ahcorde ahcorde requested a review from scpeters as a code owner September 24, 2021 18:38
@github-actions github-actions bot added Gazebo 1️1️ Dependency of Gazebo classic version 11 🏢 edifice Ignition Edifice 🏯 fortress Ignition Fortress 🏰 citadel Ignition Citadel 🔮 dome Ignition Dome labels Sep 24, 2021
@chapulina chapulina added documentation Improvements or additions to documentation OSX Mac OSX support labels Sep 24, 2021
@chapulina chapulina added the beta Targeting beta release of upcoming collection label Sep 24, 2021
@scpeters
Copy link
Member

there is an easier way; we can do brew install --only-dependencies ignition-math6 and it will pull the dependencies from the homebrew formula

this is what we do in CI: https://github.com/ignition-tooling/release-tools/blob/master/jenkins-scripts/lib/project-default-devel-homebrew-amd64.bash#L72

brew tap osrf/simulation
```

Install Ignition GUI:
Copy link
Member

Choose a reason for hiding this comment

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

GUI -> Math


2. Install dependencies
```
brew install $(sort -u $(find . -iname 'packages.brew') | tr '\n' ' ')
Copy link
Member

Choose a reason for hiding this comment

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

brew install --only-dependencies ignition-math<#>

@codecov
Copy link

codecov bot commented Sep 24, 2021

Codecov Report

Merging #252 (6ff4622) into ign-math6 (f3aeb27) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           ign-math6     #252   +/-   ##
==========================================
  Coverage      99.41%   99.41%           
==========================================
  Files             67       67           
  Lines           6347     6347           
==========================================
  Hits            6310     6310           
  Misses            37       37           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f3aeb27...6ff4622. Read the comment docs.

Signed-off-by: Alejandro Hernández <ahcorde@gmail.com>
@ahcorde ahcorde requested a review from scpeters September 24, 2021 20:20
@ahcorde
Copy link
Contributor Author

ahcorde commented Sep 24, 2021

Thank your for the tips @scpeters !

@ahcorde ahcorde merged commit e2f3870 into ign-math6 Sep 24, 2021
@ahcorde ahcorde deleted the ahcorde/macos/install branch September 24, 2021 22:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
beta Targeting beta release of upcoming collection 🏰 citadel Ignition Citadel documentation Improvements or additions to documentation 🔮 dome Ignition Dome 🏢 edifice Ignition Edifice 🏯 fortress Ignition Fortress Gazebo 1️1️ Dependency of Gazebo classic version 11 OSX Mac OSX support
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants