https://bugzilla.novell.com/show_bug.cgi?id=339027#c1 --- Comment #1 from Jiří­ Suchomel 2007-11-05 02:11:53 MST --- I do not feel this is a correct way to go. Of course it will work (now), but the problem is with the general approach. DSL::UsesDevice is a high-level function, that doesn't require from caller to know how such check is performed. DSL module maintainer could change that in future and caller (inst_auth in this case) should not care about it, because it relies on the API. The same for NetworkDevices::List : it doesn't have a sense to duplicate a code for searching DSL devices in inst_auth (or at any other place) and change it on more places when the logic changes. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=339027#c3 --- Comment #3 from Jiří­ Suchomel 2007-11-05 02:21:41 MST --- Yes, I think the correct way is providing the API in the more general module. On the other side, I do not see what is wrong on yast2-network dependency, especially when it affect only the installation. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=339027#c5 michal zugec changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEEDINFO |NEW Info Provider|mzugec@novell.com | --- Comment #5 from michal zugec 2007-11-07 02:20:15 MST --- I don't understant what's wrong on calling NetworkDevices::List("dsl")? It means "give me list of dsl network devices" and it _IS_ high level. NetworkDevices::AllDSLDevices() is nice, but should I write such functions for all arc|ath|bnep|ci|ctc|dummy|bond|escon|eth|fddi|ficon|hsi|qeth|lcs|iucv|myri|tr|usb|wlan|xp|ppp|modem|isdn|ippp? -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=339027#c7 michal zugec changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEEDINFO |NEW Info Provider|mzugec@novell.com | --- Comment #7 from michal zugec 2007-11-07 13:05:27 MST ---

From version yast2-2.16.9 you can use function NetworkDevices::ListDevicesExcept("dialup")

-- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=339027#c8 Jiří­ Suchomel changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |RESOLVED Resolution| |FIXED --- Comment #8 from Jiří­ Suchomel 2007-11-08 03:15:17 MST --- yast2-users-2.16.2 -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.