Remove dead DBus code
Bug #1657372 reported by
Adam Collard
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Landscape Client |
Fix Committed
|
Low
|
Adam Collard | ||
landscape-client (Ubuntu) |
Fix Released
|
Undecided
|
Andreas Hasenack |
Bug Description
Back in the days of Lucid landscape-client used DBus as a communication mechanism between local components. That has long since been superseded by amp and the dead code should be removed.
Related branches
lp://staging/~adam-collard/landscape-client/drop-dbus
- 🤖 Landscape Builder: Approve (test results)
- Free Ekanayaka (community): Approve
- Alberto Donato: Approve
-
Diff: 979 lines (+29/-626)28 files modifiedMakefile (+0/-7)
README (+3/-7)
dbus-1/landscape.conf (+0/-50)
debian/changelog (+6/-0)
debian/control (+2/-2)
debian/landscape-client.install (+0/-3)
debian/landscape-client.manpages (+0/-1)
debian/landscape-client.postinst (+0/-11)
debian/landscape-common.postinst (+3/-3)
debian/landscape-common.prerm (+1/-1)
debian/rules (+0/-18)
debian/source.lintian-overrides (+1/-12)
dev/run-piuparts (+0/-14)
landscape/broker/exchange.py (+1/-2)
landscape/lib/bpickle_dbus.py (+0/-65)
landscape/service.py (+0/-6)
landscape/tests/test_service.py (+0/-9)
landscape/tests/test_textmessage.py (+0/-112)
landscape/textmessage.py (+0/-103)
landscape/watchdog.py (+2/-4)
man/landscape-client.1 (+5/-5)
man/landscape-config.1 (+2/-2)
man/landscape-message.1 (+0/-51)
man/landscape-message.txt (+0/-32)
man/landscape-sysinfo.1 (+2/-2)
scripts/landscape-dbus-proxy (+0/-91)
scripts/landscape-message (+0/-8)
setup.py (+1/-5)
Changed in landscape-client: | |
status: | In Progress → Fix Committed |
Changed in landscape-client (Ubuntu): | |
assignee: | nobody → Andreas Hasenack (ahasenack) |
status: | New → In Progress |
To post a comment you must log in.
This bug was fixed in the package landscape-client - 18.01-0ubuntu1
---------------
landscape-client (18.01-0ubuntu1) bionic; urgency=medium
* New upstream release 18.01: share/landscape as its only used and created by landscape-client. release- upgrader is running before apt-update (LP: #1699179) client. docs: the README file is now a markdown file, so common. postinst: no need to single out var/lib/ landscape/ .gnupg when fixing ownerships, just do it over vm-info- to-kvm- for-aws- C5-instances. patch:
- Ported to python3 (LP: #1577850)
- move Replaces/Breaks landscape-client-ui rules to landscape-common
(LP: #1560424)
- Add a libpam-systemd Depends if built for xenial (LP: #1590838)
- Some units not reporting swift usage (LP: #1588404)
- Fix missing install directories for landscape-common and drop
usr/
(LP: #1680842)
- Fix VM detection for Xen, by returning "xen" only for paravirtualized and
HVM hosts, not for dom0. (LP: #1601818)
- Add an indication of truncation to process output that has been truncated
prior to delivery to the server. (LP: #1629000)
- add /snap/bin to the PATH when executing scripts. (LP: #1635634)
- Save the original sources.list file when a repository profile is
associated with a computer and restore it when the profile is removed.
(LP: #1607529)
- Drop the legacy HAService plugin, which is no longer used.
- Avoid double-decoding package descriptions in build_skeleton_apt, which
causes an error with Xenial python-apt. (LP: #1655395)
- Remove dead dbus code and textmessage (confirmed not supported in server
for ~2 years). (LP: #1657372)
- Move bzr-builddeb conf file from deprecated location to debian/
(LP: #1658796)
- Support for new server error message about there being too many pending
computers already (LP: #1662530)
- Add a timestamp to the package reporter result (LP: #1674252)
- Check if ubuntu-
- Implicitly trust file-local sources managed by landscape. On upgrades,
add the trusted flag to the landscape file-local apt source file if it's
not there. (LP: #1736576)
- Use local system tools to change the user's password (LP: #1743558)
* clean up packaging and getting in sync with the new landscape version:
- d/rules: drop extra:suggests which is unused since 13.07.1-0ubuntu2
- Remove antique postinst code. No supported landscape-client version
installs cronjobs anymore (since a long time).
- d/landscape-
install that instead.
- d/landscape-
/
the entire parent directory.
- guard user and group removal via an empty .cleanup.* file in post, so we
only remove the user/group if we were the ones who created them at
install time.
- lintian: remove absolute path from update-motd calls in maintainer
scripts
- d/rules: drop special handling for dapper, hardy and lucid, which are no
longer supported.
- d/rules: make sure we have an "extra:Depends=" in substvars even if it's
empty
- d/rules: drop dh_pycentral handling, it's obsolete
* Dropped (already included in this version):
- d/p/set-
...