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

newest code can not build now, but 0.8.1 is no problem in the same environment configuration #989

Closed
winnerlwn opened this issue Feb 15, 2016 · 1 comment

Comments

@winnerlwn
Copy link

Error:Execution failed for task ':imagepipeline:ndk_build_bitmaps'.

A problem occurred starting process 'command 'D:\Android\android-ndk-r10b\ndk-build''

@IanChilds
Copy link
Contributor

Looks like a problem with your NDK. Can you make sure that you have everything set up correctly?

@hearsilent hearsilent mentioned this issue Jun 11, 2018
4 tasks
facebook-github-bot pushed a commit that referenced this issue Jun 21, 2018
Summary:
Thanks for submitting a PR! Please read these instructions carefully:

- [x] Explain the **motivation** for making this change.
- [x] Provide a **test plan** demonstrating that the code is solid.
- [x] Match the **code formatting** of the rest of the codebase.
- [x] Target the `master` branch

What existing problem does the pull request solve?

Fix build failed occurred by ndk settings ([Application.mk](https://developer.android.com/ndk/guides/application_mk))
Remove armeabi support (Remove from NDK r17)

```
> Task :imagepipeline:ndk_build_bitmaps FAILED
A problem was found with the configuration of task ':imagepipeline:ndk_build_bitmaps'. Registering invalid inputs and outputs via TaskInputs and TaskOutputs methods has been deprecated and is scheduled to be removed in Gradle 5.0.
 - File '.../fresco/imagepipeline/src/main/jni/bitmaps' specified for property '$1' is not a file.
Android NDK: APP_PLATFORM not set. Defaulting to minimum supported version android-14.
Android NDK: WARNING: APP_STL gnustl_static is deprecated and will be removed in the next release. Please switch to either c++_static or c++_shared. See https://developer.android.com/ndk/guides/cpp-support.html for more information.
Android NDK: The armeabi ABI is no longer supported. Use armeabi-v7a.
.../Android/sdk/ndk-bundle/build/core/setup-app.mk:79: *** Android NDK: Aborting    Android NDK: NDK Application 'local' targets unknown ABI(s): armeabi
.  Stop.
Android NDK: Please fix the APP_ABI definition in ../Application.mk
make: Entering directory `.../fresco/imagepipeline/src/main/jni/bitmaps'
make: Leaving directory `.../fresco/imagepipeline/src/main/jni/bitmaps'
```

Relative issue: #24 #163 #286 #513 #591 #618 #938 #989 #1114 #1205 #1226 #1427 #1945

A good test plan has the exact commands you ran and their output, provides screenshots or videos if the pull request changes UI or updates the website. See [What is a Test Plan?][1] to learn more.

If you have added code that should be tested, add tests.

Build with NDK r17b
<img width="677" alt="2018-06-15 11 00 00" src="https://user-images.githubusercontent.com/5742579/41448216-de9ac710-708b-11e8-865c-70ac85459d5c.png">

Sign the [CLA][2], if you haven't already.

Small pull requests are much easier to review and more likely to get merged. Make sure the PR does only one thing, otherwise please split it.

Make sure all **tests pass** on [Circle CI][4]. PRs that break tests are unlikely to be merged.

For more info, see the [Contributing guide][4].

[1]: https://medium.com/martinkonicek/what-is-a-test-plan-8bfc840ec171#.y9lcuqqi9
[2]: https://code.facebook.com/cla
[3]: http://circleci.com/gh/facebook/fresco
[4]: https://github.com/facebook/fresco/blob/master/CONTRIBUTING.md
Closes #2134

Reviewed By: erikandre

Differential Revision: D8441064

Pulled By: oprisnik

fbshipit-source-id: 6484f7714d738c9984c273a3e0513e31f730ff30
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

No branches or pull requests

2 participants