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

Garden requirements #41

Merged
merged 11 commits into from Mar 30, 2014
Merged

Conversation

LilyFoote
Copy link
Contributor

I'm opening this pull request for some code review.

This adds support for kivy garden modules to buildozer, avoiding the need to clutter the App's source directory.

There is some cleanup still possible - e.g. simplify the pip install of the kivy_garden scripts - but the code is functional and easy to use at this point. I have only tested on linux.

Fixes #39.

return

self._ensure_virtualenv()
self.cmd('pip-2.7 install -e git+git@github.com:Ian-Foote/kivy_garden.git#egg=garden',
Copy link
Member

Choose a reason for hiding this comment

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

Would it be possible to link over the kivy_garden of the garden community instead ? (i don't know if it exist, but if not, it should :))

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I split https://github.com/Ian-Foote/kivy_garden out of the main kivy repo - I was hoping my fork would get picked up officially as the one source of the garden install scripts.

@tito
Copy link
Member

tito commented Sep 26, 2013

Nice work, as always :)

@ghost ghost assigned tito Sep 26, 2013
return

self._ensure_virtualenv()
self.cmd('pip-2.7 install -e git+https://github.com/kivy-garden/garden.git@0.1.1#egg=Kivy-Garden-0.1.1',
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I've updated this to match the work done to fix kivy/kivy#1580.

@wjanssen
Copy link

I'd find this useful.

akshayaurora added a commit that referenced this pull request Mar 30, 2014
@akshayaurora akshayaurora merged commit 744c03b into kivy:master Mar 30, 2014
@akshayaurora
Copy link
Member

@Ian-Foote thanks.

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.

Garden packages are unsupported
4 participants