Tom Mueller wrote:
Still looking for a review for this. Also, I added one additional bug
related to running on Windows:
10127 transport module uses os.statvfs which isn't available on Windows
src/modules/client/transport/engine.py:
line 37: s/'/"/
src/modules/client/api.py:
lines 66-67: s/'/"
src/setup.py:
line 475: s/'/"/ (yes, i know it wasn't this way originally but we
generally don't use ', only " for strings)
lines 517, 565: s/'/"/
src/tests/api/t_p5i.py:
lines 181-182, 200-201: s/'/"/
Otherwise, modulo Krister's comments, this looks fine.
Cheers,
--
Shawn Walker
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss