Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make the test work w/ non-default hooks directory #174

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

we3i
Copy link

@we3i we3i commented Sep 7, 2024

By default the hook is expected to locate in
/usr/share/containers/oci/hooks.d
However, "make install" uses the prefix /usr/local as default. This change enables "make test-integration" to work in this situation.

It does not affect the test in the packaging where the hook is installed in the default directory.

By default the hook is expected to locate in
/usr/share/containers/oci/hooks.d
However, "make install" uses the prefix /usr/local as default. This
change enables "make test-integration" to work in this situation.

Signed-off-by: Wei <[email protected]>
@we3i
Copy link
Author

we3i commented Sep 7, 2024

Just found that the automatic checks use PREFIX=/usr in make and make install which break the test-integration. To fix it, either make PREFIX=/usr test-integration or remove PREFIX=/usr from make and make install (use the default /usr/local)

Copy link
Member

@vrothberg vrothberg left a comment

Choose a reason for hiding this comment

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

Thanks for contributing and sorry for the delay!

The tests don't pass in CI.

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