Skip to content

Use '~/.python-for-android' as storage_dir#624

Closed
cbenhagen wants to merge 1 commit into
kivy:masterfrom
cbenhagen:patch-4
Closed

Use '~/.python-for-android' as storage_dir#624
cbenhagen wants to merge 1 commit into
kivy:masterfrom
cbenhagen:patch-4

Conversation

@cbenhagen

Copy link
Copy Markdown
Contributor

fixes #622

@cbenhagen cbenhagen changed the title Use '~/.python-for-android' as storage_dir (fixes #622) Use '~/.python-for-android' as storage_dir Feb 3, 2016
@inclement

Copy link
Copy Markdown
Member

I think it's best to use the platform standards where possible (e.g. ~/.local on linux, following the xdg spec), but it would be fine to make an exception for OS X or (if this doesn't affect everyone - I did think people had used OS X successfully) to check if the path will work before choosing a dir.

@cbenhagen

Copy link
Copy Markdown
Contributor Author

Just as a reminder: We need to update this too:
https://github.com/kivy/buildozer/blob/master/buildozer/targets/android_new.py#L43-L44

@kived

kived commented Feb 3, 2016

Copy link
Copy Markdown
Contributor

It probably just shouldn't be hard-coded in buildozer at all. buildozer should ask p4a where the dist dir is.

@cbenhagen

Copy link
Copy Markdown
Contributor Author

👍

@kived

kived commented Apr 27, 2016

Copy link
Copy Markdown
Contributor

#622 was closed by #720, which adds a --storage-dir option. Buildozer implemented this option in kivy/buildozer#323.

@kived kived closed this Apr 27, 2016
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.

Building python2 for armeabi fails due to space character in storage_dir (OS X)

3 participants