Skip to content

[github] Publish onnx2circle debian package#15476

Merged
seanshpark merged 1 commit intoSamsung:masterfrom
shs-park:github/pub-onnx2circle-launchpad
May 28, 2025
Merged

[github] Publish onnx2circle debian package#15476
seanshpark merged 1 commit intoSamsung:masterfrom
shs-park:github/pub-onnx2circle-launchpad

Conversation

@shs-park
Copy link
Copy Markdown
Contributor

This implements the steps to sign and upload the onnx2circle debian package to Launchpad.

ONE-DCO-1.0-Signed-off-by: Seungho Henry Park shs.park@samsung.com

This implements the steps to sign and upload the `onnx2circle` debian
package to Launchpad.

ONE-DCO-1.0-Signed-off-by: Seungho Henry Park <shs.park@samsung.com>
@shs-park shs-park requested a review from a team May 28, 2025 05:43
@shs-park shs-park self-assigned this May 28, 2025
@shs-park
Copy link
Copy Markdown
Contributor Author

for issue #15424
from draft #15440

Comment on lines +148 to +154
- name: Conditionally install cmake for debuild
run: |
if [ "${{ matrix.ubuntu_code }}" = "focal" ]; then
echo "Installing cmake for focal"
apt update
apt install -y cmake
fi
Copy link
Copy Markdown
Contributor Author

@shs-park shs-park May 28, 2025

Choose a reason for hiding this comment

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

The debuild and debsign, in below step, verify that cmake is installed via apt,
but the focal Docker image contains a version of cmake built directly from source.

Therefore, a temporary step has been added to install cmake via apt to satisfy the build dependency check.

Copy link
Copy Markdown
Contributor

@seanshpark seanshpark left a comment

Choose a reason for hiding this comment

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

LGTM

@seanshpark seanshpark merged commit f104900 into Samsung:master May 28, 2025
3 checks passed
@shs-park shs-park deleted the github/pub-onnx2circle-launchpad branch May 28, 2025 06:15
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