Alexander Eremin wrote: > On Wed, 2009-10-07 at 17:16 -0700, Karen Tung wrote: > >> line 109: should this os.access() check be done for the destination >> file, to >> make sure that the cpio command really did succeed? >> > I'm not shure that this is required, cpio copies all include symlinks, > after this we need to compress only files, and if else some file will > not be copied, it will be putting in destination directory by > fiocompress.. > > > Hi Alexander,
I read the code again. The check in line 109 was added by your change. Why did you add the check? It actually seems unnecessary. Thanks, --Karen