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

[DO NOT REVIEW][CPU][GHA] Try to build cross compilation framework #25673

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

yury-intel
Copy link
Contributor

Details:

  • item1
  • ...

Tickets:

  • ticket-id

@yury-intel yury-intel requested a review from a team as a code owner July 22, 2024 14:05
@github-actions github-actions bot added category: CI OpenVINO public CI github_actions Pull requests that update GitHub Actions code labels Jul 22, 2024
Copy link
Contributor

Choose a reason for hiding this comment

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

why do we need dev workflow? can we directly change existing RISC-V one?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

If nobody needs existing WF we can override it.

@yury-intel yury-intel force-pushed the gha/try_to_build_crosscompilation_framework branch from 8019fdf to e67820d Compare July 30, 2024 13:34
@yury-intel yury-intel force-pushed the gha/try_to_build_crosscompilation_framework branch from 460a5ff to 8cb45cc Compare August 8, 2024 10:53
@yury-intel yury-intel force-pushed the gha/try_to_build_crosscompilation_framework branch from 2cbbed0 to 5213ecb Compare August 27, 2024 11:57
CCACHE_DIR: /__w/openvino/openvino/ccache
CCACHE_TEMPDIR: /__w/openvino/openvino/ccache_temp
CCACHE_MAXSIZE: 2G
XUANTIE_BIN_PATH: /mount/testdata1/Xuantie-900-gcc-linux-5.15.0-glibc-x86_64-V2.8.1/
Copy link
Contributor

Choose a reason for hiding this comment

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

it should be a part of docker image, IMO

Also, docker image should export path to this toolchain via RISCV_TOOLCHAIN_ROOT (see https://github.com/openvinotoolkit/openvino/blob/master/cmake/toolchains/riscv64-100-thead-gnu.toolchain.cmake#L28)

-DCMAKE_CXX_COMPILER_LAUNCHER=${{ env.CMAKE_CXX_COMPILER_LAUNCHER }} \
-DCMAKE_C_COMPILER_LAUNCHER=${{ env.CMAKE_C_COMPILER_LAUNCHER }} \
-S ${OPENVINO_REPO} \
-B ${BUILD_DIR}
Copy link
Contributor

Choose a reason for hiding this comment

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

instead of manual copy of build steps, can you reuse existing Linux build reusable workflow ?
It's being improved in this PR #26156

@yury-intel yury-intel force-pushed the gha/try_to_build_crosscompilation_framework branch from c912b35 to e8fca64 Compare September 23, 2024 10:11
Copy link
Contributor

This PR will be closed in a week because of 2 weeks of no activity.

@github-actions github-actions bot added the Stale label Oct 17, 2024
Copy link
Contributor

This PR was closed because it has been stalled for 2 week with no activity.

@github-actions github-actions bot closed this Oct 24, 2024
@yury-intel yury-intel reopened this Oct 31, 2024
@github-actions github-actions bot removed the Stale label Nov 1, 2024
@yury-intel yury-intel force-pushed the gha/try_to_build_crosscompilation_framework branch 2 times, most recently from 90e63cf to 5556eb6 Compare November 14, 2024 12:53
@yury-intel yury-intel force-pushed the gha/try_to_build_crosscompilation_framework branch 3 times, most recently from 2717778 to 894cc05 Compare November 30, 2024 01:43
@yury-intel yury-intel force-pushed the gha/try_to_build_crosscompilation_framework branch 9 times, most recently from 6f87ddc to 2c70050 Compare December 12, 2024 20:48
@yury-intel yury-intel force-pushed the gha/try_to_build_crosscompilation_framework branch 2 times, most recently from f79cee9 to 9849287 Compare December 15, 2024 12:18
@yury-intel yury-intel force-pushed the gha/try_to_build_crosscompilation_framework branch from 9849287 to e094200 Compare December 16, 2024 09:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: CI OpenVINO public CI category: dockerfiles github_actions Pull requests that update GitHub Actions code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants