On Fri, Aug 14, 2009 at 02:41:17PM -0500, Shawn Walker wrote: > [email protected] wrote: >> Folks, >> This is a small code review that has the transport set the mtime on >> downloaded files, if the server provides a last-modified header, or >> similar information. >> >> The change is pretty small. >> >> http://cr.opensolaris.org/~johansen/webrev-10685/ > > Do you need to set filetime to -1 around line 515 of engine.py as well?
Thanks for the quick review. I've published a new webrev that makes sure we apply this check only when we're performing operations that write files out to disk. It makes double-sure not to do this for streaming file objects. http://cr.opensolaris.org/~johansen/webrev-10685-2/ Thanks, -j _______________________________________________ pkg-discuss mailing list [email protected] http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
