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

Fixes #405

Merged
merged 3 commits into from Jun 28, 2016
Merged

Fixes #405

merged 3 commits into from Jun 28, 2016

Conversation

eworm-de
Copy link
Contributor

No description provided.

@ayurchen
Copy link
Member

Philip, I don't have a problem with this patch as long as it builds on all systems we need it to build.

@philip-galera
Copy link
Contributor

@eworm-de Thank you for your contribution. Will you be willing to accept our Contributor License Agreement so that we can merge the contribution? If yes, let me know and I will provide the details.

@eworm-de
Copy link
Contributor Author

Sure. I am following the steps at https://www.clahub.com/agreements/codership/galera, but it returns:

We're sorry, but something went wrong.

@philip-galera
Copy link
Contributor

@eworm-de: Yes, this service seems to be unreliable at times. Can you please add your name to the CONTRIBUTORS.txt file , and then use commit -s to commit the changes to this file. Push the new commit so that it becomes part of the pull request. Thank you!

Signed-off-by: Christian Hesse <mail@eworm.de>
@eworm-de
Copy link
Contributor Author

Done.
Do you want the other commits with a sign-off-line as well?

@philip-galera
Copy link
Contributor

I think we are good with respect to the CLA. Your change compiles on all supported platforms so I am now merging it. Thank you.

@philip-galera philip-galera merged commit 8430674 into codership:3.x Jun 28, 2016
@eworm-de eworm-de deleted the fixes branch May 6, 2019 18:51
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.

None yet

3 participants