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.

Sign up for GitHub

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

Jump to bottom

brew test-bot errors: conflict declarations #1518

Closed
chapulina opened this issue Jun 21, 2021 · 2 comments · Fixed by #1519
Closed

brew test-bot errors: conflict declarations #1518

chapulina opened this issue Jun 21, 2021 · 2 comments · Fixed by #1519

Comments

@chapulina
Copy link
Contributor

The latest PRs are failing with the following output. I'm not sure if something changed on brew that's requiring these conflicts to be declared now, or if we did something that caused a regression.

==> brew audit --tap=osrf/simulation --except=version
==> FAILED
osrf/simulation/gazebo10:
  * Formula gazebo2 should also have a conflict declared with gazebo10
  * Formula gazebo3 should also have a conflict declared with gazebo10
  * Formula gazebo4 should also have a conflict declared with gazebo10
  * Formula gazebo5 should also have a conflict declared with gazebo10
  * Formula gazebo6 should also have a conflict declared with gazebo10
  * Formula gazebo8 should also have a conflict declared with gazebo10
osrf/simulation/gazebo11:
  * Formula gazebo2 should also have a conflict declared with gazebo11
  * Formula gazebo3 should also have a conflict declared with gazebo11
  * Formula gazebo4 should also have a conflict declared with gazebo11
  * Formula gazebo5 should also have a conflict declared with gazebo11
  * Formula gazebo6 should also have a conflict declared with gazebo11
  * Formula gazebo8 should also have a conflict declared with gazebo11
osrf/simulation/gazebo9:
  * Formula gazebo2 should also have a conflict declared with gazebo9
  * Formula gazebo3 should also have a conflict declared with gazebo9
  * Formula gazebo4 should also have a conflict declared with gazebo9
  * Formula gazebo5 should also have a conflict declared with gazebo9
  * Formula gazebo6 should also have a conflict declared with gazebo9
  * Formula gazebo8 should also have a conflict declared with gazebo9
osrf/simulation/ignition-math4:
  * Formula ignition-math2 should also have a conflict declared with ignition-math4
osrf/simulation/ignition-math5:
  * Formula ignition-math2 should also have a conflict declared with ignition-math5
osrf/simulation/ignition-math6:
  * Formula ignition-math2 should also have a conflict declared with ignition-math6
osrf/simulation/sdformat6:
  * Formula sdformat should also have a conflict declared with sdformat6
  * Formula sdformat3 should also have a conflict declared with sdformat6
osrf/simulation/sdformat7:
  * Formula sdformat should also have a conflict declared with sdformat7
  * Formula sdformat3 should also have a conflict declared with sdformat7
Error: 25 problems in 8 formulae detected
Error: Formula gazebo2 should also have a conflict declared with gazebo10
Error: Formula gazebo3 should also have a conflict declared with gazebo10
Error: Formula gazebo4 should also have a conflict declared with gazebo10
Error: Formula gazebo5 should also have a conflict declared with gazebo10
Error: Formula gazebo6 should also have a conflict declared with gazebo10
Error: Formula gazebo8 should also have a conflict declared with gazebo10
Error: Formula gazebo2 should also have a conflict declared with gazebo11
Error: Formula gazebo3 should also have a conflict declared with gazebo11
Error: Formula gazebo4 should also have a conflict declared with gazebo11
Error: Formula gazebo5 should also have a conflict declared with gazebo11
Error: Formula gazebo6 should also have a conflict declared with gazebo11
Error: Formula gazebo8 should also have a conflict declared with gazebo11
Error: Formula gazebo2 should also have a conflict declared with gazebo9
Error: Formula gazebo3 should also have a conflict declared with gazebo9
Error: Formula gazebo4 should also have a conflict declared with gazebo9
Error: Formula gazebo5 should also have a conflict declared with gazebo9
Error: Formula gazebo6 should also have a conflict declared with gazebo9
Error: Formula gazebo8 should also have a conflict declared with gazebo9
Error: Formula ignition-math2 should also have a conflict declared with ignition-math4
Error: Formula ignition-math2 should also have a conflict declared with ignition-math5
Error: Formula ignition-math2 should also have a conflict declared with ignition-math6
Error: Formula sdformat should also have a conflict declared with sdformat6
Error: Formula sdformat3 should also have a conflict declared with sdformat6
Error: Formula sdformat should also have a conflict declared with sdformat7
Error: Formula sdformat3 should also have a conflict declared with sdformat7

Error: Process completed with exit code 1.
@chapulina chapulina mentioned this issue Jun 21, 2021
Merged
@scpeters
Copy link
Member

it was an upstream brew change: Homebrew/brew#11541

I think we can just delete the conflicts between gazebo9/11 and the outdated versions. I'll make a PR to that effect

scpeters added a commit to scpeters/homebrew-simulation that referenced this issue Jun 21, 2021
@scpeters
Remove asymmetric conflicts_with statements
7215767 
A recent change to brew audit disallows asymmetric
conflicts_with statements. Since these all involve
old formulae, this removes the conflicts_with
statements rather than trying to make them symmetric.

Fixes osrf#1518.
@scpeters scpeters mentioned this issue Jun 21, 2021
Merged
@scpeters
Copy link
Member

fix in #1519

scpeters added a commit that referenced this issue Jun 21, 2021
@scpeters
Remove asymmetric conflicts_with statements (#1519)
5840fa5 
A recent change to brew audit disallows asymmetric
conflicts_with statements. Since these all involve
old formulae, this removes the conflicts_with
statements rather than trying to make them symmetric.

Fixes #1518.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove asymmetric conflicts_with statements scpeters/homebrew-simulation
2 participants
@scpeters @chapulina