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

net-p2p/litecoin*: Fixing dependency bug and version bumps #2143

Closed
wants to merge 4 commits into from
Closed

net-p2p/litecoin*: Fixing dependency bug and version bumps #2143

wants to merge 4 commits into from

Conversation

marcpopp
Copy link
Contributor

This pull request is fixing 3 bugs.
I didn't file one each, I don't know, what your preference is.

https://bugs.gentoo.org/show_bug.cgi?id=592018
https://bugs.gentoo.org/show_bug.cgi?id=592100
https://bugs.gentoo.org/show_bug.cgi?id=592112

Changes to existing ebuild and new ebuilds tested locally with qt4 and qt5.

@monsieurp
Copy link
Member

Thanks a lot. I'm going to have to close this PR due to your first commit. Do not pull from master after creating your branch please, it messes up the branch altogether. File a similar PR without pulling from master this time round. Cheers.

@monsieurp monsieurp closed this Aug 25, 2016
@monsieurp
Copy link
Member

monsieurp commented Aug 25, 2016

Also, fixing three bugs in one single PR is fine and encouraged. 👍

@blueness
Copy link
Contributor

each fix should be one commit, and multiple related commits in one pr is fine

@marcpopp
Copy link
Contributor Author

No Problem! Thank you for the info. I opened a new Request: #2144

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