Bug 51626 - Recursive package dependency: univention-ldap-overlay-memberof and univention-ldap-server
Recursive package dependency: univention-ldap-overlay-memberof and univention...
Status: RESOLVED MOVED
Product: UCS
Classification: Unclassified
Component: LDAP
UCS 5.0
Other Linux
: P5 normal (vote)
: UCS 5.0-6-errata
Assigned To: Philipp Hahn
UCS maintainers
https://git.knut.univention.de/univen...
:
Depends on: 46066
Blocks:
  Show dependency treegraph
 
Reported: 2020-07-06 14:45 CEST by Dirk Wiesenthal
Modified: 2024-02-24 15:33 CET (History)
3 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):
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Dirk Wiesenthal univentionstaff 2020-07-06 14:45:04 CEST
We may want to fix it as this leads to unpredictable installations and upgrade behavior.
Comment 1 Dirk Wiesenthal univentionstaff 2020-07-06 14:55:51 CEST
root@master50:~# LC_ALL=C apt depends univention-ldap-server univention-ldap-overlay-memberof 
univention-ldap-server
  PreDepends: slapd (>= 2.4.35-1.162.201407020834)
  PreDepends: univention-ldap-client
  Depends: bind9-host
  Depends: db5.3-util
  Depends: makepasswd
  Depends: python-univention-lib (>= 8.0.1-29)
  Depends: shell-univention-lib (>= 6.0.9-8)
  Depends: univention-base-files
  Depends: univention-directory-manager-tools (>> 7.0.147)
  Depends: univention-ipcalc
  Depends: univention-join
  Depends: univention-ldap-config (= 16.0.0-5A~5.0.0.202007061206)
  Depends: univention-ldap-overlay-memberof
  Depends: univention-newsid
  Depends: univention-ssl
  Depends: whois
  Depends: univention-config (>= 7.0.25)
  Conflicts: <univention-5250> (<< 0.5)
  Recommends: univention-virtual-machine-manager-schema
univention-ldap-overlay-memberof
  Depends: shell-univention-lib (>= 1.0.80)
  Depends: univention-ldap-server
  Depends: univention-config (>= 7.0.25)
Comment 2 Philipp Hahn univentionstaff 2024-02-24 15:33:46 CET
While investigating Bug #57078 I had to investigate the mechanism within of USS to install component packages: That mechanism is still used to force-install `univention-ldap-overlay-memberof`, which is already a hard dependency of `univention-ldap-server` since UCS 4.3 Bug #46066 and thus always installed.

On top of removing the circular dependency to guarantee correct *configure* order I did a global search for `univention-ldap-overlay-memberof` and found to references in USS and JOIN, where the package was explicitly installed and checked for. Both are unneeded and have been removed to simplify our code base.

Also the documentation on <https://docs.software-univention.de/manual/5.0/de/groups.html#groups-memberof> contained several subtile errors, which have been fixed; this is Bug #33685

[phahn/memberOf] 55ff2d7a00 refactor(USS): component package installation
 base/univention-system-setup/umc/python/setup/util.py | 11 +++--------
 1 file changed, 3 insertions(+), 8 deletions(-)

[phahn/memberOf] d1e1da9e2d refactor(USS): Remove explicit memberOf install
 base/univention-system-setup/debian/changelog         | 6 ++++++
 base/univention-system-setup/umc/python/setup/util.py | 6 +-----
 doc/errata/staging/univention-system-setup.yaml       | 7 +++++--
 3 files changed, 12 insertions(+), 7 deletions(-)

[phahn/memberOf] a6e87af01f refactor(join): No explicit check for memberOf
 doc/errata/staging/univention-join.yaml     | 12 ++++++++++++
 management/univention-join/debian/changelog |  3 ++-
 management/univention-join/univention-join  |  3 ---
 3 files changed, 14 insertions(+), 4 deletions(-)

[phahn/memberOf] 0f8ca278ad doc(memberOf): updates and fixes
 doc/manual/groups.rst                       | 31 ++++++++++--------
 doc/manual/locales/de/LC_MESSAGES/groups.po | 85 ++++++++++++++++++++++++++++++++-----------------
 2 files changed, 73 insertions(+), 43 deletions(-)

[phahn/memberOf] 5694329f92 refactor(memberOf): Python 3.7+
 management/univention-ldap-overlay-memberof/debian/control      |  2 +-
 .../univention-ldap-overlay-memberof/univention-update-memberof | 31 ++++++++++-------------------
 2 files changed, 12 insertions(+), 21 deletions(-)

[phahn/memberOf] a75f3b96c9 refactor(memberOf): Remove unneeded postinst
 management/univention-ldap-overlay-memberof/debian/control      |  1 -
 .../debian/univention-ldap-overlay-memberof.postinst            | 52 -----------------------------
 2 files changed, 53 deletions(-)

[phahn/memberOf] 6361804ab8 refactor(memberOf): Move UCRV ldap/overlay/memberof to default
 .../univention-ldap-overlay-memberof/debian/univention-ldap-overlay-memberof.postinst       | 2 --
 .../debian/univention-ldap-overlay-memberof.univention-config-registry-variables            | 2 +-
 2 files changed, 1 insertion(+), 3 deletions(-)

[phahn/memberOf] 46a5bc3455 fix(memberOf): Remove recursive package dependency
 doc/errata/staging/univention-ldap-overlay-memberof.yaml     | 11 +++++++++++
 management/univention-ldap-overlay-memberof/debian/changelog |  3 ++-
 management/univention-ldap-overlay-memberof/debian/control   |  3 ++-
 3 files changed, 15 insertions(+), 2 deletions(-)