Indicator-loader uses incorrect serialisation for indicator-network menu icons

[Jussi Pakkanen]

Currently indicator-network exports icons to show like this (gdbus dump):

'icons': <
          [
           <('themed', <['gsm-3g-full']>)>,
           <('themed', <['nm-signal-50-secure']>)>
          ]
         >,

That is they are an array of variants (that happen to be tuples). However this seems to be incorrect. At least if you use the indicator-loader it chokes on these and refuses to draw them. The form that it accepts looks like this:

'icons': <
          [
           ('themed', <['gsm-3g-full']>),
           ('themed', <['simcard-locked']>),
           ('themed', <['nm-signal-75-secure']>)
          ]
         >

That is, an array of tuple variants. One of these two gets it wrong and since indicator-loader is the older of the two, I'm guessing that is the correct one. Feel free to reassign as appropriate if you feel that Unity8 is doing the right thing here.

[Nick Dedekind (nick-dedekind)]

the indicator-loader is supplied by libindicator3-tools.

[Ted Gould (ted)]

indicator loader is correct here, by the spec:

https://wiki.ubuntu.com/SystemComponents#Root_Item