LEAP support PATCH for knm

[Helmut Schaa]

Ping

Helmut

Am Donnerstag, 31. Mai 2007 schrieb 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
> _______________________________________________
> kde-networkmanager mailing list
> kde-networkmanager at kde.org
> https://mail.kde.org/mailman/listinfo/kde-networkmanager



-- 
SUSE LINUX Products GmbH, GF: Markus Rex, HRB 16746 (AG Nürnberg)