Better but not perfect. renamed existing icla added second icla2/icla.pdf and icla.pdf.asc updated iclas.txt with changed email address
failed while updating public name in LDAP. of course there is no LDAP entry so this step should not be done. Similarly, it should not attempt to change the LDAP entry for email. And the ack message should omit: You can change your password using https://id.apache.org . So this icla is processed (manually sent email) but still needs a tweak. Thanks for hopping on this, Sam Craig > On Feb 11, 2018, at 7:05 AM, Sam Ruby <[email protected]> wrote: > > On Sun, Feb 11, 2018 at 3:51 AM, Craig Russell <[email protected]> wrote: >> Filing the second icla for Kamesh I encountered an exception. >> >> The original icla is a simple <name>.pdf. The second icla is signed with >> icla.pdf and icla.pdf.asc. I suspect this is the problem. >> >> I can do this manually but I don't think there is a big rush. >> >> Craig >> >> #<NoMethodError: undefined method `legal_name' for nil:NilClass> > > The problem is that the ICLA wasn't found, so extracting the > legal_name from the ICLA fails. > > The ICLA wasn't found as it was attempted to be located by id, and > this individual is not a committer. > > I made a fix to match based on original email address too, and > improved the error message when it fails: > > https://github.com/apache/whimsy/commit/ecbacc3fa3e4be15bcf61e0de2d541e700f8fbb2 > > I've verified that this fixes the problem reported, but haven't tested > that the actual commit will succeed. > > - Sam Ruby Craig L Russell Secretary, Apache Software Foundation [email protected] http://db.apache.org/jdo
