Skip to content
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.

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

ndk-build: Remove unparsable quotes from Android.mk for ndk-gdb #344

Merged
merged 1 commit into from
Sep 9, 2022

Conversation

MarijnS95
Copy link
Member

On Windows it seems the ndk-gdb.py script that invokes make to dump the variables from Android.mk fails to strip quotation marks from the file correctly, resulting in erroneous output like (notice double quotes at the end of the path):

OSError: [WinError 123] The filename, directory name, or volume
label syntax is incorrect:
'android-ndk-rs\\target\\debug\\apk\\examples\\""'

And:

Application cannot run on the selected device.
Application ABIs: "arm64-v8a"
Device ABIs: arm64-v8a, armeabi-v7a, armeabi

Solve this by removing the quotes, allowing ndk-gdb to launch sucessfully.

On Windows it seems the `ndk-gdb.py` script that invokes `make` to dump
the variables from `Android.mk` fails to strip quotation marks from the
file correctly, resulting in erroneous output like (notice double quotes
at the end of the path):

	OSError: [WinError 123] The filename, directory name, or volume
	label syntax is incorrect:
	'android-ndk-rs\\target\\debug\\apk\\examples\\""'

And:

	Application cannot run on the selected device.
	Application ABIs: "arm64-v8a"
	Device ABIs: arm64-v8a, armeabi-v7a, armeabi

Solve this by removing the quotes, allowing `ndk-gdb` to launch
sucessfully.
@MarijnS95
Copy link
Member Author

Tested on Windows and Linux.

@MarijnS95 MarijnS95 merged commit 17a1450 into master Sep 9, 2022
@MarijnS95 MarijnS95 deleted the android-mk-no-quotes branch September 9, 2022 16:06
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.

None yet

1 participant