Skip to content

Conversation

@piotrrak
Copy link

@piotrrak piotrrak commented Mar 7, 2025

Quick fix for Ubuntu when OpenBLAS is build as subproject

Resolves: #2996

Self evaluation:

  1. Build test: [*]Passed [ ]Failed []Skipped
  2. Run test: [ ]Passed [ ]Failed [*]Skipped

How to evaluate:
Not applicable

@piotrrak
Copy link
Author

piotrrak commented Mar 7, 2025

@gkisalapl Fine for windows?

For linux (Ubuntu) it is best if we build OpenBLAS as shared library.
If system OpenBLAS is availble it will be used.

Signed-off-by: Piotr Rak <[email protected]>
@piotrrak piotrrak force-pushed the linux-blas-subproject branch from 093999e to 0e34a8f Compare March 7, 2025 14:02
@piotrrak
Copy link
Author

piotrrak commented Mar 7, 2025

Fixed typo s/window/windows in OS name.

Copy link
Contributor

@djeong20 djeong20 left a comment

Choose a reason for hiding this comment

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

Since we added OpenBLAS as a subproject, could we remove remove the script jni/prepare_openblas.sh?

@piotrrak
Copy link
Author

Since we added OpenBLAS as a subproject, could we remove remove the script jni/prepare_openblas.sh?

If and only if we'd provide Android NDK based standalone toolchain and https://developer.android.com/ndk/guides/other_build_systems and meson toolchain definition to allow cross compilation for android.
Right now it uses ndk-build to cross compile for android where this script is used.
As such this change is not related.

@myungjoo myungjoo merged commit 57e0402 into nnstreamer:main Mar 10, 2025
20 checks passed
@piotrrak piotrrak deleted the linux-blas-subproject branch March 10, 2025 11:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

nntrainer won't build on Linux x86_64 with OpenBLAS provided by subproject

6 participants