Bug 22092 - ucslint für UCS 3.0
ucslint für UCS 3.0
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: ucslint
UCS 3.0
Other Linux
: P5 enhancement (vote)
: UCS 3.0 - MS2
Assigned To: Sönke Schwardt-Krummrich
Alexander Kläser
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2011-04-05 09:12 CEST by Sönke Schwardt-Krummrich
Modified: 2011-12-13 15:51 CET (History)
3 users (show)

See Also:
What kind of report is it?: ---
What type of bug is this?: ---
Who will be affected by this bug?: ---
How will those affected feel about the bug?: ---
User Pain:
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Sönke Schwardt-Krummrich univentionstaff 2011-04-05 09:12:12 CEST
ucslint muss noch für UCS 3.0 angepasst werden.
Comment 1 Stefan Gohmann univentionstaff 2011-05-09 11:45:20 CEST
Das funktioniert aktuell noch nicht:

ucs3:~/univention-directory-listener# dpkg -l ucslint
Gewünscht=Unbekannt/Installieren/R=Entfernen/P=Vollständig Löschen/Halten
| Status=Nicht/Installiert/Config/U=Entpackt/halb konFiguriert/
         Halb installiert/Trigger erWartet/Trigger anhängig
|/ Fehler?=(kein)/R=Neuinstallation notwendig (Status, Fehler: GROSS=schlecht)
||/ Name                                        Version                                     Beschreibung
+++-===========================================-===========================================-======================================================================================================
ii  ucslint                                     2.0.3-1.14.201105061822                     ucslint - Univention packaging check tool
ucs3:~/univention-directory-listener# dpkg -L ucslint
/.
/usr
/usr/share
/usr/share/pyshared
/usr/share/pyshared/ucslint
/usr/share/pyshared/ucslint/univention
/usr/share/pyshared/ucslint/univention/ucslint
/usr/share/pyshared/ucslint/univention/ucslint/0007-Changelog.py
/usr/share/pyshared/ucslint/univention/ucslint/0008-Translations.py
/usr/share/pyshared/ucslint/univention/ucslint/0002-CopyPasteErrors.py
/usr/share/pyshared/ucslint/univention/ucslint/__init__.py
/usr/share/pyshared/ucslint/univention/ucslint/0011-Control.py
/usr/share/pyshared/ucslint/univention/ucslint/0006-CheckPostinst.py
/usr/share/pyshared/ucslint/univention/ucslint/base.py
/usr/share/pyshared/ucslint/univention/ucslint/0004-CheckUCR.py
/usr/share/pyshared/ucslint/univention/ucslint/0009-Python.py
/usr/share/pyshared/ucslint/univention/ucslint/0010-Copyright.py
/usr/share/pyshared/ucslint/univention/ucslint/0001-CheckJoinScript.py
/usr/share/python-support
/usr/share/python-support/ucslint.public
/usr/share/doc
/usr/share/doc/ucslint
/usr/share/doc/ucslint/changelog.Debian.gz
/usr/bin
/usr/bin/ucslint.py
ucs3:~/univention-directory-listener# ucslint
-bash: ucslint: Kommando nicht gefunden.
ucs3:~/univention-directory-listener# ucslint.py
using fallback ucslint.base instead of univention.ucslint.base
using fallback ucslint.base instead of univention.ucslint.base
Traceback (most recent call last):
  File "/usr/bin/ucslint.py", line 39, in <module>
    import ucslint.base as uub
  File "/usr/bin/ucslint.py", line 39, in <module>
    import ucslint.base as uub
ImportError: No module named base
ucs3:~/univention-directory-listener#
Comment 2 Stefan Gohmann univentionstaff 2011-05-10 09:03:14 CEST
Das scheint noch nicht zu funktionieren:

Aus /var/univention/buildsystem2//logs/ucs_3.0-0/ucs_3.0-0.pam-runasroot.201105100859.log.bz2:

...
Setting up ucslint (2.0.4-2.16.201105091540) ...
...
make[1]: Leaving directory `/var/build/temp/tmp.LAZpicMYoH/3.0-0-0/pam-runasroot/pam-runasroot-5.0.1'
   debian/rules override_dh_auto_test
make[1]: Entering directory `/var/build/temp/tmp.LAZpicMYoH/3.0-0-0/pam-runasroot/pam-runasroot-5.0.1'
ucslint
using fallback ucslint.base instead of univention.ucslint.base
Traceback (most recent call last):
  File "/usr/bin/ucslint", line 39, in <module>
    import ucslint.base as uub
ImportError: No module named ucslint.base
make[1]: *** [override_dh_auto_test] Error 1
Comment 3 Philipp Hahn univentionstaff 2011-05-10 15:32:31 CEST
Noch ein Problem:
> Build auf dimma schlägt fehl: E:0007-2: latest changelog entry does not contain bug or ticket number

Ursache ist hier, das durch unseren Build-Prozeß automatisch ein neuer ChnageLog-Eintrag generiert wird, um die Build-Nummer und den Zeitstempel in die Versionsnummer des Paketes zu bekommen. Der einzige Eintrag "* auto-build" enthält aber keine Referenz auf einen Bug oder ein Ticket, weshalb usclint jetzt den erfolgreichen Bau des Paketes verhindert.

Als workaround würde es reicher, daraus erstmal eine Warnung zu machen, aber schöner wäre langfristig, wenn ucslint diesen letzten automatisch generierten ChangeLog-Eintrag ignoriert.
Comment 4 Sönke Schwardt-Krummrich univentionstaff 2011-05-10 15:36:23 CEST
(In reply to comment #2)
> make[1]: Entering directory
> `/var/build/temp/tmp.LAZpicMYoH/3.0-0-0/pam-runasroot/pam-runasroot-5.0.1'
> ucslint
> using fallback ucslint.base instead of univention.ucslint.base
> Traceback (most recent call last):
>   File "/usr/bin/ucslint", line 39, in <module>
>     import ucslint.base as uub
> ImportError: No module named ucslint.base
> make[1]: *** [override_dh_auto_test] Error 1

Fixed.
Comment 5 Sönke Schwardt-Krummrich univentionstaff 2011-05-10 16:00:37 CEST
(In reply to comment #3)
> Noch ein Problem:
> > Build auf dimma schlägt fehl: E:0007-2: latest changelog entry does not contain bug or ticket number
>
> Als workaround würde es reicher, daraus erstmal eine Warnung zu machen, aber
> schöner wäre langfristig, wenn ucslint diesen letzten automatisch generierten
> ChangeLog-Eintrag ignoriert.

Es wurde jetzt erstmal eine Warnung daraus gemacht. Das Parsen des Changelogfiles steht noch aus.
Comment 6 Stefan Gohmann univentionstaff 2011-05-13 08:19:44 CEST
univention-install-config-registry-info wird automatisch von univention-install-config-registry aufgerufen, weshalb die Beschwerde von ucslint falsch ist:

ucslint
E:0004-26: ./debian/univention-maintenance.univention-config-registry-variables exists but debian/rules contains no univention-install-config-registry-info
Comment 7 Stefan Gohmann univentionstaff 2011-05-13 08:49:10 CEST
(In reply to comment #6)
> univention-install-config-registry-info wird automatisch von
> univention-install-config-registry aufgerufen, weshalb die Beschwerde von
> ucslint falsch ist:
> 
> ucslint
> E:0004-26: ./debian/univention-maintenance.univention-config-registry-variables
> exists but debian/rules contains no univention-install-config-registry-info

Ich habe daraus jetzt temporär eine Style-Meldung von gemacht, damit ich die anderen Pakete mit aktiviertem ucslint bauen kann.

Sönke, ich denke der komplette Test sollte entfernt werden.
Comment 8 Sönke Schwardt-Krummrich univentionstaff 2011-05-16 18:01:39 CEST
(In reply to comment #7)
> (In reply to comment #6)
> > ucslint
> > E:0004-26: ./debian/univention-maintenance.univention-config-registry-variables
> > exists but debian/rules contains no univention-install-config-registry-info
> 
> Sönke, ich denke der komplette Test sollte entfernt werden.

Test wurde entfernt.
Comment 9 Sönke Schwardt-Krummrich univentionstaff 2011-06-16 11:36:09 CEST
Die bisherigen Punkte sind alle erledigt oder werden über weitere Bugs adressiert.
Comment 10 Alexander Kläser univentionstaff 2011-09-27 10:44:26 CEST
QA: OK → VERIFIED.

(In reply to comment #0)
> ucslint muss noch für UCS 3.0 angepasst werden.

→ OK, ucslint funktioniert generell für 3.0, Pakete können gebaut werden, die Tests laufen soweit korrekt durch.

(In reply to comment #5)
> > Noch ein Problem:
> > Build auf dimma schlägt fehl: E:0007-2: latest changelog entry does not contain bug or ticket number
> 
> Es wurde jetzt erstmal eine Warnung daraus gemacht. Das Parsen des
> Changelogfiles steht noch aus.

→ OK, Test funktioniert und eine Warnung wird bei fehlgeschlagenem Test ausgegeben.

(In reply to comment #8)
> > > ucslint
> > > E:0004-26: ./debian/univention-maintenance.univention-config-registry-variables
> > > exists but debian/rules contains no univention-install-config-registry-info
> > 
> > Sönke, ich denke der komplette Test sollte entfernt werden.
> 
> Test wurde entfernt.

→ OK, Test nicht mehr vorhanden.
Comment 11 Sönke Schwardt-Krummrich univentionstaff 2011-12-13 15:51:09 CET
UCS 3.0-0 wurde veröffentlicht. Sollte der hier beschriebene Bug mit einer
neueren Version von UCS erneut auftreten, so sollte dieser Bug dupliziert
werden: "Clone This Bug"