gazebo-tooling / release-tools

8 stars 9 forks source link

Add ci_matching_branch/ for bottle install jobs #1005

Closed scpeters closed 11 months ago

scpeters commented 1 year ago

We have started documenting how to use the ci_matching_branch/ naming pattern to use custom branches from tooling repositories in pull request CI in https://github.com/gazebosim/docs/pull/377. This PR will add the ci_matching_branch/ functionality to the install_bottle jobs in the following way:

As an example for when this could be used, I'd like to re-enable a test for the gz-sim7 formula that has been disabled for macOS Monterey since a fix in https://github.com/gazebosim/gz-physics/pull/529 has been merged and released. The test is enabled in https://github.com/osrf/homebrew-simulation/commit/2bf74acf05b9388fe12ae5942f9dc7866ebd1e1c in the ci_matching_branch/install_bottle branch (which matches the branch name of this release-tools pull request) and tested in the following job, which happened to run on mac-five.monterey and passed the enabled test:

# BEGIN SECTION: run tests
+ brew linkage --test gz-sim7
+ brew ruby -e 'exit '\''gz-sim7'\''.f.test_defined?'
+ brew test gz-sim7
==> Testing osrf/simulation/gz-sim7
==> /usr/local/opt/ruby/bin/ruby /usr/local/opt/gz-tools2/bin/gz sim -s --iterat
==> cmake ..
==> make
==> ./test_cmake
==> ! grep -rnI 'Applications[/]Xcode' /usr/local/Cellar/gz-sim7/7.5.0_8
+ brew audit --strict gz-sim7
+ echo '# END SECTION'
# END SECTION
scpeters commented 11 months ago

I've applied your linting suggestions