[Yum-devel] Pylint errors

[Mike Bonnet]

On 04/30/2010 02:01 PM, Seth Vidal wrote:
> 
> 
> On Fri, 30 Apr 2010, Seth Vidal wrote:
> 
>>
>>
>> On Wed, 28 Apr 2010, Tim Lauridsen wrote:
>>
>>> Hi, i have fixed most of the pylint errors/warning in current yum,
>>> but i not sure about how to
>>> solve this one.
>>>
>>>
>>> ************* Module rpmUtils.arch
>>> W0109: 14: Duplicate key 'sparc64v' in dictionary
>>>
>>> there is 2 indentical key in arch dictionary, but what is the right
>>> solution ?
>>>
>>>     # sparc
>>>     "sparc64v": "sparc64",
>>>     "sparc64v": "sparcv9v",
>>>
>>
>> I think this fall through is on purpose, but I know who would know.
>> I'll ask spot or dgilmore.
> 
> I asked dgilmore he said:
> [13:59] <dgilmore> skvidal: there both right
> [13:59] <tibbs> Hmm, OK.
> [13:59] <skvidal> dgilmore: seriously?
> [14:00] <dgilmore> skvidal: yep
> [14:00] <skvidal> ooooooookie doke
> [14:00] <dgilmore> skvidal: sparcv9 -> sparcv9v ->sparc64 ->sparc64v
> [14:00] <dgilmore> skvidal: sparcv9 -> sparc64
> [14:00] <dgilmore> depending on the hardware
> 
> so - leave them as is - so we can properly traverse them.

Isn't the second entry just going to overwrite the first?

>>> test = {"sparc64v": "sparc64", "sparc64v": "sparcv9v"}
>>> test
{'sparc64v': 'sparcv9v'}

Looks like you need to pick one.