10000 Updated pylibpd recipe. by n1zzo · Pull Request #433 · kivy/python-for-android · GitHub
[go: up one dir, main page]

Skip to content

Updated pylibpd recipe. #433

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

Closed
wants to merge 4 commits into from
Closed

Updated pylibpd recipe. #433

wants to merge 4 commits into from

Conversation

n1zzo
Copy link
@n1zzo n1zzo commented Aug 18, 2015

The old version of the patch refers to wrong line numbers and fails to patch the setup.py file.
I've updated the patch file creating a diff with the github repo version of setup.py.
Now the recipe uses git clone --recursive for downloading libpd so we can include all the submodules in one step.
The libpd makefile has been patched to be coherent with the pylibpd patch.

@n1zzo n1zzo changed the title Updated pylibpd thread fix patch. Updated pylibpd recipe. Sep 21, 2015
@n1zzo n1zzo changed the title Updated pylibpd recipe. [Revamp] Updated pylibpd recipe. Sep 21, 2015
@n1zzo n1zzo changed the title [Revamp] Updated pylibpd recipe. [revamp] Updated pylibpd recipe. Sep 21, 2015
@inclement inclement changed the title [revamp] Updated pylibpd recipe. Updated pylibpd recipe. Sep 21, 2015
@inclement
Copy link
Member

Thanks for the contribution. Just to be clear, the [revamp] tag refers to the new revamp branch which is a somewhat different rewrite of python-for-android, but your changes are for the master branch, so I've removed the tag. We'll include your changes in the revamp as and when the recipes are ported, so don't worry about them being missed.

@n1zzo
Copy link
Author
n1zzo commented Sep 21, 2015

Ok, thanks for the informations, have a nice day!

@inclement
Copy link
Member

I've merged this manually to the old toolchain. Thanks for the contribution!

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.

2 participants
0