Installations Check preuft nicht gegen Net::LDAP

[wulf@…]

2746ccd0d46 Date: Wed Mar 14 13:46:46 2012 +0100

eigentlich sollte das in SL::InstallationCheck::check_for_conditional_dependencies passieren. Hier wird aber gegen die Konfiguration geprueft und nur die Abhaengigkeit erwartet, wenn auch LDAP als Auth-Modul ausgewaehlt wurde. Die Konfiguration liegt in $::lx_office_conf beim Konsolenaufruf aber nicht vor. Das will man auch nicht, da scripts/installation_check.pl ja nicht schon selbst Abhaengigkeiten enthalten soll (Config::Std -> Henne - Ei).

Loesungsansaetze:

1. man nimmt { 'name' => 'Net::LDAP', 'url' => '​http://search.cpan.org/~gbarr/', debian => 'libnet-ldap-perl' } in required auf. Waere mein Favorit, da in der Regel die Konfiguration nicht abgeschlossen ist, wenn man den Installationscheck durchfuehrt. Ich finde man kann damit leben, dass damit Net:LDAP zur zwingenden Abhaengigkeit wird, auch wenn es nicht unbedingt benoetigt wird.

2. man verwendet perl Bordmittel (!= Config::Std) um lx_office.conf bezueglich des Auth-Moduls auszuwerten.

3. man liest die Config ein, nach dem man gegen Config::Std geprueft hat und holt dann den Test gegen Net::LDAP nach.

4. Deine Idee hier ...

Variante a) gefällt mir nicht. Hast du mal Bugzilla installiert? Hast du da die Liste der optionalen Pakete gesehen? Stell dir vor, die wären alle erzwungen -- dann dürfte ich mir die fetten Pakete für die Behandlung asiatischer Sprachen installieren, müsste drei Arten von externem Interface unterstützen können, auch wenn ich die alle nicht nutzen will etc. Da will ich Kivitendo nicht hin haben.

b) ist OK. Hier genügt ein echt simpler Parser für 99% aller Fälle:

1. Suche nach [authentication]
2. Suche nach \s*module\s*=\s*([^\s]+), solange nicht die nächste ^[section gefunden wird

c) Mag ich wieder nicht so, weil... Hmm... wäre auch OK. Mir gefällt b) trotzdem besser, weil dann direkt nach einem Aufruf schon zu sehen ist, was die Benutzerin alles installieren muss.

Sinnvoll wäre auch gleich das Paket

"libio-socket-ssl-perl"

mitinstallieren zu lassen, um verschlüsselte Verbindungen zum LDAP-Server aufbauen zu können. Die Option wird ja in der lx_office.conf zumindest angeboten.

Dem stimme ich zu.

mech, im irc liest keiner.

a) was mosu sagt
b) gefällt mir nicht. gibt nen grund dass man dafür ein standard modul benutzt
c) gefällt mir eher. das einzig doofe ist halt, dass versteckte abhängigkeiten dazukommen können, und nicht sofot erichtlich wird, was alles benötigt wird.

IO::Socket::SSL - pack ich gleich in die abhängigkeiten, das ist sinnvoll.

Done in e3ff1eb