Skip to content

Update perforce from 19.1-1845410 to 19.1-1865205 #70981

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.

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

Merged
merged 1 commit into from
Oct 16, 2019

Conversation

dscho
Copy link
Contributor

@dscho dscho commented Oct 16, 2019

After making all changes to the cask:

  • brew cask audit --download {{cask_file}} is error-free.
  • brew cask style --fix {{cask_file}} reports no offenses.
  • The commit message includes the cask’s name and version.
  • The submission is for a stable version or documented exception.

I don't have a macOS machine available to test all these. The problem was detected during a PR build: https://dev.azure.com/gitgitgadget/git/_build/results?buildId=18491&view=logs. I updated the values following https://github.com/Homebrew/homebrew-cask/pull/68754/files as an example, using the SHA-256 value from the failed build and the version number from https://www.perforce.com/downloads/helix-core-p4d.

@BrewTestBot BrewTestBot merged commit 159bd9c into Homebrew:master Oct 16, 2019
@dscho dscho deleted the patch-2 branch October 17, 2019 07:55
dscho added a commit to dscho/git that referenced this pull request Oct 17, 2019
It still fails! *Still!*

Even after upgrading the cask's definition in the PR
Homebrew/homebrew-cask#70981 does it fail.

The explanation is, as Gábor Szeder dug out from the commit history:
the local definitions might get stale and need to be updated.

So let's just update them in case `brew cask install perforce` fails,
and just try again!

This will still fail, of course, when `homebrew-cask` falls behind
Perforce's release schedule. But once it is updated, we can now simply
re-run the failed jobs and they will pick up that update.

As to updating `homebrew-cask`: I started automating this, via
https://dev.azure.com/gitgitgadget/git/_build?definitionId=11&_a=summary
(I plan on finishing it once the next Perforce upgrade comes around.)

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
dscho added a commit to git-for-windows/git that referenced this pull request Oct 17, 2019
It still fails! *Still!*

Even after upgrading the cask's definition in the PR
Homebrew/homebrew-cask#70981 does it fail.

The explanation is, as Gábor Szeder dug out from the commit history:
the local definitions might get stale and need to be updated.

So let's just update them in case `brew cask install perforce` fails,
and just try again!

This will still fail, of course, when `homebrew-cask` falls behind
Perforce's release schedule. But once it is updated, we can now simply
re-run the failed jobs and they will pick up that update.

As to updating `homebrew-cask`: I started automating this, via
https://dev.azure.com/gitgitgadget/git/_build?definitionId=11&_a=summary
(I plan on finishing it once the next Perforce upgrade comes around.)

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
@lock lock bot added the outdated label Jan 8, 2020
@lock lock bot locked as resolved and limited conversation to collaborators Jan 8, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants