Bug 31151 - dh-umc-module-* can be used for LDAP-only modules
dh-umc-module-* can be used for LDAP-only modules
Status: RESOLVED WORKSFORME
Product: UCS
Classification: Unclassified
Component: UMC (Generic)
UCS 3.1
i386 Linux
: P5 enhancement (vote)
: ---
Assigned To: UMC maintainers
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-04-24 08:26 CEST by Philipp Hahn
Modified: 2016-10-07 15:39 CEST (History)
2 users (show)

See Also:
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): API change, Cleanup, Further conceptual development, Usability
Max CVSS v3 score:


Attachments
Fixes + Cleanup (63.81 KB, patch)
2013-04-24 08:26 CEST, Philipp Hahn
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Philipp Hahn univentionstaff 2013-04-24 08:26:33 CEST
Created attachment 5190 [details]
Fixes + Cleanup

dh-umc-module-build and dh-umc-module-install cannot be used to add additional
UDM modules, which only require an umc/*.xml file, but no additional
umc/python/ and/or umc/javascript/ files.

An option should be added to allow such simple extensions.
Currently the functionality has to be re-implemented by hand, that is
- Install icons
- Install message catalog
- postinst magic
- ...

The attached patch tried this, but it was not yet tested.
The following change would be needed in
univention-management-console/dev/dh_umc.py:
- for required in (MODULE, PYTHON, DEFINITION, JAVASCRIPT):
+ for required in (MODULE, DEFINITION,):
The --core function looks very internal (the module name is hard-coded to
"umc-core"); it should be replaced by a more lenient switch to make the Python
and JavaScript code optional.

Some other issues:
- The command line parsing of umc-create-module is wrong: «for "$@"» and
«shift» don't work together.
- Fix permissions of installed files (644)
- Move umc reload from prerm to postrm: Restarting UMC while the modules are
still installed is useless.
- Fix Bug #31123
- Update DTD to match reality (Bug #26275)
- Fix XSD
- Add RelaxNG schema
Comment 1 Florian Best univentionstaff 2016-10-07 15:39:23 CEST
The UCS 4.2 dh_umc is capable of this.