Bug 42122

Summary: download-packages is called upon u-s-s-boot update
Product: UCS Reporter: Erik Damrose <damrose>
Component: System setupAssignee: Erik Damrose <damrose>
Status: CLOSED FIXED QA Contact: Alexander Kläser <klaeser>
Severity: normal    
Priority: P5 CC: klaeser
Version: UCS 4.1   
Target Milestone: UCS 4.1-3-errata   
Hardware: Other   
OS: Linux   
See Also: https://forge.univention.org/bugzilla/show_bug.cgi?id=42402
What kind of report is it?: Bug Report What type of bug is this?: 4: Minor Usability: Impairs usability in secondary scenarios
Who will be affected by this bug?: 5: Will affect all 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.229 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:

Description Erik Damrose univentionstaff 2016-08-25 10:59:41 CEST
if [ "$1" = "configure" ]; then
        # download packages when package is updated
        if [ -n "$2" ]; then
                /usr/share/univention-system-setup/download-packages
        fi
...

is called in u-s-s-boot.postinst, but i do not see the reason. It delays installations for a long time if u-s-s-boot is updated. I think it should be called only on new installations
Comment 1 Alexander Kläser univentionstaff 2016-08-25 12:58:58 CEST
As discussed, the logic could be adapted by including a check for
> is_ucr_true system/setup/boot/installer
... this UCR variable is true if the Debian installer executed the setup process with Firefox.
Comment 2 Erik Damrose univentionstaff 2016-08-29 11:11:13 CEST
I removed the call to download-packages completely. It is done in startxwithfirefox anyway when using the DVD installer and quitting firefox. For appliances, we call it manually in our scripts.

r71983 univention-system-setup 9.0.4-37
r71984 yaml
Comment 3 Erik Damrose univentionstaff 2016-08-31 11:27:08 CEST
r72101 merge to ucs 4.2
Comment 4 Alexander Kläser univentionstaff 2016-08-31 13:31:20 CEST
Changes: OK
Merge to 4.2-0: OK
YAML file: I adapted the entry to be less technical [r72116]
Comment 5 Janek Walkenhorst univentionstaff 2016-09-07 18:41:57 CEST
<http://errata.software-univention.de/ucs/4.1/244.html>