Comment 4 for bug 573471

Revision history for this message
Parth Malwankar (parthm) wrote : Re: [Bug 573471] Re: KeyError: 'canonical' using scan-merge-proposals when run by non-canonical user

On Mon, May 3, 2010 at 8:24 AM, Martin Pool <email address hidden> wrote:
> On 3 May 2010 12:39, Parth Malwankar <email address hidden> wrote:
>> The docstring in the script does mention.
>>
>>  * this script will only work when run by members of ~canonical,
>>   because it's a private team
>>
>> I thought that might have caused this wasn't sure. I suppose the
>> easiest fix would be to just give a clearer error message.
>>

> Perhaps if you're not allowed to see that team (ie you get a KeyError)
> you should continue without it, but print a message along with the
> list at the end.
>

Sounds like a good solution to me.

Regards,
Parth