LGTM, see one minor comment. https://codereview.appspot.com/11039043/diff/1/src/client/fix-chain.py File src/client/fix-chain.py (right): https://codereview.appspot.com/11039043/diff/1/src/client/fix-chain.py#newcod... src/client/fix-chain.py:116: print "Skipping issuer URL:", str(loc) No need to call str - print does that implicitly: http://docs.python.org/2/reference/datamodel.html#basic-customization
Pushed. https://codereview.appspot.com/11039043/diff/1/src/client/fix-chain.py File src/client/fix-chain.py (right): https://codereview.appspot.com/11039043/diff/1/src/client/fix-chain.py#newcod... src/client/fix-chain.py:116: print "Skipping issuer URL:", str(loc) On 2013/07/09 13:20:07, Eran wrote: > No need to call str - print does that implicitly: > http://docs.python.org/2/reference/datamodel.html#basic-customization Done.