[Commotion-admin] [serval-crypto] added option to specify keyring for serval sign (28e6b56)

hawkinswnaf notifications at github.com
Fri Nov 8 07:29:47 UTC 2013


This looks really good, @gradyoti . The only possible suggestion I have is that you check your indentation and formatting (specifically the else statement on line 36 in serval-sign.c) so that it matches @danstaples 's style.

I can't wait to get these tested and merged! :-) 

---
Reply to this email directly or view it on GitHub:
https://github.com/opentechinstitute/serval-crypto/commit/28e6b5690975edfe9d54ee04edcafe0cffc2c3dc#commitcomment-4545632
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.chambana.net/pipermail/commotion-admin/attachments/20131107/0df39edd/attachment.html>


More information about the Commotion-admin mailing list