Skip to content

Commit e885739

Browse files
authored
Merge branch 'master' into doc_and_cosmetics
2 parents df428cb + 4f57863 commit e885739

File tree

47 files changed

+372
-263
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

47 files changed

+372
-263
lines changed

.github/ISSUE_TEMPLATE.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
<!--
22
The issue tracker is a tool to address bugs.
3-
Please use the #kivy IRC channel on freenode or Stack Overflow for
4-
support questions, more information at https://github.com/kivy/python-for-android#support
3+
Please use the Disord community or Stack Overflow for support questions,
4+
more information at https://github.com/kivy/python-for-android#support
55
66
Before opening a new issue, make sure you do the following:
77
* check that your issue isn't already filed: https://github.com/kivy/python-for-android/issues

.travis.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,8 @@ services:
77

88
before_install:
99
- sudo apt-get update -qq
10-
- sudo apt-get install -qq python-tox
10+
- sudo apt-get install -qq python2.7 python3
11+
- sudo pip install tox>=2.0
1112

1213
env:
1314
- COMMAND='. venv/bin/activate && cd testapps/ && python setup_testapp_python2.py apk --sdk-dir /opt/android/android-sdk --ndk-dir /opt/android/android-ndk'

Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
# TODO:
1212
# - delete archives to keep small the container small
1313
# - setup caching (for apt, pip, ndk, sdk and p4a recipes downloads)
14-
FROM ubuntu:16.04
14+
FROM ubuntu:18.04
1515

1616

1717
# get the latest version from https://developer.android.com/ndk/downloads/index.html

README.rst

Lines changed: 2 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -75,11 +75,7 @@ If you need assistance, you can ask for help on our mailing list:
7575
- User Group: https://groups.google.com/group/kivy-users
7676
7777

78-
We also have an IRC channel:
79-
80-
- Server: irc.freenode.net
81-
- Port: 6667, 6697 (SSL only)
82-
- Channel: #kivy
78+
We also have `#support Discord channel <https://chat.kivy.org/>`_.
8379

8480
Contributing
8581
============
@@ -94,11 +90,7 @@ discussions about developing the Kivy framework and its sister projects:
9490
- Dev Group: https://groups.google.com/group/kivy-dev
9591
9692

97-
IRC channel:
98-
99-
- Server: irc.freenode.net
100-
- Port: 6667, 6697 (SSL only)
101-
- Channel: #kivy or #kivy-dev
93+
We also have `#dev Discord channel <https://chat.kivy.org/>`_.
10294

10395
License
10496
=======

doc/source/services.rst

Lines changed: 13 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -68,15 +68,23 @@ code), you must use PyJNIus to interact with the java class
6868
python-for-android creates for each one, as follows::
6969

7070
from jnius import autoclass
71-
service = autoclass('your.package.name.ServiceMyservice')
71+
service = autoclass('your.package.domain.package.name.ServiceMyservice')
7272
mActivity = autoclass('org.kivy.android.PythonActivity').mActivity
7373
argument = ''
7474
service.start(mActivity, argument)
7575

76-
Here, ``your.package.name`` refers to the package identifier of your
77-
APK as set by the ``--package`` argument to python-for-android, and
78-
the name of the service is ``ServiceMyservice``, in which ``Myservice``
79-
is the identifier that was previously passed to the ``--service``
76+
Here, ``your.package.domain.package.name`` refers to the package identifier
77+
of your APK.
78+
79+
If you are using buildozer, the identifier is set by the ``package.name``
80+
and ``package.domain`` values in your buildozer.spec file.
81+
The name of the service is ``ServiceMyservice``, where ``Myservice``
82+
is the name specied by one of the ``services`` values, but with the first
83+
letter upper case.
84+
85+
If you are using python-for-android directly, the identifier is set by the ``--package``
86+
argument to python-for-android. The name of the service is ``ServiceMyservice``,
87+
where ``Myservice`` is the identifier that was previously passed to the ``--service``
8088
argument, but with the first letter upper case. You must also pass the
8189
``argument`` parameter even if (as here) it is an empty string. If you
8290
do pass it, the service can make use of this argument.

doc/source/troubleshooting.rst

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ Add the ``--debug`` option to any python-for-android command to see
1010
full debug output including the output of all the external tools used
1111
in the compilation and packaging steps.
1212

13-
If reporting a problem by email or irc, it is usually helpful to
13+
If reporting a problem by email or Discord, it is usually helpful to
1414
include this full log, via e.g. a `pastebin
1515
<http://paste.ubuntu.com/>`_ or `Github gist
1616
<https://gist.github.com/>`_.
@@ -23,7 +23,7 @@ get help with any problems using the same channels as Kivy itself:
2323

2424
- by email to the `kivy-users Google group
2525
<https://groups.google.com/forum/#!forum/kivy-users>`_
26-
- by irc in the #kivy room at irc.freenode.net
26+
- on `#support Discord channel <https://chat.kivy.org/>`_
2727

2828
If you find a bug, you can also post an issue on the
2929
`python-for-android Github page

pythonforandroid/bdistapk.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -136,7 +136,7 @@ def _set_user_options():
136136
for i, arg in enumerate(sys.argv):
137137
if arg.startswith('--'):
138138
if ('=' in arg or
139-
(i < (len(sys.argv) - 1) and not sys.argv[i+1].startswith('-'))):
139+
(i < (len(sys.argv) - 1) and not sys.argv[i+1].startswith('-'))):
140140
user_options.append((arg[2:].split('=')[0] + '=', None, None))
141141
else:
142142
user_options.append((arg[2:], None, None))

pythonforandroid/bootstraps/pygame/__init__.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ def run_distribute(self):
5454
if not exists('python-install'):
5555
shprint(sh.cp, '-a', self.ctx.get_python_install_dir(), './python-install')
5656

57-
self.distribute_libs(arch, [join(self.build_dir, 'libs', arch.arch), self.ctx.get_libs_dir(arch.arch)]);
57+
self.distribute_libs(arch, [join(self.build_dir, 'libs', arch.arch), self.ctx.get_libs_dir(arch.arch)])
5858
self.distribute_aars(arch)
5959
self.distribute_javaclasses(self.ctx.javaclass_dir)
6060

pythonforandroid/bootstraps/pygame/build/build.py

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -133,10 +133,10 @@ def select(fn):
133133
fn = realpath(fn)
134134
assert(fn.startswith(d))
135135
fn = fn[len(d):]
136-
if (fn.startswith('/site-packages/') or
137-
fn.startswith('/config/') or
138-
fn.startswith('/lib-dynload/') or
139-
fn.startswith('/libpymodules.so')):
136+
if (fn.startswith('/site-packages/')
137+
or fn.startswith('/config/')
138+
or fn.startswith('/lib-dynload/')
139+
or fn.startswith('/libpymodules.so')):
140140
return False
141141
return fn
142142

@@ -183,7 +183,6 @@ def select(fn):
183183
tf = tarfile.open(tfn, 'w:gz', format=tarfile.USTAR_FORMAT)
184184
dirs = []
185185
for fn, afn in files:
186-
# print('%s: %s' % (tfn, fn))
187186
dn = dirname(afn)
188187
if dn not in dirs:
189188
# create every dirs first if not exist yet

pythonforandroid/bootstraps/pygame/build/buildlib/argparse.py

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -651,8 +651,8 @@ def _split_lines(self, text, width):
651651

652652
def _fill_text(self, text, width, indent):
653653
text = self._whitespace_matcher.sub(' ', text).strip()
654-
return _textwrap.fill(text, width, initial_indent=indent,
655-
subsequent_indent=indent)
654+
return _textwrap.fill(
655+
text, width, initial_indent=indent, subsequent_indent=indent)
656656

657657
def _get_help_string(self, action):
658658
return action.help
@@ -705,7 +705,7 @@ def _get_action_name(argument):
705705
if argument is None:
706706
return None
707707
elif argument.option_strings:
708-
return '/'.join(argument.option_strings)
708+
return '/'.join(argument.option_strings)
709709
elif argument.metavar not in (None, SUPPRESS):
710710
return argument.metavar
711711
elif argument.dest not in (None, SUPPRESS):
@@ -2070,8 +2070,8 @@ def _parse_optional(self, arg_string):
20702070

20712071
# if multiple actions match, the option string was ambiguous
20722072
if len(option_tuples) > 1:
2073-
options = ', '.join([option_string
2074-
for action, option_string, explicit_arg in option_tuples])
2073+
options = ', '.join(
2074+
[option_string for action, option_string, explicit_arg in option_tuples])
20752075
tup = arg_string, options
20762076
self.error(_('ambiguous option: %s could match %s') % tup)
20772077

0 commit comments

Comments
 (0)