Skip to content

Refresh installed list before dependency check#8998

Merged
MikeMcQuaid merged 1 commit intoHomebrew:masterfrom
muneebmahmed:dependent-check
Oct 27, 2020
Merged

Refresh installed list before dependency check#8998
MikeMcQuaid merged 1 commit intoHomebrew:masterfrom
muneebmahmed:dependent-check

Conversation

@muneebmahmed
Copy link
Copy Markdown
Contributor

@muneebmahmed muneebmahmed commented Oct 27, 2020

  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same change?
  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Have you written new tests for your changes? Here's an example.
  • Have you successfully run brew style with your changes locally?
  • Have you successfully run brew tests with your changes locally?
  • Have you successfully run brew man locally and committed any changes?

Possibly fixes #8997. I call FormulaInstaller.installed.to_a after upgrading dependents, in order to refresh the list of formulae before checking for broken dependents. With this change, I ran brew upgrade x264 and both x264 and its dependent ffmpeg were upgraded without errors. When I checked out master, brew upgrade ruby ran into errors when upgrading its dependent cocoapods.

I did run into one error when running brew tests:

  1) Utils::Git#cherry_pick! aborts when cherry picking an existing hash
     Failure/Error:
       expect {
         described_class.cherry_pick!(HOMEBREW_CACHE, file_hash1)
       }.to raise_error(ErrorDuringExecution, /Merge conflict in README.md/)

       expected ErrorDuringExecution with message matching /Merge conflict in README.md/, got #<ErrorDuringExecution: Failure while executing; `/usr/local/Homebrew/Library/Homebrew/shims/scm/git -C /tmp/homebrew-tests-20201026-44218-1ytlg8a/cache cherry-pick 36c62d2` exited with 0. Here's the output:

However, I got this exact same error when running brew tests on master


Possibly fixes Homebrew/homebrew-core#63497 as well

@MikeMcQuaid MikeMcQuaid merged commit 8f5988f into Homebrew:master Oct 27, 2020
@MikeMcQuaid
Copy link
Copy Markdown
Member

Thanks so much for your first contribution! Without people like you submitting PRs we couldn't run this project. You rock, @muneebmahmed!

@BrewTestBot BrewTestBot added the outdated PR was locked due to age label Dec 1, 2020
@Homebrew Homebrew locked as resolved and limited conversation to collaborators Dec 1, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

outdated PR was locked due to age

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Error when upgrading dependency of upgradable formula brew install imagemagick fails at the end

3 participants