<div dir="ltr"><div dir="ltr">> They can all be part of one series. Please add the makefiles in the last</div>patch of the series.<div>Acknowledged<br><div dir="ltr"><br></div><div dir="ltr">> Thank you for the patch.</div><div>You're welcome!</div><div dir="ltr"><br></div><div dir="ltr">> I think you meant bug 156.<br></div><div>Correct. Fixed</div><div><br></div><div>>I don't lije this much. What other options do we have on Android to<br>verify signatures ?<br></div></div><div>We have boringssl, Google's AOSP-compatible OpenSSL fork, which guarantees neither API nor ABI compatibility. Technically this does work, but disabling this seems less likely to break and reduces complexity, plus why not give people the option? I'd like to understand your preference here better-- why wouldn't we want to let users choose to disable module isolation?</div><div><br></div><div>I'll leave this as the last commit in the chain so we can discuss. Excluding the option looks like it should still work for the time being, so that's what the first patches will assume.</div><div><br></div><div>> The addition of LOG() calls may need to be further split in multiple patches<br></div><div>I like my logging messages :-). On a more serious note, I think these are useful, but I'd like to read your thoughts too. I'll be happy to split these up further per your comments if that's what you'd like.</div><div><br></div><div>> In v2, could you also please send patches inline, not as attachments ? I<br>> highly recommend usage of git-send-email (with a cover letter) to avoid<br>> lots of potential issues with various mail clients.</div><div>Sounds good-- I'll try this out for v2.</div></div>