iterative/scmrepo

Version restriction on asyncssh

ghisvail opened this issue · 4 comments

What are the rationales for the version restrictions for asyncssh:

asyncssh>=2.7.1,<2.9

It currently makes dvc uninstallable on Nix due to asyncssh being newer.

I was wondering whether the upper bound could be dropped?

It's to match https://github.com/fsspec/sshfs but in both cases the restrictions just weren't tested/updated when asyncssh released 2.9.

We should probably just change the upper restriction in both repos to the next major version instead of minor (so < 3 instead of <2.x)

Ok, so there are API changes in 2.9 that break both this repo and sshfs (see the linked PR's), we'll have to update both repos before we can loosen the upper bound

It's to match https://github.com/fsspec/sshfs but in both cases the restrictions just weren't tested/updated when asyncssh released 2.9.

I am not quite sure I follow. You are not depending on sshfs directly, are you?

I am not quite sure I follow. You are not depending on sshfs directly, are you?

This package does not depend on sshfs, but DVC does, so supporting asyncssh 2.9 in scmrepo but not sshfs would not work as a solution to your issue