Univention Bugzilla – Full Text Bug Listing |
Summary: | Prepare App Center for Self Service (2) | ||
---|---|---|---|
Product: | UCS | Reporter: | Dirk Wiesenthal <wiesenthal> |
Component: | App Center | Assignee: | Dirk Wiesenthal <wiesenthal> |
Status: | CLOSED FIXED | QA Contact: | Florian Best <best> |
Severity: | normal | ||
Priority: | P3 | ||
Version: | UCS 4.1 | ||
Target Milestone: | UCS 4.1-2-errata | ||
Hardware: | Other | ||
OS: | Linux | ||
What kind of report is it?: | Development Internal | 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: | |||
Bug Depends on: | |||
Bug Blocks: | 40996 |
Description
Dirk Wiesenthal
2016-05-26 15:17:20 CEST
Did some code changes in r70010. Also r70011 for simplifying selfservice Also r70225, r70412, r70415. Removed dpkg-dev from the dependecy list in r70455 REOPEN: as discussed, error handling around os.chmod(result_file, 0644). The cache parameter is now configurable for univention-app update. But the app.py class always uses the hardcoded one. Is this okay? → you can't use e.g. AppManager instances in univention-app update then. This looks inconsistent IMHO. IMHO appcenter.utils.mkdir should use os.makedirs(). (In reply to Florian Best from comment #5) > REOPEN: as discussed, error handling around os.chmod(result_file, 0644). > univention-appcenter 5.0.21-12.175.201606211353 > The cache parameter is now configurable for univention-app update. But the > app.py class always uses the hardcoded one. Is this okay? → you can't use > e.g. AppManager instances in univention-app update then. This looks > inconsistent IMHO. Yes, this has been done for selfservice. It uses the all new (AppManager inspired) AppCache. > > IMHO appcenter.utils.mkdir should use os.makedirs(). You are right. Not done in this bug. univention-appcenter (5.0.21-13): r70476 | Bug #41360: Error handling for copy_file in univention-appcenter-dev OK r70455 | Bug #41360: Remove dpkg-dev dependency ~OK (The splitting works, I hope also in the future or in some special circumstances e.g. if whitespace in version, ...) r70415 | Bug #41360: Add cache dir parameter to univention-app update. OK r70412 | Bug #41360: Add cache dir parameter to univention-app update. OK r70225 | Bug #41360: Fix typo OK r70010 | Bug #41360: Prepare App Center for Self Service OK |