remove _lsLine from sftpd.py #2704

Open
opened 2016-01-05 15:27:09 +00:00 by leif · 1 comment
Owner

The _lsLine function in source:src/allmydata/frontends/sftpd.py was copied from Twisted and has a comment mentioning that after Twisted 8.2 is/was released we should use the Twisted version instead. Twisted's version of the function has changed significantly since then; we should make sure the new one works and then remove this from the Tahoe-LAFS code.

The `_lsLine` function in source:src/allmydata/frontends/sftpd.py was copied from Twisted and has a comment mentioning that after Twisted 8.2 is/was released we should use the Twisted version instead. Twisted's version of the function has changed significantly since then; we should make sure the new one works and then remove this from the Tahoe-LAFS code.
tahoe-lafs added the
unknown
normal
defect
1.10.2
labels 2016-01-05 15:27:09 +00:00
tahoe-lafs added this to the undecided milestone 2016-01-05 15:27:09 +00:00
tahoe-lafs added
code-frontend-ftp-sftp
and removed
unknown
labels 2016-08-07 17:54:36 +00:00
exarkun commented 2020-01-17 16:30:13 +00:00
Author
Owner

The functions have diverged sufficiently now that it's not a simple replacement. It seems like this might still be a good idea but someone will have to read both functions carefully and figure out if they are still basically accomplishing the same thing or not.

The functions have diverged sufficiently now that it's not a simple replacement. It seems like this might still be a good idea but someone will have to read both functions carefully and figure out if they are still basically accomplishing the same thing or not.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: tahoe-lafs/trac-2024-07-25#2704
No description provided.