Univention Bugzilla – Bug 23465
ucslint sollte nicht Fehler in Kommentaren suchen
Last modified: 2012-12-12 21:09:39 CET
0001-CheckJoinScript: line contains no $@: samba4servicedcs=$(ldapsearch -ZZ -LLL -D "$binddn" -y "$DCPWD" "(&(univentionService=${NAME})(objectClass=univentionDomainController))" cn | sed -n 's/^cn: \(.*\)/\1/p') ## currently there is no udm module computers/dc E:0001-2: 96univention-samba4.inst: join script contains 1 lines with missing "$@"
Ähnliches gilt für die "exit 0"-Überprüfung am Ende der Datei; wenn danach noch Kommentarzeilen folgen, liefert der Test ein False-Positive $ ucsline E:0006-6: debian/univention-management-console-module-uvmm.postinst: script contains no "exit 0" at end of file $ tail -n 2 debian/univention-virtual-machine-manager-daemon.postinst exit 0 # vim:set ft=sh ts=4 sw=4 et:
Kommentare werden jetzt zumindest am Zeilenanfang ignoriert. Diese in der Zeile zu suchen ist nicht ohne weiteres Möglich, da dazu die Syntax der Sprache und deren Quoting-Regeln bzw. Regeln für Zeichenketten umgesetzt werden müssten. → ucslint.overrides svn34332, ucslint_3.0.0-1
Zumindest die Tests der Join- und Postinst Skripte ignorieren jetzt Kommentare am Zeilenanfang. Für 0017-Shell.py könnte das auch noch übernommen werden, daher nochmal reopened.
(In reply to comment #3) > Zumindest die Tests der Join- und Postinst Skripte ignorieren jetzt Kommentare > am Zeilenanfang. > > Für 0017-Shell.py könnte das auch noch übernommen werden, > daher nochmal reopened. IMHO nein, denn selbst in Kommentaren sollte das korrigiert werden, damit sich niemand daran ein schlechtes Beispiel nimmt. Außerdem wird dort UPCFileTester() verwendet, was natürlich nicht die Quoting, Zeichenketten- und Kommentarregeln der Datei kennt, weil diese lediglich als Textdatei untersucht wird. Um solche Dinge korrekt zu lösen müsste man erst noch einen Scanner für Shell- und Python-Skript implementieren, was IMHO in keinster Weise den Aufwand rechtfertigt, wenn man sich den Nutzen anschaut. Von daher wieder RESOLVED. PS: für 0001 wurde das Ignorieren von Kommentaren im Rahmen von svn34333 angepasst.
Dann VERIFIED. Changelogeintrag vorhanden.
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".