Age | Commit message (Collapse) | Author |
|
|
|
|
|
Bug:
Test:
Change-Id: I986d8eb6a93bdfe121ccdd40a1a3818e3968cd6e
|
|
Allows mypy to pass on ndkgdb.py.
Bug: None
Test: mypy ndkgdb.py
Change-Id: Idc312001d8e1bb47b52563cd7d41d9bb07d510b4
|
|
Bug: None
Test: ./checkbuild.py pythonlint
Change-Id: I722a480b297d15671129e8e66ee59f51f05e4817
|
|
Bug: None
Test: ./checkbuild.py ndk-gdb ndk-stack
Change-Id: Ic50171909bdf0de87b81dbadff9dc6eddead36bc
|
|
|
|
* changes:
Make ndkstack mypy clean.
Make ndkgdb pylint clean.
|
|
Bug: None
Test: mypy ndkstack
Change-Id: I9410dde2c259e776b19f5cfc3dd5a59d6b1bc1b0
|
|
Bug: None
Test: pylint ndkgdb.py
Change-Id: Iaddafcae49bce0f41790957d4b6d17f755bbfc6c
|
|
|
|
|
|
|
|
Bug: None
Test: built and ran ndk-gdb --help
Change-Id: I008958c70af173b33118cba2dedf54283f1c5272
|
|
Prep work to make tests easier to write.
This isn't complete and can't be until gdbrunner is a real package with
type annotations. I'll do that next.
Bug: None
Test: mypy ndkgdb.py
Change-Id: Ib312c7d8084b4480cd3789534f16f0923d52bce3
|
|
xml.etree.ElementTree automatically uses cElementTree when available.
Explicit imports of cElementTree have been deprecated since Python 3.3.
Bug: None
Test: None
Change-Id: I981bce1fde9a3ffc2bff8ef88ae0051f53219001
|
|
This was collateral damage when deleting the now unnecessary sys.path
manipulation. Discovered while trying to make this lint cleanly.
Bug: None
Test: trying to get lint clean so I can reasonably write tests...
Change-Id: I5bfb9a83482cc576e5f9da21056b5ca4c3804618
|
|
Bug: None
Test: lldb.sh # on macOS
Change-Id: If049b72f994fc48bdf7ae675da374a0bb218cca3
|
|
|
|
For the reason explained in the comment, this is impossible to comply
with, and I'm tired of forgetting to pass
--permissive-python-environment for everything I do.
Bug: None
Test: ./run_tests.py # on macOS with a non-prebuilt python
Change-Id: I54b9f0cb0171ba1127bc366f3a34d8e2d218e003
|
|
* changes:
Remove unused MultiFileModule.
Move ndk-stack to PythonApplication as well.
Convert ndk-gdb to a PEP-441 zip application.
|
|
Bug: None
Test: ./checkbuild.py pythonlint
Change-Id: Ic6286481d0990cec9b4b682babff503a368e2aea
|
|
Bug: None
Test: $NDK/ndk-stack --help
Change-Id: I94e1370ea3969343d808be2020f52f29bb9d60c0
|
|
Python supports executing zipped applications and running them in that
manner makes dependency discovery easier, since the application
directory will automatically be added to sys.path.
We do have to stop using hyphens in our script names though, because
those aren't valid python modules.
This does potentially allow us to do more cool things. If we moved
ndkgdb.py into ndk.tools, we could include the rest of the ndk package
so we can use things like ndk.hosts and ndk.abis. Moving it there also
makes it easier to ensure that it'll be covered by our linters and
tests, which was the real motivation for this change.
Bug: None
Test: ./checkbuild.py ndk-gdb ndk-gdb-shortcut && $NDK/ndk-gdb --help
Change-Id: Ibdd0c2d42a2eccaf46d188221421601f740b02d9
|
|
|
|
|
|
Includes the fix for libc++abi TLS.
Bug: None
Test: ./checkbuild.py && ./run_tests.py
Change-Id: I0cb1bc6f5a0d842b535a0b391cfe7e839430a4ec
|
|
Bug: None
Test: None
Change-Id: I913c143e96532acc3f67772e2d888ee553c9a067
|
|
|
|
Bug: https://github.com/android/ndk/issues/1883
Test: None
Change-Id: Ic99cdf916bd890dd8583d61ba7b7af65819377c2
|
|
|
|
Bug: https://github.com/android/ndk/issues/1861
Bug: https://github.com/android/ndk/issues/1880
Test: None
Change-Id: I2473eaf5bf042e57c42910d773c5fd3eff28e1c4
|
|
|
|
We probably don't have any bugs here since we're unlikely to write
anything not in the ASCII range, and most platforms use UTF-8 as the
default anyway. The changes in ndk-build might be symptomatic for some
inputs on Windows though.
Bug: None
Test: poetry run ./checkbuild.py pythonlint
Change-Id: I1069d165db8eae1ea6fb31ce3216ca58a72e8e9e
|
|
|
|
* changes:
Remove libc++ testing support.
Use the libc++ from the toolchain.
|
|
This is now tested as part of the toolchain release. We don't have the
source of the tests that match the toolchain's libc++, so we can't run
them any more. Even if we could, it's not useful to run them this
late.
Plus, look at all the code that we get to delete! This was always a
gross hack and I'm glad to see it go. This also makes our test
artifacts significantly smaller, and building/running the tests it
*much* faster (7 minutes to build, 40 seconds to run).
Bug: None
Test: poetry run ./run_tests.py --rebuild.
Change-Id: I512417dd9d088c5b797bbc034a387670cffcf955
|
|
Bug: https://github.com/android/ndk/issues/1530
Test: ./checkbuild.py && ./run_tests.py
Change-Id: Id8f99069e1c0bbe532626ebd99054689872caa72
|
|
Bug: https://github.com/android/ndk/issues/1764
In Python 2, map() returns a list which you can freely concatenate to another list. In Python 3 map() returns a lazy map object (a generator) and cant be immediately concatenated to a list
Fix suggested in: https://github.com/android/ndk/issues/1764#issuecomment-1242521200
Change-Id: I705afb179d3f71162c3637b748ceb4c06de04f93
|
|
|
|
|
|
Bug: None
Test: None
Change-Id: I9a354a3875f1b13286263a8b8fd28ac2dd14bf68
|
|
|
|
Even after r19 made this unnecessary, ndk-build continued to do it to
prevent incorrectly built libraries from poisoning the app with the
wrong unwinder. The mistakes that allowed that to happen were fixed a
long time ago (in our build systems, anyway), and when getting libc++
from the toolchain it's quite a bit more difficult to keep our
workaround functioning. This drops it and brings ndk-build's behavior
more in line with CMake's (and probably every other build system, since
I don't think any system other than ndk-build was ever this thorough).
Bug: None
Test: treehugger
Change-Id: Ifc46ed144b98e952e1b86736424567f384b758f7
|
|
|
|
This is needed by the next patch which delegates C++ stdlib linking to
the driver. Without this, C-only modules will be linked to the C++
stdlib.
Bug: None
Test: existing tests pass (LOCAL_HAS_CPP* covers this behavior well)
Change-Id: Ic723b2296d2e634537c56aed7670e2a9bf3d0a2d
|
|
* changes:
Allow black to check the build system and tests.
Format overlooked files.
|
|
These were excluded by default because black thinks they're build output
rather than build tools and tests.
Bug: None
Test: poetry run ./checkbuild.py black
Change-Id: I421c9dca525976b388d98f75d4da35c3a8a95dbf
|
|
The default black config excludes anything that matches "build". The
next commit fixes the config to check these.
Bug: None
Test: None
Change-Id: I24d2d89ee035a7b93c4b25e40df3a09941b314f7
|
|
Rather than only using the `readelf` that is in PATH, support (in order
of preference):
1. The llvm-readelf in --ndk
2. The llvm-readelf in the out directory
3. The llvm-readelf in PATH
4. The readelf in PATH
Bug: None
Test: poetry run ./parse_elfnote.py --ndk <installed NDK> <library>
Test: poetry run ./parse_elfnote.py <library> # in dev tree
Change-Id: Ie6cf2dec11d311b1cef360e57b080e001e83f0d1
|