Skip to content
This repository has been archived by the owner on Jul 4, 2023. It is now read-only.

groonga: add HEAD support #45792

Closed
wants to merge 2 commits into from

Conversation

cosmo0920
Copy link
Contributor

Some of Groonga users want to try new features as soon as possible.
It's reasonable to support build with --HEAD option.
How do you think about it?

args << "--enable-benchmark" if build.with? "benchmark"
args << "--with-mecab" if build.with? "mecab"
args << "--with-lz4" if build.with? "lz4"

system "./autogen.sh" if build.head?
Copy link
Member

Choose a reason for hiding this comment

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

You could put this in one block :)

if build.head?
 args << "--with-ruby"
 system "./autogen.sh"
end

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks, I've fixed it!

@DomT4
Copy link
Member

DomT4 commented Nov 8, 2015

Merged in 6e43e9a. Thank you once more @cosmo0920! 😺

@DomT4 DomT4 closed this in 6e43e9a Nov 8, 2015
@cosmo0920
Copy link
Contributor Author

Thank you for your advice and quick merging! 😋

@cosmo0920 cosmo0920 deleted the support-head-groonga branch November 8, 2015 23:03
@Homebrew Homebrew locked and limited conversation to collaborators Jul 10, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants