]> sjero.net Git - wget/commitdiff
[svn] When did the bug get fixed where recursive FTP would fail if logging in put
authordan <devnull@localhost>
Wed, 10 Jan 2001 04:17:06 +0000 (20:17 -0800)
committerdan <devnull@localhost>
Wed, 10 Jan 2001 04:17:06 +0000 (20:17 -0800)
you in a directory other than "/"?  I don't see a src/ChangeLog entry for
it.  In any case, my testing shows that it's fixed in 1.7-dev, but TODO and
a comment in src/ftp.c were not changed to reflect this.

ChangeLog
TODO
src/ChangeLog
src/ftp.c

index 9887f30a3469953c525c943da8507fcbae82399e..3f2de993fdb89ae8527d1f4a3208a4c860f47c5b 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,6 +1,9 @@
 2001-01-09  Dan Harkless  <wget@harkless.org>
 
        * TODO: If -c is on, don't re-download a 100%-downloaded file.
+       Also, the bug where you couldn't recurse into ftp directories if
+       logging in put you somewhere else besides the server's "/"
+       directory got fixed without the TODO entry for it being removed.
 
 2001-01-06  Dan Harkless  <wget@harkless.org>
 
diff --git a/TODO b/TODO
index f7e97eaefe9082c76a3561db664dfdd58910dda8..d43301697754002d5e28183949259fec0214beed 100644 (file)
--- a/TODO
+++ b/TODO
@@ -51,9 +51,6 @@ changes.
 * Make -K compare X.orig to X and move the former on top of the latter if 
   they're the same, rather than leaving identical .orig files laying around.
 
-* Allow mirroring of FTP URLs where logging in puts you somewhere else besides 
-  '/'. 
-
 * If CGI output is saved to a file, e.g. cow.cgi?param, -k needs to change the
   '?' to a "%3F" in links to that file to avoid passing part of the filename as
   a parameter.
index c2fc00d23e18d0fc9cf2e89be806b9724c6bc5f2..064e72f5c0e7e5f7e77a1bf0a403f0c9f80dc774 100644 (file)
@@ -6,6 +6,11 @@
 
        * url.c (write_backup_file): Clarified a comment.
 
+       * ftp.c (ftp_retrieve_dirs): I don't see a ChangeLog entry for
+       this, but the bug where recursion into FTP directories didn't work
+       if logging in put you in a directory other than "/" is apparently
+       fixed now.  Removed the comment here warning of the bug.
+
 2001-01-06  Dan Harkless  <wget@harkless.org>
 
        * ChangeLog: The '[Not in 1.6 branch.]'s were decided not to be
index d38ed012cb4b39b5db47300bc81afa09e45130b9..74efff5e5263ba00efbc402b562b6d47057b6d2b 100644 (file)
--- a/src/ftp.c
+++ b/src/ftp.c
@@ -1442,9 +1442,6 @@ ftp_retrieve_dirs (struct urlinfo *u, struct fileinfo *f, ccon *con)
       if (len > current_length)
        current_container = (char *)alloca (len);
       u->dir = current_container;
-      /* When retrieving recursively, all directories must be
-        absolute.  This restriction will (hopefully!) be lifted in
-        the future.  */
       sprintf (u->dir, "/%s%s%s", odir + (*odir == '/'),
              (!*odir || (*odir == '/' && !* (odir + 1))) ? "" : "/", f->name);
       if (!accdir (u->dir, ALLABS))