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

groonga 13.0.1 #126947

Merged
merged 7 commits into from
Mar 29, 2023
Merged

groonga 13.0.1 #126947

merged 7 commits into from
Mar 29, 2023

Conversation

ZhongRuoyu
Copy link
Member

@ZhongRuoyu ZhongRuoyu commented Mar 29, 2023

Created by brew bump


Created with brew bump-formula-pr.

resource blocks may require updates.

@BrewTestBot BrewTestBot added the bump-formula-pr PR was created using `brew bump-formula-pr` label Mar 29, 2023
# Temporary workaround for `msgpack` rename. Remove when resolved:
# https://github.com/groonga/groonga/issues/1536
args << "--with-message-pack=#{Formula["msgpack"].opt_prefix}"

if build.head?
args << "--with-ruby"
Copy link
Member

Choose a reason for hiding this comment

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

Any idea why we're only doing this for head?

Copy link
Member Author

Choose a reason for hiding this comment

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

Not really sure. This was added in Homebrew/legacy-homebrew#45792. I don't have enough context to reason. But I think it probably should be applied to stable versions too: https://groonga.org/docs/contribution/development/build/unix_autotools.html

@BrewTestBot BrewTestBot added the ruby Ruby use is a significant feature of the PR or issue label Mar 29, 2023
@ZhongRuoyu ZhongRuoyu force-pushed the bump-groonga-13.0.1 branch from 6038e96 to 07437de Compare March 29, 2023 09:43
@SMillerDev
Copy link
Member

  Making install in mruby
  touch "./mruby-file-stat/src/config.h"
  touch: ./mruby-file-stat/src/config.h: No such file or directory

Looks better to drop it

@BrewTestBot BrewTestBot removed the ruby Ruby use is a significant feature of the PR or issue label Mar 29, 2023
@ZhongRuoyu
Copy link
Member Author

Same error. Might be something in std_configure_args.

@github-actions
Copy link
Contributor

🤖 A scheduled task has requested bottles to be published to this PR.

@github-actions github-actions bot added the CI-published-bottle-commits The commits for the built bottles have been pushed to the PR branch. label Mar 29, 2023
@BrewTestBot BrewTestBot enabled auto-merge March 29, 2023 18:13
@BrewTestBot BrewTestBot merged commit 9824f0a into Homebrew:master Mar 29, 2023
@ZhongRuoyu ZhongRuoyu deleted the bump-groonga-13.0.1 branch March 29, 2023 18:31
abetomo added a commit to abetomo/nroonga that referenced this pull request Apr 20, 2023
When installing Groonga on Mac with Homebrew, Ruby plugin is no longer included.
Homebrew/homebrew-core#126947

Signed-off-by: abetomo <[email protected]>
kou pushed a commit to nroonga/nroonga that referenced this pull request Apr 20, 2023
When installing Groonga on Mac with Homebrew, Ruby plugin is no longer
included.
Homebrew/homebrew-core#126947

Signed-off-by: abetomo <[email protected]>
@github-actions github-actions bot added the outdated PR was locked due to age label Apr 29, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 29, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bump-formula-pr PR was created using `brew bump-formula-pr` CI-published-bottle-commits The commits for the built bottles have been pushed to the PR branch. outdated PR was locked due to age
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants