Skip to content

gh-116622: Redirect stdout and stderr to system log when embedded in an Android app #118063

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.

Sign up for GitHub

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

Merged
vstinner merged 19 commits into from
Apr 30, 2024
Merged

gh-116622: Redirect stdout and stderr to system log when embedded in an Android app #118063

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
1f65ec6
Redirect stdout and stderr to system log when embedded in an Android app
mhsmith Apr 17, 2024
d5cadd5
Deal with variations in older Android versions
mhsmith Apr 18, 2024
8e4d56a
Deal with even older Android versions
mhsmith Apr 18, 2024
6da6ef0
Update stdlib_module_names.h
mhsmith Apr 18, 2024
c6d9809
Move ctypes import from top level into a function, since it's not ava…
mhsmith Apr 18, 2024
1f21141
Add #ifdef to avoid unused-function warning on other platforms
mhsmith Apr 18, 2024
a8b6760
Avoid importing ctypes on startup
mhsmith Apr 24, 2024
bbe816f
Clarify comment
mhsmith Apr 24, 2024
aea0b8b
Fix issues with nulls and newlines
mhsmith Apr 24, 2024
41736d6
Improve identification of bytes-like objects
mhsmith Apr 24, 2024
02cf414
Fix C coding style
mhsmith Apr 24, 2024
61d7e98
Use LOOPBACK_TIMEOUT to wait for subprocesses, and add breadcrumb to …
mhsmith Apr 27, 2024
3014e6c
Add tests for str subclasses
mhsmith Apr 27, 2024
839de0f
Add tests for other bytes-like objects
mhsmith Apr 27, 2024
0ffb19f
Add tests for Unicode surrogates and non-standard line separators
mhsmith Apr 27, 2024
b2e2ef6
Bytes cleanups
mhsmith Apr 30, 2024
56e6f85
Str cleanups
mhsmith Apr 30, 2024
53ba406
Update configure.ac
mhsmith Apr 30, 2024
5874687
Autoreconf
mhsmith Apr 30, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Add tests for str subclasses
  • Loading branch information
@mhsmith
mhsmith committed Apr 27, 2024
commit 3014e6c375a81dd4fc809d1e5efe64c31f8ea446
4 changes: 4 additions & 0 deletions Lib/_android_support.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -47,6 +47,10 @@ def write(self, s):
raise TypeError(
f"write() argument must be str, not {type(s).__name__}")

# In case `s` is a str subclass that writes itself to stdout or stderr
# when we call its methods, convert it to an actual str.
s = str.__str__(s)

# We want to emit one log message per line wherever possible, so split
# the string before sending it to the superclass. Note that
# "".splitlines() == [], so nothing will be logged for an empty string.
Expand Down
10 changes: 10 additions & 0 deletions Lib/test/test_android.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -154,6 +154,16 @@ def write(s, lines=None):
write("hello\r\nworld\r\n", ["hello", "world"])
write("\r\n", [""])

# String subclasses are accepted, and if their methods write
# themselves, this doesn't cause infinite recursion.
class CustomStr(str):
def splitlines(self, *args, **kwargs):
sys.stdout.write(self)
return super().splitlines(*args, **kwargs)
Comment thread
serhiy-storchaka marked this conversation as resolved.
Outdated

write(CustomStr("custom\n"), ["custom"])

# Non-string classes are not accepted.
for obj in [b"", b"hello", None, 42]:
with self.subTest(obj=obj):
with self.assertRaisesRegex(
Expand Down