On 24 November 2015 at 04:39, Craig L Russell <[email protected]> wrote:
>
>> On Nov 23, 2015, at 2:34 PM, sebb <[email protected]> wrote:
>>
>> On 23 November 2015 at 20:54, Craig L Russell <[email protected]> 
>> wrote:
>>>
>>>> On Nov 23, 2015, at 9:38 AM, sebb <[email protected]> wrote:
>>>>
>>>> On 22 November 2015 at 16:14, Craig L Russell <[email protected]> 
>>>> wrote:
>>>>>
>>>>>> On Nov 22, 2015, at 7:22 AM, sebb <[email protected]> wrote:
>>>>>>
>>>>>> On 21 November 2015 at 03:50, Craig L Russell <[email protected]> 
>>>>>> wrote:
>>>>>>>
>>>>>>>> On Nov 20, 2015, at 7:17 PM, Sam Ruby <[email protected]> wrote:
>>>>>>>>
>>>>>>>> On Fri, Nov 20, 2015 at 10:10 PM, Craig L Russell
>>>>>>>> <[email protected]> wrote:
>>>>>>>>> I noticed during the recent work that the email address in iclas.txt 
>>>>>>>>> is out of date with reality. At least a couple of addresses are in 
>>>>>>>>> the sun.com domain, clearly not current.
>>>>>>>>>
>>>>>>>>> How could I send email to the address in iclas.txt and also to all 
>>>>>>>>> addresses in mail aliases? Something like:
>>>>>>>>>
>>>>>>>>> to: [email protected] cc: 
>>>>>>>>> [email protected],[email protected],[email protected] reflecting 
>>>>>>>>> both the iclas.txt address plus all of the mail aliases.
>>>>>>>>
>>>>>>>> First, this is not possible for entries with 'notinavail’,
>>>>>>>
>>>>>>> not really needed, since notinavail entries can simply be removed after 
>>>>>>> sending email once.
>>>>>>
>>>>>> I'm not sure that's true - see recent e-mail from Sam
>>>>>>
>>>>>> https://mail-search.apache.org/members/private-arch/board/201511.mbox/%3ccafg6u8e+v_j_r8ccmfs5vqahssuf_vngrvqxx5qqsh0sowv...@mail.gmail.com%3e
>>>>>>
>>>>>> Don't we need to keep a record of all ICLAs submitted?
>>>>>
>>>>> Yes. I’m no longer thinking that we should remove the iclas.txt entry, 
>>>>> but need to modify the public name to indicate that the icla went missing.
>>>>
>>>> "modify the public name"
>>>>
>>>> Not sure that is a good idea.
>>>> Surely the place to record missing ICLA files is the 5th field which
>>>> has a suffix (after the semi-colon) for recording the ICLA file name.
>>>
>>> The problem is that a PMC just looks at 
>>> http://people.apache.org/committer-index.html and sees that an ICLA is on 
>>> file. But these 33 entries have no ICLA. So if the PMC request an account, 
>>> root would have to research whether there is actually an icla. I’m trying 
>>> to avoid that.
>>>
>>> My proposal: Instead of e.g. Charles McLouth showing up, it would be 
>>> Charles McLouth (no ICLA) and surely there is no need for research by root.
>>>
>>> The alternative to avoid the above is to remove the iclas.txt entry, but 
>>> that has its own problems.
>>
>> Or fix the people.a.o app to take note of another indicator in the file.
>
> Ok. The bad entries with notinavail come in three flavors:
>
> no icla stub
> icla stub but no document in documents/iclas/stub

Not sure how one can detect that easily - or do you mean that the file
name field is missing?

> comment  (+=r9120) or  (+=r9154) or (page 3 missing)
>
> If the acreq tool can be updated to deny these cases, then there is nothing 
> more needed to flag these entries.

Seems to me that an unambiguous way to fix these would be to remove or
replace the 'Signed CLA' text for cases that don't have valid CLAs.

> Craig
>
>>
>>> Craig
>>>
>>>>
>>>>> There are only 33 cases of missing iclas for notinavail, so the tool will 
>>>>> simply be full of those folks that we cannot locate. What would be bad is 
>>>>> for these folks to fail to respond to the request for a replacement, and 
>>>>> for a PMC to then offer commit privileges.
>>>>>
>>>>
>>>> " the tool will simply be full ..."
>>>>
>>>> Not sure I understand what tool is being referenced here.
>>>>
>>>>> Craig
>>>>>>
>>>>>>
>>>>>>>> but here
>>>>>>>> are a few things to explore.  The way I would recommend you explore
>>>>>>>> these values is to type "irb -r whimsy/asf" from the command line and
>>>>>>>> then enter the expressions listed below.
>>>>>>>
>>>>>>> I’ll give these a shot and let you know.
>>>>>>>
>>>>>>> Craig
>>>>>>>>
>>>>>>>> Here is how to find all email addresses:
>>>>>>>>
>>>>>>>> ASF::Person.find('clr').all_mail
>>>>>>>>
>>>>>>>> Here is how to remove the 'to' address:
>>>>>>>>
>>>>>>>> ASF::Person.find('clr').all_mail - ['[email protected]']
>>>>>>>>
>>>>>>>> Here is how to return the results as a comma separated string:
>>>>>>>>
>>>>>>>> (ASF::Person.find('clr').all_mail - 
>>>>>>>> ['[email protected]']).join(',')
>>>>>>>>
>>>>>>>>> Craig
>>>>>>>>>
>>>>>>>>> Craig L Russell
>>>>>>>>> Architect, Oracle
>>>>>>>>> http://db.apache.org/jdo
>>>>>>>>> 408 276-5638 mailto:[email protected]
>>>>>>>>> P.S. A good JDO? O, Gasp!
>>>>>>>>
>>>>>>>> - Sam Ruby
>>>>>>>
>>>>>>> Craig L Russell
>>>>>>> Architect, Oracle
>>>>>>> http://db.apache.org/jdo
>>>>>>> 408 276-5638 mailto:[email protected]
>>>>>>> P.S. A good JDO? O, Gasp!
>>>>>>>
>>>>>
>>>>> Craig L Russell
>>>>> Architect, Oracle
>>>>> http://db.apache.org/jdo
>>>>> 408 276-5638 mailto:[email protected]
>>>>> P.S. A good JDO? O, Gasp!
>>>
>>> Craig L Russell
>>> Architect, Oracle
>>> http://db.apache.org/jdo
>>> 408 276-5638 mailto:[email protected]
>>> P.S. A good JDO? O, Gasp!
>>>
>
> Craig L Russell
> Architect, Oracle
> http://db.apache.org/jdo
> 408 276-5638 mailto:[email protected]
> P.S. A good JDO? O, Gasp!
>

Reply via email to