Skip to content

Add MACOS install instructions#252

Merged
ahcorde merged 2 commits intoign-math6from
ahcorde/macos/install
Sep 24, 2021
Merged

Add MACOS install instructions#252
ahcorde merged 2 commits intoign-math6from
ahcorde/macos/install

Conversation

@ahcorde
Copy link
Copy Markdown
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 beta Targeting beta release of upcoming collection labels Sep 24, 2021
@scpeters
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown

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
Copy Markdown
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.

5 participants