Hi Jean -
I hadn't given you feedback. Wanted to let you know....
thanks,
ginnie
On 11/03/10 09:47, jean.mccormack wrote:
svr4.py:
line 80 : Don't use is
gone
line 157: I don't think cancel takes timeout anymore.
removed
ips.py:
line 76: I think we need to change this to the external URL
done
line 197: Do you want to try/except around this for version skew as done
in get_ips_api_inst?
Alok said that he just wanted the version mismatch between entire and
the system logged, but not to interrupt the process.
line 305: Please put a space at the end of the string.
line 325: ditto
I actually enhanced this further with Sue's feedback. She suggested a
colon space.
line 667: Should this line be deleted?
I've deleted it.
line 747& 748: Can this be one name?
yes.
cpio.py:
line 172 & 173: Do we need both lines?
per our conversation, I'm removing that log_progress function.
Jean
On 11/ 2/10 06:58 PM, Ginnie Wray wrote:
I've incorporate the code review comments from the first round of
code reviews and would like follow up feedback. The code is posted at:
http://cr.opensolaris.org/~ginnie/transfer/
I've posted the diffs from the first code review, so I would like
to clarify: Keith suggested changing the names of the files from
transfer_*.py to just *.py, which I've done. That, however, made it
difficult to post the diffs from the original transfer_*.py files.
What I've done is included the new file names, and then also posted
the diffs from the old files. So, there is no need to review any of
the files marked as new. They are identical to the transfer_*.py file.
My thought was that it would make it a little easier to review the diffs.
Let me know if you have any questions or comments. I would like to
push on Nov. 16th, so if I could get feedback by the end of the week,
I would appreciate it.
thanks,
ginnie
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
