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

Ensure xattr checksum caching is tested #236

Open
malmond77 opened this issue Mar 15, 2021 · 0 comments
Open

Ensure xattr checksum caching is tested #236

malmond77 opened this issue Mar 15, 2021 · 0 comments

Comments

@malmond77
Copy link
Contributor

Context: https://github.com/rpm-software-management/librepo/pull/234/files/2743583e444d745526e9bb8fad249ce6ea08e0f9#r594141462

@m-blaha highlights a bug in the test suite where xattrs are tested. The temporary files often end up in /tmp which on some systems can be tmpfs, which doesn't support user attributes.

Two directions are possible:

  1. Force tests to use non tmpfs, which improves the odds of getting a filesystem that supports xattrs with user attributes
  2. Mock the xattr calls, removing the hard dependency on real xattr support
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

No branches or pull requests

1 participant