LEAP support PATCH for knm

[Helmut Schaa]

Am Freitag, 25. Mai 2007 schrieb Stefan Skotte:
> Hi all,
>
> Here's the LEAP patch for the latest svn checkout.
>
> A few notes about the patch:
>
> (1) I've tried to keep the patch as similar to the current code (get/set
> methods etc.), please change if needed, and do tell about any wrong
> doings :)

Looks promising.

> (2) The LEAP implementation for NM 0.6.5 seems to only accept the
> IEEE8021X/WPA-EAP as char/string arguments, unlike the ones for
> PEAP/TLS/TTLS which uses defines in the NetworkManager.h include file
> (shouldn't it be like this for LEAP too?).

I think so too. That would be a good question for the NetworkManager list ;-)

> (3) When adding the LEAP options in the WPA Enterprise combo, I
> initially wanted to redefine the GUI, since it didn't seem all that
> obvious to select WPA Enterprise, while WEP is actually supported for
> LEAP also (i've tested this). In the end I just added two more options
> for the combo though: http://enhance-it.dk/knm-options.png
>
> I guess a redesign of the GUI might be up to a later discussion :)
> (sign me up)

We could do it the same way as nm-applet does.They have one more entry 
("LEAP") in the combobox providing WEP, WPA-PSK and WPA-EAP.

> (4) We need someone to test LEAP/WPA-EAP also (I've just set the
> editboxes to the likes of PEAP now - is any more needed?)

I have no possibilty to try :-(

> (5) My qt3-dev is newer than the one initially used for designing the
> .ui files, so you can skip the dimensions/version info from the patch in
> cryptowidget.ui

Ok.

Regards
Helmut