Skip to content

allow setting the value for android:allowBackup#1438

Merged
AndreMiras merged 2 commits into
masterfrom
allow_backup
Nov 4, 2018
Merged

allow setting the value for android:allowBackup#1438
AndreMiras merged 2 commits into
masterfrom
allow_backup

Conversation

@tshirtman

Copy link
Copy Markdown
Member

No description provided.

@AndreMiras

Copy link
Copy Markdown
Member

Travis seems unhappy about this one:
https://api.travis-ci.org/v3/job/450094772/log.txt

Second install (expected to work)
('cwd is', '/home/user/.local/share/python-for-android/build/other_builds/pyjnius-python2-sdl2/armeabi/pyjnius')
()
('cwd is', '/home/user/.local/share/python-for-android/build/other_builds/kivy-python2-sdl2/armeabi/kivy')
()
objs_paths are ['/home/user/.local/share/python-for-android/build/bootstrap_builds/sdl2_gradle-python2/collated_objects']
Traceback (most recent call last):
  File "setup_testapp_python2.py", line 30, in <module>
    package_data={'testapp': ['*.py', '*.png']}
  File "/usr/lib/python2.7/distutils/core.py", line 151, in setup
    dist.run_commands()
  File "/usr/lib/python2.7/distutils/dist.py", line 953, in run_commands
    self.run_command(cmd)
  File "/usr/lib/python2.7/distutils/dist.py", line 972, in run_command
    cmd_obj.run()
  File "/home/user/pythonforandroid/bdistapk.py", line 81, in run
    main()
  File "/home/user/pythonforandroid/toolchain.py", line 999, in main
    ToolchainCL()
  File "/home/user/pythonforandroid/toolchain.py", line 532, in __init__
    getattr(self, args.subparser_name.replace('-', '_'))(args)
  File "/home/user/pythonforandroid/toolchain.py", line 146, in wrapper_func
    func(self, args)
  File "/home/user/pythonforandroid/toolchain.py", line 774, in apk
    build_args = build.parse_args(args.unknown_args)
  File "/home/user/.local/share/python-for-android/dists/bdisttest_python2/build.py", line 579, in parse_args
    make_package(args)
  File "/home/user/.local/share/python-for-android/dists/bdisttest_python2/build.py", line 365, in make_package
    url_scheme=url_scheme)
  File "/home/user/.local/share/python-for-android/dists/bdisttest_python2/build.py", line 76, in render
    template = environment.get_template(template)
  File "/home/user/venv/local/lib/python2.7/site-packages/jinja2/environment.py", line 830, in get_template
    return self._load_template(name, self.make_globals(globals))
  File "/home/user/venv/local/lib/python2.7/site-packages/jinja2/environment.py", line 804, in _load_template
    template = self.loader.load(self, name, globals)
  File "/home/user/venv/local/lib/python2.7/site-packages/jinja2/loaders.py", line 125, in load
    code = environment.compile(source, name, filename)
  File "/home/user/venv/local/lib/python2.7/site-packages/jinja2/environment.py", line 591, in compile
    self.handle_exception(exc_info, source_hint=source_hint)
  File "/home/user/venv/local/lib/python2.7/site-packages/jinja2/environment.py", line 780, in handle_exception
    reraise(exc_type, exc_value, tb)
  File "/home/user/.local/share/python-for-android/dists/bdisttest_python2/templates/AndroidManifest.tmpl.xml", line 56, in template
    android:allowBackup="{% args.allow_backup %}"
jinja2.exceptions.TemplateSyntaxError: Encountered unknown tag 'args'.

@tshirtman

Copy link
Copy Markdown
Member Author

Hm, indeed i must have messed up, looking again.

@AndreMiras AndreMiras left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@AndreMiras AndreMiras merged commit e6cf4a9 into master Nov 4, 2018
@AndreMiras AndreMiras deleted the allow_backup branch November 4, 2018 00:30
@SomberNight

Copy link
Copy Markdown
Contributor

Is there a way to set this from buildozer.spec?

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.

3 participants