Univention Bugzilla – Bug 22434
Nicht vorhandene Beschreibung als Warning
Last modified: 2012-12-12 21:07:39 CET
Eine nicht vorhandene UCR Variablenbeschreibung sollte nur Warning sein.
(In reply to comment #0) > Eine nicht vorhandene UCR Variablenbeschreibung sollte nur Warning sein. Da hatte ich eine Fehlermeldung von ucslint falsch gelesen. Wenn ich es richtig sehe, dann wird derzeit nicht geprüft, ob eine Beschreibung vorhanden ist. Das sollte umgesetzt werden.
Die Prüfung sollte ausserdem prüfen, ob im Template überhaupt eine UCR-Variable referenziert wird. Es gibt eine Reihe von UCR-Templates, die nicht auf UCR-Variablen zugreifen, z.B. das Template für atftpd aus univention-tftp.
(In reply to comment #1) > Da hatte ich eine Fehlermeldung von ucslint falsch gelesen. Wenn ich es richtig > sehe, dann wird derzeit nicht geprüft, ob eine Beschreibung vorhanden ist. Das > sollte umgesetzt werden. Das ist nicht sinnvoll umsetzbar: "ldap/base" wird z.B. in univention-base-files zur beschrieben, aber von sehr vielen anderen Paketen mitbenutzt, ohne das diese die Variable nochmals beschreiben. Von daher ist das eine paketübergreifende Eigenschaft, die ucslint so nicht abdecken kann. Der Code dafür existiert jetzt bereits in ucslint, ist aber deaktiviert. (In reply to comment #2) > Die Prüfung sollte ausserdem prüfen, ob im Template überhaupt eine UCR-Variable > referenziert wird. Es gibt eine Reihe von UCR-Templates, die nicht auf > UCR-Variablen zugreifen, z.B. das Template für atftpd aus univention-tftp. Auch das ist teilweise so gewollt, wenn z.B. ein Init-Skript durch eine Dummy-Variante ersetzt wird; Durch die Verwendung von UCR besteht dann weiterhin die Möglichkeit, daß ein Benutzer das über die Vorlagen anpassen kann. Trotzdem wurde das jetzt umgesetzt und sollte bei absichtlicher Verwendung im vorstehenden Sinne dann per ucslint.overrides explizit als als gut geheißen werden. svn34414,34415, ucslint_3.0.0-1
(In reply to comment #3) > (In reply to comment #2) > > Die Prüfung sollte ausserdem prüfen, ob im Template überhaupt eine UCR-Variable > > referenziert wird. Es gibt eine Reihe von UCR-Templates, die nicht auf > > UCR-Variablen zugreifen, z.B. das Template für atftpd aus univention-tftp. > > Auch das ist teilweise so gewollt, wenn z.B. ein Init-Skript durch eine > Dummy-Variante ersetzt wird; Durch die Verwendung von UCR besteht dann > weiterhin die Möglichkeit, daß ein Benutzer das über die Vorlagen anpassen > kann. > Trotzdem wurde das jetzt umgesetzt und sollte bei absichtlicher Verwendung im > vorstehenden Sinne dann per ucslint.overrides explizit als als gut geheißen > werden. > > svn34414,34415, ucslint_3.0.0-1 Die Prüfung scheint nicht immer alle Dateien zu berücksichtigen. So wird eine fehlerhafte Datei "file1" unterhalb von "conffiles" zwar bemängelt, eine identische Datei "file2" unterhalb von "conffiles/etc" hingegen nicht.
(In reply to comment #4) > Die Prüfung scheint nicht immer alle Dateien zu berücksichtigen. So wird eine > fehlerhafte Datei "file1" unterhalb von "conffiles" zwar bemängelt, eine > identische Datei "file2" unterhalb von "conffiles/etc" hingegen nicht. Kann ich so nicht nachvollziehen; ich vermute das nur die Datei unterhalb von conffile/ verschoben wurde, aber debian/$pkg.univention-config-registry nicht entsprechend angepasst wurde. ucslint arbeitet nur auf den dort angegebenen Dateien da auch nur diese installiert werden. $ cd testframework/0004-56 $ PYTHONPATH=../.. ../../bin/ucslint -m 4 using fallback ucslint.base instead of univention.ucslint.base I:0004-56: conffiles/file1: No UCR variables used $ mkdir conffiles/etc $ mv conffiles/file1 conffiles/etc/ $ sed -i '/^File:/s|file1|/etc/&|' debian/test1.univention-config-registry $ PYTHONPATH=../.. ../../bin/ucslint -m 4 using fallback ucslint.base instead of univention.ucslint.base I:0004-56: conffiles/etc/file1: No UCR variables used
(In reply to comment #5) > (In reply to comment #4) > > Die Prüfung scheint nicht immer alle Dateien zu berücksichtigen. So wird eine > > fehlerhafte Datei "file1" unterhalb von "conffiles" zwar bemängelt, eine > > identische Datei "file2" unterhalb von "conffiles/etc" hingegen nicht. > > Kann ich so nicht nachvollziehen; ich vermute das nur die Datei unterhalb von > conffile/ verschoben wurde, aber debian/$pkg.univention-config-registry nicht > entsprechend angepasst wurde. ucslint arbeitet nur auf den dort angegebenen > Dateien da auch nur diese installiert werden. Tritt auch bei mir nicht mehr auf. Changelog passt, Verified.
UCS 3.1-0 has been released: http://forum.univention.de/viewtopic.php?f=54&t=2125 If this error occurs again, please use "Clone This Bug".