Merge lp://staging/~xnox/unity/no-more-upstart-dep into lp://staging/unity

Proposed by Dimitri John Ledkov
Status: Merged
Approved by: Marco Trevisan (Treviño)
Approved revision: no longer in the source branch.
Merged at revision: 4231
Proposed branch: lp://staging/~xnox/unity/no-more-upstart-dep
Merge into: lp://staging/unity
Diff against target: 85 lines (+6/-11)
5 files modified
debian/control (+0/-1)
services/CMakeLists.txt (+0/-1)
services/panel-service.c (+3/-5)
tests/CMakeLists.txt (+1/-1)
unity-shared/UpstartWrapper.cpp (+2/-3)
To merge this branch: bzr merge lp://staging/~xnox/unity/no-more-upstart-dep
Reviewer Review Type Date Requested Status
Marco Trevisan (Treviño) Approve
Review via email: mp+322283@code.staging.launchpad.net

Commit message

Drop libupstart-dev dependency, and simply include verbantim dbus object paths.

Description of the change

Drop libupstart-dev dependency, and simply include verbantim dbus object paths.

As per request from infinity.

To post a comment you must log in.
Revision history for this message
Marco Trevisan (Treviño) (3v1n0) wrote :

Is this going to be removed from the archive?

I didn't use verbatim strings as I preferred to use defined values, but... As you and inifinity wish...

review: Approve
Revision history for this message
Dimitri John Ledkov (xnox) wrote :

On 20 April 2017 at 09:59, Marco Trevisan (Treviño) <mail@3v1n0.net> wrote:
> Review: Approve
>
> Is this going to be removed from the archive?
>

Yes. As soon as aa-series open

> I didn't use verbatim strings as I preferred to use defined values, but... As you and inifinity wish...

Whilst true, I will be dropping upstart into universe and completely
out of the archive fairly quickly once aa-series open, and definitely
ahead of desktop iso switching from unity7 to gnome.

--
Regards,

Dimitri.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.