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.

Sign up for GitHub

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

Jump to bottom

Remove xattr as hard dependency #602

Merged
merged 7 commits into from
Feb 16, 2023
Merged

Remove xattr as hard dependency #602

Show file tree
Hide file tree
Changes from 3 commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
553fe13
remove xattr and modify warning
simonbohnen Feb 16, 2023
4146c89
Merge branch 'main' into remove-xattr
simonbohnen Feb 16, 2023
59591ab
add repo variable
simonbohnen Feb 16, 2023
47d0cf8
change repo env variable
simonbohnen Feb 16, 2023
482c5a6
fix env variable name
simonbohnen Feb 16, 2023
a1d2fe7
maybe it is using the correct ref?
simonbohnen Feb 16, 2023
253ac87
remove commented out stuff
simonbohnen Feb 16, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions .github/workflows/ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -162,3 +162,4 @@ jobs:
token: ${{ secrets.GITHUB_TOKEN }}
env:
RH_BRANCH: ${{ steps.branch-name.outputs.current_branch }}
RH_REPO: ${{ github.repository }}
janjagusch marked this conversation as resolved.
Show resolved Hide resolved
2 changes: 2 additions & 0 deletions quetz/pkgstores.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@
import os.path as path
import shutil
import tempfile
import warnings
from contextlib import contextmanager
from os import PathLike
from threading import Lock
Expand All @@ -26,6 +27,7 @@
has_xattr = True
except ImportError:
has_xattr = False
warnings.warn("xattr not installed, not using it for file metadata")

from quetz.errors import ConfigError

Expand Down
1 change: 0 additions & 1 deletion setup.cfg
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,6 @@ install_requires =
typing_extensions
ujson
uvicorn
xattr
zstandard

[options.entry_points]
Expand Down