-
Notifications
You must be signed in to change notification settings - Fork 523
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
Conversation
Requires local copy of garden script.
…_requirements Conflicts: buildozer/__init__.py
return | ||
|
||
self._ensure_virtualenv() | ||
self.cmd('pip-2.7 install -e git+git@github.com:Ian-Foote/kivy_garden.git#egg=garden', |
There was a problem hiding this comment.
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 :))
There was a problem hiding this comment.
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.
Nice work, as always :) |
Tweak garden_requirements.
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', |
There was a problem hiding this comment.
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.
I'd find this useful. |
@Ian-Foote thanks. |
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.