Skip to content

Commit

Permalink
fixup! [wip][feature] Add support for fsspec backends
Browse files Browse the repository at this point in the history
  • Loading branch information
mxmlnkn committed Oct 6, 2024
1 parent e883d6c commit 0a687fd
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 4 deletions.
9 changes: 6 additions & 3 deletions .github/workflows/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -192,16 +192,19 @@ jobs:
working-directory: core
run: |
python3 -m build
ls -la dist/
zipinfo dist/*whl
twine check dist/*
python3 -m pip install "$( find dist -name '*.tar.gz' | head -1 )"[full]
- name: Test Installation From Tarball
run: |
python3 -m build
ls -la dist/
zipinfo dist/*whl
base64 dist/*whl
unzip -o dist/*whl
echo "Try to repair wheel"
zip -F dist/*whl --out fixed
mv fixed dist/*whl
#zip -FF dist/*whl
twine check dist/*
python3 -m pip install "$( find dist -name '*.tar.gz' | head -1 )"[full]
Expand Down
3 changes: 2 additions & 1 deletion core/ratarmountcore/FSSpecMountSource.py
Original file line number Diff line number Diff line change
Expand Up @@ -52,9 +52,11 @@ class FSSpecMountSource(MountSource):
# 1173 fsspec/implementations/reference.py
# I guess git is the most obvious candidate because it is the most interesting and most barebone implementation.

# pylint: disable=unused-argument
def __init__(self, urlOrOpenFile, **options) -> None:
"""
urlOrOpenFile : Take a URL or an already opened fsspec Filesystem object.
Note that this might take an AbstractFileSystem-derived object in the future.
"""
# Note that fsspec.implementations.ssh did not use ~/.ssh/config!
# That's one of the many reasons why fsspec/sshfs based on asyncssh instead of paramiko is used.
Expand All @@ -63,7 +65,6 @@ def __init__(self, urlOrOpenFile, **options) -> None:
fsspec.open(urlOrOpenFile) if isinstance(urlOrOpenFile, str) else urlOrOpenFile
)
self.fileSystem: fsspec.AbstractFileSystem = self.openFile.fs
self.options = options
self.rootFileInfo = createRootFileInfo(userdata=["/"])

# The fsspec filesystems are not uniform! http:// expects the arguments to isdir with prefixed
Expand Down

0 comments on commit 0a687fd

Please sign in to comment.