Skip to content

Serialize Android samples build in SDK script to prevent raise between gradle instances#24607

Merged
asmorkalov merged 1 commit intoopencv:4.xfrom
asmorkalov:as/serialize_android_samples_build
Nov 29, 2023
Merged

Serialize Android samples build in SDK script to prevent raise between gradle instances#24607
asmorkalov merged 1 commit intoopencv:4.xfrom
asmorkalov:as/serialize_android_samples_build

Conversation

@asmorkalov
Copy link
Copy Markdown
Contributor

@asmorkalov asmorkalov commented Nov 28, 2023 •
edited
Loading

Tries to fix build errors like this:

2023-11-27T14:14:56.4627503Z FAILURE: Build failed with an exception.
2023-11-27T14:14:56.4627530Z 
2023-11-27T14:14:56.4627946Z * What went wrong:
2023-11-27T14:14:56.4629021Z Execution failed for task ':tutorial-1-camerapreview:lintVitalAnalyzeRelease'.
2023-11-27T14:14:56.4630451Z > Could not resolve all files for configuration ':tutorial-1-camerapreview:releaseCompileClasspath'.
2023-11-27T14:14:56.4636192Z    > Failed to transform out.aar (project :opencv) to match attributes {artifactType=android-lint-exploded-aar, com.android.build.api.attributes.AgpVersionAttr=7.3.1, com.android.build.api.attributes.BuildTypeAttr=release, com.android.build.gradle.internal.attributes.VariantAttr=release, org.gradle.usage=java-api, org.jetbrains.kotlin.platform.type=androidJvm}.
2023-11-27T14:14:56.4638063Z       > Execution failed for ExtractAarTransform: /home/ci/build/o4a/opencv_android/opencv/build/intermediates/local_aar_for_lint/release/out.aar.
2023-11-27T14:14:56.4638630Z          > unexpected EOF
2023-11-27T14:14:56.4638644Z 
2023-11-27T14:14:56.4638837Z * Try:
2023-11-27T14:14:56.4639515Z > Run with --stacktrace option to get the stack trace.
2023-11-27T14:14:56.4640066Z > Run with --debug option to get more log output.
2023-11-27T14:14:56.4640492Z > Run with --scan to get full insights.

Pull Request Readiness Checklist

See details at https://github.com/opencv/opencv/wiki/How_to_contribute#making-a-good-pull-request

  • I agree to contribute to the project under Apache 2 License.
  • To the best of my knowledge, the proposed patch is not based on a code under GPL or another license that is incompatible with OpenCV
  • The PR is proposed to the proper branch
  • There is a reference to the original bug report and related work
  • There is accuracy test, performance test and test data in opencv_extra repository, if applicable
    Patch to opencv_extra has the same branch name.
  • The feature is well documented and sample code can be built with the project CMake

@asmorkalov asmorkalov added this to the 4.9.0 milestone Nov 28, 2023
@asmorkalov asmorkalov requested a review from mshabunin November 28, 2023 11:39
@asmorkalov asmorkalov changed the title WIP: Serialize Android samples build in SDK script to prevent raise between gradle instances Serialize Android samples build in SDK script to prevent raise between gradle instances Nov 28, 2023
@asmorkalov
Copy link
Copy Markdown
Contributor Author

asmorkalov commented Nov 28, 2023

@mshabunin could you take a look?

@asmorkalov
Copy link
Copy Markdown
Contributor Author

asmorkalov commented Nov 28, 2023

@mshabunin I triggered CI several times and the original failure is not visible any more.

mshabunin
mshabunin approved these changes Nov 29, 2023
View reviewed changes
Copy link
Copy Markdown
Contributor

@mshabunin mshabunin left a comment

Choose a reason for hiding this comment

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

Let's try it.
(though it could be a symptom of dependency problem)

@asmorkalov asmorkalov merged commit 75f619f into opencv:4.x Nov 29, 2023
@asmorkalov asmorkalov mentioned this pull request Jan 19, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mshabunin mshabunin mshabunin approved these changes

Assignees

@mshabunin mshabunin

Labels

category: build/install platform: android

Projects

None yet

Milestone

4.9.0

Development

Successfully merging this pull request may close these issues.

2 participants

@asmorkalov @mshabunin