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.

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

mysql-search-replace: needs php for runtime and license update #179684

Closed
wants to merge 1 commit into from

Conversation

chenrui333
Copy link
Member

  • Have you followed the guidelines for contributing?
  • Have you ensured that your commits follow the commit style guide?
  • Have you checked that there aren't other open pull requests for the same formula update/change?
  • Have you built your formula locally with HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>, where <formula> is the name of the formula you're submitting?
  • Is your test running fine brew test <formula>, where <formula> is the name of the formula you're submitting?
  • Does your build pass brew audit --strict <formula> (after doing HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>)? If this is a new formula, does it pass brew audit --new <formula>?

seeing test failure in https://github.com/Homebrew/homebrew-core/actions/runs/10228064152/job/28300209098

need to do the rebottle due to

==> Verifying attestation for mysql-search-replace
Error: The bottle for mysql-search-replace has an invalid build provenance attestation.

This may indicate that the bottle was not produced by the expected
tap, or was maliciously inserted into the expected tap's bottle
storage.

Additional context:

no attestation matches subject: 3afd88aefdb93f6e76febdddda041122da71f11ebba26fd43be90776502cd438--mysql-search-replace--4.1.2.all.bottle.tar.gz

also update the license and license ref is in here, https://github.com/interconnectit/Search-Replace-DB/blob/24583f38e49d8e053043704f84ffc0f8a4307686/index.php#L37-L51

@chenrui333 chenrui333 added the license License PRs label Aug 3, 2024
@github-actions github-actions bot added the php PHP use is a significant feature of the PR or issue label Aug 3, 2024
@github-actions github-actions bot closed this Aug 3, 2024
@chenrui333 chenrui333 reopened this Aug 3, 2024
@Homebrew Homebrew deleted a comment from github-actions bot Aug 3, 2024
@chenrui333
Copy link
Member Author

  ==> /home/linuxbrew/.linuxbrew/Cellar/mysql-search-replace/4.1.2/bin/srdb --help
  /usr/bin/env: ‘php -q’: No such file or directory
  /usr/bin/env: use -[v]S to pass options in shebang lines

@chenrui333 chenrui333 added test failure CI fails while running the test-do block CI-no-fail-fast Continue CI tests despite failing GitHub Actions matrix builds. labels Aug 3, 2024
@chenrui333 chenrui333 force-pushed the mysql-search-replace-rebottle branch from 15640e5 to 10a73d5 Compare August 5, 2024 17:06
Signed-off-by: Rui Chen <[email protected]>

mysql-search-replace: add php 8.3+ build patch

Signed-off-by: Rui Chen <[email protected]>
@chenrui333 chenrui333 force-pushed the mysql-search-replace-rebottle branch from 10a73d5 to 926328a Compare August 5, 2024 17:12
@chenrui333 chenrui333 removed the test failure CI fails while running the test-do block label Aug 5, 2024
@chenrui333 chenrui333 added the in progress Stale bot should stay away label Aug 16, 2024
@chenrui333
Copy link
Member Author

will retry later.

@chenrui333 chenrui333 closed this Sep 15, 2024
@github-actions github-actions bot deleted the mysql-search-replace-rebottle branch September 15, 2024 02:55
@chenrui333 chenrui333 added the help wanted Task(s) needing PRs from the community or maintainers label Sep 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI-no-fail-fast Continue CI tests despite failing GitHub Actions matrix builds. help wanted Task(s) needing PRs from the community or maintainers in progress Stale bot should stay away license License PRs php PHP use is a significant feature of the PR or issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant