Add C++ signal interfaces
Bug #1590442 reported by
Ted Gould
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
ubuntu-app-launch (Ubuntu) |
Fix Released
|
High
|
Unassigned |
Bug Description
Current the observers haven't been ported to the C++ interface. Also need to refine the manager interface as proposed in the headers.
Related branches
lp:~ted/ubuntu-app-launch/app-object-signals
- dobey (community): Needs Fixing
- Charles Kerr (community): Approve
- unity-api-1-bot: Needs Fixing (continuous-integration)
- PS Jenkins bot: Pending (continuous-integration) requested
-
Diff: 3388 lines (+1673/-939)24 files modifiedlibubuntu-app-launch/application-impl-base.cpp (+15/-16)
libubuntu-app-launch/application.cpp (+5/-0)
libubuntu-app-launch/helper.h (+12/-0)
libubuntu-app-launch/libubuntu-app-launch.map (+1/-0)
libubuntu-app-launch/registry-impl.cpp (+565/-29)
libubuntu-app-launch/registry-impl.h (+88/-11)
libubuntu-app-launch/registry.cpp (+49/-6)
libubuntu-app-launch/registry.h (+123/-14)
libubuntu-app-launch/ubuntu-app-launch.cpp (+298/-436)
tests/CMakeLists.txt (+1/-0)
tests/failure-test.cc (+123/-96)
tests/libual-cpp-test.cc (+192/-141)
tools/CMakeLists.txt (+21/-2)
tools/ubuntu-app-info.cpp (+18/-9)
tools/ubuntu-app-launch.cpp (+56/-68)
tools/ubuntu-app-list-pids.cpp (+7/-3)
tools/ubuntu-app-list.cpp (+1/-1)
tools/ubuntu-app-pid.cpp (+8/-4)
tools/ubuntu-app-stop.cpp (+7/-3)
tools/ubuntu-app-triplet.cpp (+1/-1)
tools/ubuntu-app-watch.cpp (+61/-88)
tools/ubuntu-helper-list.cpp (+1/-1)
tools/ubuntu-helper-start.cpp (+10/-5)
tools/ubuntu-helper-stop.cpp (+10/-5)
Superseded
for merging
into
lp:ubuntu-app-launch/16.10
- PS Jenkins bot (community): Needs Fixing (continuous-integration)
- Indicator Applet Developers: Pending requested
-
Diff: 3184 lines (+1583/-880)22 files modifiedlibubuntu-app-launch/application-impl-libertine.cpp (+1/-2)
libubuntu-app-launch/libubuntu-app-launch.map (+1/-0)
libubuntu-app-launch/registry-impl.cpp (+493/-11)
libubuntu-app-launch/registry-impl.h (+76/-11)
libubuntu-app-launch/registry.cpp (+42/-2)
libubuntu-app-launch/registry.h (+120/-14)
libubuntu-app-launch/ubuntu-app-launch.cpp (+339/-413)
tests/CMakeLists.txt (+1/-0)
tests/failure-test.cc (+123/-96)
tests/libual-cpp-test.cc (+186/-141)
tools/CMakeLists.txt (+21/-2)
tools/ubuntu-app-info.cpp (+18/-9)
tools/ubuntu-app-launch.cpp (+56/-68)
tools/ubuntu-app-list-pids.cpp (+7/-3)
tools/ubuntu-app-list.cpp (+1/-1)
tools/ubuntu-app-pid.cpp (+8/-4)
tools/ubuntu-app-stop.cpp (+7/-3)
tools/ubuntu-app-triplet.cpp (+1/-1)
tools/ubuntu-app-watch.cpp (+61/-88)
tools/ubuntu-helper-list.cpp (+1/-1)
tools/ubuntu-helper-start.cpp (+10/-5)
tools/ubuntu-helper-stop.cpp (+10/-5)
Superseded
for merging
into
lp:ubuntu-app-launch/16.04
- Indicator Applet Developers: Pending requested
-
Diff: 391 lines (+117/-167)6 files modifiedlibubuntu-app-launch/registry-impl.cpp (+1/-3)
libubuntu-app-launch/registry-impl.h (+0/-4)
libubuntu-app-launch/registry.h (+0/-2)
tools/CMakeLists.txt (+2/-2)
tools/ubuntu-app-launch.cpp (+61/-68)
tools/ubuntu-app-watch.cpp (+53/-88)
To post a comment you must log in.
This bug was fixed in the package ubuntu-app-launch - 0.9+17. 04.20170124- 0ubuntu1
--------------- 04.20170124- 0ubuntu1) zesty; urgency=medium
ubuntu-app-launch (0.9+17.
* Change signals into C++ core::signal objects (LP: #1590442)
* Abstract out jobs backends and make Upstart one
* Move the signals into the jobs classes
* Make the signals support finding the instance of the application
(LP: #1589637)
* Jobs interface specific tests
-- Ted Gould <email address hidden> Tue, 24 Jan 2017 14:15:17 +0000