Skip to content

WIP: Added pipeline to test Android configuration with real phone#128

Closed
asmorkalov wants to merge 1 commit intoopencv:mainfrom
asmorkalov:as/android_phone_test
Closed

WIP: Added pipeline to test Android configuration with real phone#128
asmorkalov wants to merge 1 commit intoopencv:mainfrom
asmorkalov:as/android_phone_test

Conversation

@asmorkalov
Copy link
Copy Markdown
Contributor

No description provided.

@asmorkalov asmorkalov force-pushed the as/android_phone_test branch from eb14423 to 22aa22d Compare December 13, 2023 12:46
@asmorkalov asmorkalov force-pushed the as/android_phone_test branch from 22aa22d to 8292bed Compare December 13, 2023 12:48
paths:
- '.github/workflows/OCV-PR-4.x-Android.yaml'
workflow_call:
- '.github/workflows/OCV-4.x-Android-SDK.yaml'
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

concurrency: is not updated.

uses: actions/upload-artifact@v3
if: ${{ always() && steps.java-test.outcome == 'success' && steps.build-opencv.outcome == 'success' }}
with:
name: junit-html-macos-arm64
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

macos?

- name: Configure OpenCV
timeout-minutes: 60
run: |
cmake -B build -G Ninja ${{ env.EXTRA_CMAKE_OPTIONS }} opencv
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

There is already -G Ninja

@asmorkalov asmorkalov closed this Jan 29, 2024
asmorkalov added a commit that referenced this pull request Jan 31, 2024
Replaces #128
Depends on opencv/opencv#24910

* Added pipeline to test Android configuration with real phone.

* Cleanup environment.

* Use proper label to select host.

* Use proper label to select host.

* Set binaries cache location.

* Binaries cache path fix.

* Removed limitation.

* Set git cache location in pipeline.
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.

2 participants