Bug 41418 - bashism in ucs-school-umc-csv-import postinst
bashism in ucs-school-umc-csv-import postinst
Status: CLOSED FIXED
Product: UCS@school
Classification: Unclassified
Component: UMC - CSV Import
UCS@school 4.1
Other Linux
: P5 normal (vote)
: UCS@school 4.1 R2 v10
Assigned To: Sönke Schwardt-Krummrich
Daniel Tröder
:
: 43187 (view as bug list)
Depends on:
Blocks: 43699
  Show dependency treegraph
 
Reported: 2016-06-02 11:29 CEST by Florian Best
Modified: 2017-03-21 12:35 CET (History)
1 user (show)

See Also:
What kind of report is it?: Bug Report
What type of bug is this?: 3: Simply Wrong: The implementation doesn't match the docu
Who will be affected by this bug?: 3: Will affect average number of installed domains
How will those affected feel about the bug?: 2: A Pain – users won’t like this once they notice it
User Pain: 0.103
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:
best: Patch_Available+


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Florian Best univentionstaff 2016-06-02 11:29:25 CEST
ucs-school-umc-csv-import (3.0.2-1.45.201605311905) wird eingerichtet ...
/var/lib/dpkg/info/ucs-school-umc-csv-import.postinst: 32:
/var/lib/dpkg/info/ucs-school-umc-csv-import.postinst: configure: not found

patch:
diff --git a/ucs-school-umc-csv-import/debian/ucs-school-umc-csv-import.postinst b/ucs-school-umc-csv-import/debian/ucs-school-umc-csv-import.postinst
index 825de26..1dfbef4 100644
--- a/ucs-school-umc-csv-import/debian/ucs-school-umc-csv-import.postinst
+++ b/ucs-school-umc-csv-import/debian/ucs-school-umc-csv-import.postinst
@@ -32 +32 @@
-if "$1" == "configure" && dpkg --compare-versions "$2" lt 2.0.7-1; then
+if [ "$1" = "configure" ] && dpkg --compare-versions "$2" lt 2.0.7-1; then
Comment 1 Florian Best univentionstaff 2016-12-20 10:41:08 CET
*** Bug 43187 has been marked as a duplicate of this bug. ***
Comment 2 Sönke Schwardt-Krummrich univentionstaff 2016-12-20 14:44:07 CET
Please check if the UCR variable has ever been set in any UCS@school version. Fixing this line will set the UCR variable for the first time. Maybe we should change the default or remove the 3 lines at all.
Comment 3 Sönke Schwardt-Krummrich univentionstaff 2017-03-02 15:21:37 CET
The corresponding code block has never been executed and has been removed from postinst.

Package: ucs-school-umc-csv-import
Version: 3.0.5-1.54.201703021514
Branch: ucs_4.1-0
Scope: ucs-school-4.1r2

ucs-school-umc-csv-import.yaml:
r77249 | Bug #41387, 41418: added advisory

ucs-school-umc-csv-import (3.0.5-1):
r77251 | Bug #41418: remove broken code block in postinst
Comment 4 Daniel Tröder univentionstaff 2017-03-02 17:58:50 CET
OK: advisory (fixed typo in r77277)
OK: removal of unused code (cleanup)
OK: advisory
OK: merge to UCS@school 4.2
Comment 5 Sönke Schwardt-Krummrich univentionstaff 2017-03-21 12:35:54 CET
UCS@school 4.1 R2 v10 has been released.

http://docs.software-univention.de/changelog-ucsschool-4.1R2v10-de.html

If this error occurs again, please clone this bug.