> On March 2, 2014, 12:14 a.m., Dawit Alemayehu wrote: > > I am curious if stripping the leading "/" and sending a relative path with > > the SIZE command would work for every FTP server?
It's worth a try. I took at look at the very old svn history, and it has just always been an absolute path, we never tried relative. (so it's not like this would be reverting some bugfix) I think it was just a way to avoid sending one more command (cwd). But it seems logical to do that (cwd, then size, then get). - David ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116524/#review51562 ----------------------------------------------------------- On March 2, 2014, 12:12 a.m., Dawit Alemayehu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/116524/ > ----------------------------------------------------------- > > (Updated March 2, 2014, 12:12 a.m.) > > > Review request for kdelibs and David Faure. > > > Bugs: 326292 > http://bugs.kde.org/show_bug.cgi?id=326292 > > > Repository: kdelibs > > > Description > ------- > > This patch changes Ftp::ftpSize such that it has support for servers that do > not allow absolute paths with the SIZE command. That means when sending the > command "SIZE /somefile" fails, it will try sending "SIZE somefile" before > giving up. See bug report for details. > > > Diffs > ----- > > kioslave/ftp/ftp.cpp 5bb2e8d > > Diff: https://git.reviewboard.kde.org/r/116524/diff/ > > > Testing > ------- > > Installed Ftp server from bug report on an Android device and run tests. > > > Thanks, > > Dawit Alemayehu > >
