On Thu, Feb 04, 2010 at 11:37:57AM -0600, George McCollister wrote: > Sumit, > > I've attached a new version of the patch with the changes you suggested. > > just some minor nitpicks: > > > > - please reformat the commit message to use multiple lines not longer > > than 80 characters > > > Fixed. > > - can you rename SSS_SOURCES to SSS_CRYPT_SOURCES > > > Thats what I meant to do in the first place. Fixed. > > - can you update the comment at the beginning of crypto_sha512crypt.c so > > that it is clear that this file uses calls from libcrypto and not from > > libnss3 > > > Done. > > - please fix the following whitespace issues: > > > > Applying: Added option to use libcrypto instead of NSS. > > /tmp/sssd/.git/rebase-apply/patch:117: trailing whitespace. > > [ --enable-crypto Use OpenSSL crypto instead of NSS], > > /tmp/sssd/.git/rebase-apply/patch:271: trailing whitespace. > > > > /tmp/sssd/.git/rebase-apply/patch:292: trailing whitespace. > > * The final result will be added to the first context. */ > > /tmp/sssd/.git/rebase-apply/patch:147: new blank line at EOF. > > + > > warning: 4 lines add whitespace errors. > > > > > I ran the last version of the patch through git apply and I only > received 3 errors. I fixed everything I could find and now git apply is > giving me no errors. > > Cheers, > George McCollister >
ACK bye, Sumit _______________________________________________ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://fedorahosted.org/mailman/listinfo/sssd-devel