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

genders 1.22 #1366

Closed
wants to merge 1 commit into from
Closed

genders 1.22 #1366

wants to merge 1 commit into from

Conversation

zioproto
Copy link
Contributor

compile with --with-java-extensions=no because the build is broken.
note that current version 1.20 lead to segmentation fault when
using psdh with genders

compile with --with-java-extensions=no because the build is broken.
note that current version 1.20 lead to segmentation fault when
using psdh with genders
@UniqMartin UniqMartin changed the title genders: upgrade to 1.22 genders 1.22 May 23, 2016
@@ -16,6 +16,8 @@ class Genders < Formula
def install
args = ["--prefix=#{prefix}"]

args << "--with-java-extensions=no"
Copy link
Contributor

Choose a reason for hiding this comment

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

Since this isn't a conditional configure argument (unlike the one just below), you can merge it with the initialization of the args array from above like this:

args = %W[
  --prefix=#{prefix}
  --with-java-extensions=no
]

Copy link
Member

Choose a reason for hiding this comment

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

Changed this in post.

@UniqMartin UniqMartin added the needs response Needs a response from the issue/PR author label May 23, 2016
@DomT4 DomT4 closed this in 0870f99 May 28, 2016
@DomT4
Copy link
Member

DomT4 commented May 28, 2016

Merged in 0870f99. Thank you for your contribution to Homebrew @zioproto; we appreciate it! 😺

@Homebrew Homebrew locked and limited conversation to collaborators May 4, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
needs response Needs a response from the issue/PR author
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants