On 5/16/2013 2:05 PM, Abhishek Koneru wrote: Just one thing, in pkimanifest.py and pkiparser.py the original exception is discarded and new exception is raised. In general it would be better to re-throw the original exception, unless there's a specific reason (e.g. to hide low level details). ACK with above issue fixed. This is for future enhancement. In places where the sys.exit() was removed, the code logs an error message, then it throws an exception containing the same error message. It would be better to remove the logging code from these locations, then log the error in the main program. -- Endi S. Dewata