remove _lsLine from sftpd.py #2704

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

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.
leif added the
unknown
normal
defect
1.10.2
labels 2016-01-05 15:27:09 +00:00
leif added this to the undecided milestone 2016-01-05 15:27:09 +00:00
warner added
code-frontend-ftp-sftp
and removed
unknown
labels 2016-08-07 17:54:36 +00:00

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 Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

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