Skip to content

build: replace AppImage with static build binary#2884

Merged
igaw merged 1 commit intolinux-nvme:masterfrom
MaisenbacherD:master
Jul 29, 2025
Merged

build: replace AppImage with static build binary#2884
igaw merged 1 commit intolinux-nvme:masterfrom
MaisenbacherD:master

Conversation

@MaisenbacherD
Copy link
Copy Markdown
Contributor

@MaisenbacherD MaisenbacherD commented Jul 29, 2025

The AppImage build was introduced to give users access to a test binary. Before the switch to meson, the static build was used for this. In the meantime we figured out how to build a static build with meson, so there is no need for the AppImage workflow.

Start to add to the release versions of nvme the tag as postfix. Furthermore, upload the artifacts to github.

Fixes: #2876 #2398

Comment thread scripts/build.sh
Comment thread .github/workflows/upload.yml Outdated
echo "VERSION=${VERSION}" >> $GITHUB_OUTPUT
echo $VERSION

make static
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

build.sh static

The AppImage build was introduced to give users access to a test binary.
Before the switch to meson, the static build was used for this. In the
meantime we figured out how to build a static build with meson, so there
is no need for the AppImage workflow.

Start to add to the release versions of nvme the tag as postfix.
Furthermore, upload the artifacts to github.

Signed-off-by: Daniel Wagner <[email protected]>
Co-authored-by: Dennis Maisenbacher <[email protected]>
@igaw igaw merged commit 2236f57 into linux-nvme:master Jul 29, 2025
13 of 14 checks passed
@igaw
Copy link
Copy Markdown
Collaborator

igaw commented Jul 29, 2025

Thanks a lot. Let's see if all works nicely.

@MaisenbacherD
Copy link
Copy Markdown
Contributor Author

From the build log in https://github.com/linux-nvme/nvme-cli/actions/runs/16595459723/job/46940875602:

/usr/bin/ld: subprojects/openssl-3.0.8/libcrypto.a(crypto_bio_bio_addr.c.o): in function `BIO_lookup_ex':
bio_addr.c:(.text+0xb07): warning: Using 'getaddrinfo' in statically linked applications requires at runtime the shared libraries from the glibc version used for linking

@igaw
Copy link
Copy Markdown
Collaborator

igaw commented Jul 29, 2025

well there isn't much we can do, either disable the openssl support or irngore this error

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.

Replace AppImage artifcat with the static build artifact

2 participants