Skip to content

[TEST] test https://github.com/composefs/composefs-rs/pull/321#2266

Draft
giuseppe wants to merge 1 commit into
bootc-dev:mainfrom
giuseppe:test-composefs-rs-cdynlib
Draft

[TEST] test https://github.com/composefs/composefs-rs/pull/321#2266
giuseppe wants to merge 1 commit into
bootc-dev:mainfrom
giuseppe:test-composefs-rs-cdynlib

Conversation

@giuseppe

Copy link
Copy Markdown

No description provided.

@bootc-bot bootc-bot Bot requested a review from henrywang June 24, 2026 14:00
@giuseppe

Copy link
Copy Markdown
Author

@cgwalters is this test what you had in mind?

@cgwalters cgwalters left a comment

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.

Yes, doing it this way runs more tests, but there is also support for revdep testing in composefs-rs that we could wire things up to directly as well

Comment thread Dockerfile Outdated
cd /composefs-rs/src
cargo build --release -p composefs-capi
mkdir -p /out
cp target/release/libcomposefs_capi.so /out/

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.

Would be better to have e.g. make install DESTDIR= support. Maybe it should be make install-capi.

@cgwalters cgwalters added the ci/merge Run full CI suite (all OSes) — equivalent to merge queue label Jun 24, 2026
@cgwalters

Copy link
Copy Markdown
Collaborator

For PRs like this, add the ci/merge label to trigger all of the tests

@giuseppe giuseppe force-pushed the test-composefs-rs-cdynlib branch from 9f9b495 to 6b85e72 Compare June 24, 2026 14:38
Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
@giuseppe giuseppe force-pushed the test-composefs-rs-cdynlib branch from 6b85e72 to fc7e828 Compare June 24, 2026 16:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci/merge Run full CI suite (all OSes) — equivalent to merge queue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants