[email protected] wrote:
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/

Seems right,
--
Shawn Walker
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

Reply via email to