Skip to content

[infra/onert] Disable SentencePiece as default all tizen platforms#16272

Merged
hseok-oh merged 1 commit intoSamsung:masterfrom
hseok-oh:disable_tizen_ggma
Nov 6, 2025
Merged

[infra/onert] Disable SentencePiece as default all tizen platforms#16272
hseok-oh merged 1 commit intoSamsung:masterfrom
hseok-oh:disable_tizen_ggma

Conversation

@hseok-oh
Copy link
Copy Markdown
Contributor

@hseok-oh hseok-oh commented Nov 5, 2025

This commit disables SentencePiece as default for all tizen platforms.
BUILD_GGMA_API also disabled, but enabled by nnfw.spec.

ONE-DCO-1.0-Signed-off-by: Hyeongseok Oh hseok82.oh@samsung.com

This commit disables SentencePiece as default for all tizen platforms.
BUILD_GGMA_API also disabled, but enabled by nnfw.spec.

ONE-DCO-1.0-Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>
@hseok-oh hseok-oh added the PR/ready for review It is ready to review. Please review it. label Nov 5, 2025
@hseok-oh
Copy link
Copy Markdown
Contributor Author

hseok-oh commented Nov 5, 2025

It will resolve build failure #15030 (comment)

@hseok-oh hseok-oh requested a review from a team November 5, 2025 06:22
@hseok-oh
Copy link
Copy Markdown
Contributor Author

hseok-oh commented Nov 6, 2025

Already disabled on armv7l platform by #16267, but not disabled on others - x86-64, aarch64, armv7hl

@Samsung/one_onert PTAL

@hseok-oh
Copy link
Copy Markdown
Contributor Author

hseok-oh commented Nov 6, 2025

Daily package build fails because of this issue

Copy link
Copy Markdown
Contributor

@ragmani ragmani left a comment

Choose a reason for hiding this comment

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

LGTM

@hseok-oh hseok-oh merged commit bddcd02 into Samsung:master Nov 6, 2025
10 checks passed
@hseok-oh hseok-oh deleted the disable_tizen_ggma branch November 6, 2025 07:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR/ready for review It is ready to review. Please review it.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants