This is the mail archive of the
cygwin-apps
mailing list for the Cygwin project.
Re: [PATCH 4/6] Report failure extracting a file from package to the user
On Apr 8 15:43, Jon TURNEY wrote:
> At the moment, all errors in archive::extract_file() are assumed to be due
> to a failure to open the file for writing due to it being opened by another
> process.
>
> Distinguish when the error is due to an inability to read the file from the
> source archive, and report that.
>
> 2011-04-08 Jon TURNEY <...>
>
> * install.cc (extract_replace_on_reboot): New function containg code
> extracted from...
> (installOne): Report read errors differently to write errors
> * archive.cc (extract_file): Distinguish read errors from write errors
This loooks good to me. Just a question...
> + /* XXX FIXME: prefix may not be / for in use files -
> + * although it most likely is
> + * - we need a io method to get win32 paths
> + * or to wrap this system call
This comment doesn't make any sense to me anymore. The incoming path
should already be a full Win32 path. The function wouldn't work without
it and AFAIK the paths from the tar file are converted to full Win32
paths anyway. There's also no reason to wrap this system call any further.
Bottom line is, just drop the comment, please.
Thanks,
Corinna
--
Corinna Vinschen Please, send mails regarding Cygwin to
Cygwin Project Co-Leader cygwin AT cygwin DOT com
Red Hat