home *** CD-ROM | disk | FTP | other *** search
/ NetNews Usenet Archive 1992 #26 / NN_1992_26.iso / spool / gnu / utils / bug / 2017 < prev    next >
Encoding:
Text File  |  1992-11-08  |  5.2 KB  |  250 lines
  1. Newsgroups: gnu.utils.bug
  2. Path: sparky!uunet!think.com!rpi!zaphod.mps.ohio-state.edu!cis.ohio-state.edu!ced.berkeley.edu!jeg
  3. From: jeg@ced.berkeley.edu (James Ganong)
  4. Subject: more on tar-1.11.1 incremental dumps
  5. Message-ID: <9211090201.AA16987@ced.berkeley.edu>
  6. Sender: gnulists@ai.mit.edu
  7. Organization: GNUs Not Usenet
  8. Distribution: gnu
  9. Date: Mon, 9 Nov 1992 02:01:08 GMT
  10. Approved: bug-gnu-utils@prep.ai.mit.edu
  11. Lines: 237
  12.  
  13. i sent in a patch for tar-1.11.1/gnu.c which i have repeated below.
  14.  
  15. one of the effects of the bug was to backup more directories
  16. than strictly necessary, which was a good thing because it
  17. mooted another bug that cause some dirs that should be backed up to get 
  18. skipped.
  19.  
  20. so if you use my first fix you probably want to use something
  21. like the second fix i am sending you in this note also.
  22.  
  23. here is my first patch:
  24.  
  25. ------- gnu.c -------
  26. 162,172c162,170
  27. <         strp=buf;
  28. <         dev=atol(strp);
  29. <         while(isdigit(*strp))
  30. <             strp++;
  31. <         ino=atol(strp);
  32. <         while(isspace(*strp))
  33. <             strp++;
  34. <         while(isdigit(*strp))
  35. <             strp++;
  36. <         strp++;
  37. <         add_dir(un_quote_string(strp),dev,ino,(char *)0);
  38. ---
  39. >         {
  40. >             unsigned long t,u;
  41. >             char scratch[1000];
  42. > 
  43. >             sscanf(buf, "%lu %lu %s", &t, &u, scratch);
  44. >             dev=t;
  45. >             ino=u;
  46. >         add_dir(un_quote_string(scratch),dev,ino,(char *)0);
  47. >         }
  48.  
  49. with just this patch in place the following test
  50. winds up with the file d/e/hi not being restored.
  51.  
  52. # set up the test data
  53.  
  54. rm -r d
  55. mkdir d
  56. mkdir d/d
  57. mkdir d/e
  58. echo hi > d/d/hi
  59.  
  60. # do a level 0 backup
  61.  
  62. rm date0
  63. tar -cvf tar0 -g date0 d
  64.  
  65. sleep 2
  66. mv d/d/hi d/e
  67.  
  68. # do a level 1 backup
  69. cp date0 date1
  70. tar -cvf tar1 -g date1 d
  71.  
  72. # now simulate loosing it all
  73. rm -r d
  74.  
  75. # do the restore
  76.  
  77. tar -xvf tar0 -G
  78.  
  79. tar -xvf tar1 -G
  80.  
  81.  
  82.  
  83. #end of test case
  84.  
  85. after this test case is run d/e/hi does not exist
  86. and wish i did.
  87.  
  88. i think the problem is that tar is not backup up the contents
  89. of the dir d/e when it is doing the level 1 backup:
  90. tar -cvf tar1 -g date1 d
  91. + tar -cvf tar1 -g date1 d
  92. d/
  93. d/d/
  94. d/e/
  95.  
  96.  
  97. tar-1.11.1 DOES backup the contents
  98. of the dir d/e, but i think for the wrong reasons, because
  99. it backs up the contents of ALL the directories.
  100.  
  101. tar -cvf tar1 -g date1 d
  102. tar: directory d/d has been renamed.
  103. tar: directory d/e has been renamed.
  104. d/
  105. d/d/
  106. d/e/
  107. d/e/hi
  108.  
  109. you can see the problem with the following test case with tar-1.11.1
  110.  
  111. # set up the test data
  112.  
  113. rm -r d
  114. mkdir d
  115. mkdir d/d
  116. mkdir d/e
  117. mkdir d/f
  118. echo hi > d/d/hi
  119. echo goodbye > d/f/bye
  120.  
  121. # do a level 0 backup
  122.  
  123. rm date0
  124. tar -cvf tar0 -g date0 d
  125.  
  126. sleep 2
  127. mv d/d/hi d/e
  128.  
  129. # do a level 1 backup
  130. cp date0 date1
  131. tar -cvf tar1 -g date1 d
  132.  
  133. # now simulate loosing it all
  134. rm -r d
  135.  
  136. # do the restore
  137.  
  138. tar -xvf tar0 -G
  139.  
  140. tar -xvf tar1 -G
  141.  
  142.  
  143.  
  144.  
  145. # end of test case
  146.  
  147. the problem is that dir d/f gets backed up in the level 1 backup even though
  148. nothing about it has changed:
  149.  
  150. tar -cvf tar1 -g date1 d
  151. tar: directory d/d has been renamed.
  152. tar: directory d/e has been renamed.
  153. tar: directory d/f has been renamed.
  154. d/
  155. d/d/
  156. d/e/
  157. d/f/
  158. d/e/hi
  159. d/f/bye
  160.  
  161. i noticed that when tar is deciding to incrementally dump a dir (or not
  162. it only looks at the inode number and device, not at the date.
  163. here is my patch to address this.  (along with my first patch,
  164. apply it to tar-1.11.1/gnu.c)  note that i don't really know what
  165. is going on here, so please look at it carefully before using it.
  166.  
  167.  
  168. *** gnu.c.orig    Sun Nov  8 17:53:14 1992
  169. --- gnu.c    Sun Nov  8 17:53:27 1992
  170. ***************
  171. *** 159,176 ****
  172.           strp= &buf[strlen(buf)];
  173.           if(strp[-1]=='\n')
  174.               strp[-1]='\0';
  175. !         strp=buf;
  176. !         dev=atol(strp);
  177. !         while(isdigit(*strp))
  178. !             strp++;
  179. !         ino=atol(strp);
  180. !         while(isspace(*strp))
  181. !             strp++;
  182. !         while(isdigit(*strp))
  183. !             strp++;
  184. !         strp++;
  185. !         add_dir(un_quote_string(strp),dev,ino,(char *)0);
  186. !     }
  187.       fclose(fp);
  188.   }
  189.   
  190. --- 159,174 ----
  191.           strp= &buf[strlen(buf)];
  192.           if(strp[-1]=='\n')
  193.               strp[-1]='\0';
  194. !         {
  195. !           unsigned long t,u;
  196. !           char scratch[1000];
  197. !        
  198. !           sscanf(buf, "%lu %lu %s", &t, &u, scratch);
  199. !           dev=t;
  200. !           ino=u;
  201. !           add_dir(un_quote_string(scratch),dev,ino,(char *)0);
  202. !         }
  203. !           }
  204.       fclose(fp);
  205.   }
  206.   
  207. ***************
  208. *** 377,391 ****
  209.                       if(   dp->dev!=hs.st_dev
  210.                           || dp->ino!=hs.st_ino) {
  211.                              if(f_verbose)
  212. !                             msg("directory %s has been renamed.",namebuf);
  213.                           dp->allnew=1;
  214.                           dp->dev=hs.st_dev;
  215.                           dp->ino=hs.st_ino;
  216.                       }
  217.                       dp->dir_text="";
  218.                   } else {
  219.                       if(f_verbose)
  220. !                         msg("Directory %s is new",namebuf);
  221.                       add_dir(namebuf,hs.st_dev,hs.st_ino,"");
  222.                       dp=get_dir(namebuf);
  223.                       dp->allnew=1;
  224. --- 375,398 ----
  225.                       if(   dp->dev!=hs.st_dev
  226.                           || dp->ino!=hs.st_ino) {
  227.                              if(f_verbose)
  228. !                           msg("%s is a new dir reusing an old dir name.",namebuf);
  229.                           dp->allnew=1;
  230.                           dp->dev=hs.st_dev;
  231.                           dp->ino=hs.st_ino;
  232. + 
  233. +                           } else if (f_new_files &&
  234. +                              new_time>hs.st_mtime &&
  235. +                              new_time>hs.st_mtime &&
  236. +                              (f_new_files>1 || new_time>hs.st_ctime)) {
  237. +                         if (f_verbose) {
  238. +                           msg("dir %s has been changed since previous .",namebuf);
  239. +                         }
  240. +                         dp->allnew=1;
  241.                       }
  242.                       dp->dir_text="";
  243.                   } else {
  244.                       if(f_verbose)
  245. !                       msg("%s is a new dir name",namebuf);
  246.                       add_dir(namebuf,hs.st_dev,hs.st_ino,"");
  247.                       dp=get_dir(namebuf);
  248.                       dp->allnew=1;
  249.  
  250.