Skip to content

Comments

Autobuild: Hotfix: Remove caching and pinning of create-dmg#3149

Merged
pljones merged 2 commits intojamulussoftware:mainfrom
ann0see:hotfix/removeBrewCaching
Aug 22, 2023
Merged

Autobuild: Hotfix: Remove caching and pinning of create-dmg#3149
pljones merged 2 commits intojamulussoftware:mainfrom
ann0see:hotfix/removeBrewCaching

Conversation

@ann0see
Copy link
Member

@ann0see ann0see commented Aug 20, 2023

Short description of changes

Currently the brew install of create-dmg fails during the extract step of brew. This happens in deploy_mac.sh. This PR removes caching and pinning logic.

CHANGELOG: SKIP

Context: Fixes an issue?
Fixes macOS build.

Does this change need documentation? What needs to be documented and how?
Yes: Needs an issue raised that this needs fixing. Also link https://stackoverflow.com/questions/63672189/use-github-actions-cache-with-brew-install there.

Status of this Pull Request
To be tested by CI.

What is missing until this pull request can be merged?
Working CI.

Checklist

  • I've verified that this Pull Request follows the general code principles
  • I tested my code and it does what I want
  • My code follows the style guide
  • I waited some time after this Pull Request was opened and all GitHub checks completed without errors.
  • I've filled all the content above

@ann0see ann0see added this to the Release 3.10.0 milestone Aug 20, 2023
@ann0see ann0see added bug Something isn't working dependencies Pull requests that update a dependency file labels Aug 20, 2023
@ann0see ann0see self-assigned this Aug 20, 2023
@ann0see ann0see requested a review from pljones August 20, 2023 20:50
@ann0see
Copy link
Member Author

ann0see commented Aug 20, 2023

One of our dependencies (macOS notarization etc) just disappeared. So everything is failing.

@ann0see ann0see force-pushed the hotfix/removeBrewCaching branch from 8061c49 to 0dc5070 Compare August 21, 2023 20:12
@ann0see
Copy link
Member Author

ann0see commented Aug 21, 2023

Also uses alternative Xcode Staple/notarization actions. They should be verified before they get into production.

Added them to the whitelist in this repo too.

The original author deleted his repository, so using alternatives.
@ann0see ann0see force-pushed the hotfix/removeBrewCaching branch from 0dc5070 to 219bf50 Compare August 21, 2023 20:16
Copy link
Collaborator

@pljones pljones left a comment

Choose a reason for hiding this comment

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

Green build!

@pljones pljones merged commit 56136de into jamulussoftware:main Aug 22, 2023
@ann0see ann0see deleted the hotfix/removeBrewCaching branch August 22, 2023 18:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working dependencies Pull requests that update a dependency file

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants