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

Support --enable-mutlithread for MinGW build #54

Merged
merged 1 commit into from
Oct 2, 2016

Conversation

kou
Copy link
Contributor

@kou kou commented Aug 14, 2015

We don't need to check pthread availability for MinGW build because
Windows provides thread API by default.

Note that we need to regenerate configure.

We don't need to check pthread availability for MinGW build because
Windows provides thread API by default.

Note that we need to regenerate configure.
@k-takata k-takata merged commit 32b1da3 into k-takata:master Oct 2, 2016
@k-takata
Copy link
Owner

k-takata commented Oct 2, 2016

Thank you and sorry for the very late response. Merged.

@kou kou deleted the enable-multithread-windows branch October 3, 2016 00:59
@kou
Copy link
Contributor Author

kou commented Oct 3, 2016

Thanks. Could you update configure to apply this change? Or should I create a pull request that updates configure?

@k-takata
Copy link
Owner

k-takata commented Oct 4, 2016

I will update it later.

@kou
Copy link
Contributor Author

kou commented Oct 5, 2016

Thanks.

k-takata added a commit that referenced this pull request Oct 7, 2016
Update by using autoreconf-2.69.

Related: #54
@k-takata
Copy link
Owner

k-takata commented Oct 7, 2016

Updated.

kou added a commit to groonga/groonga that referenced this pull request Oct 8, 2016
It includes MinGW + thread fix by k-takata/Onigmo#54.
@kou
Copy link
Contributor Author

kou commented Oct 8, 2016

Thanks!

@coveralls
Copy link

Coverage Status

Coverage increased (+0.9%) to 79.775% when pulling aa828e4 on kou:enable-multithread-windows into 05da593 on k-takata:master.

@k-takata
Copy link
Owner

Hmm, strange. Why coveralls report this on a two-years-old PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants