Provider for Plainbox - Canonical Certification (Legacy)

Merge ~cypressyew/plainbox-provider-canonical-certification:fix-client-launcher into plainbox-provider-canonical-certification:master-client

  1. Git
  2. lp:~cypressyew/plainbox-provider-canonical-certification
  3. fix-client-launcher
  4. Merge into master-client
Proposed by Po-Hsu Lin
Status: Merged
Approved by: Sylvain Pineau
Approved revision: 00234ad890ffc5af1f192b725104b774f23388cc
Merged at revision: 00234ad890ffc5af1f192b725104b774f23388cc
Proposed branch: ~cypressyew/plainbox-provider-canonical-certification:fix-client-launcher
Merge into: plainbox-provider-canonical-certification:master-client
Diff against target: 33 lines (+8/-19)
1 file modified
launcher/canonical-certification-cli (+8/-19)
Reviewer Review Type Date Requested Status
Sylvain Pineau (community) Approve
Review via email: mp+288739@code.qastaging.launchpad.net

Description of the change

Fix the launcher issue after migrating to the new launcher.

This fixes bug 1555927

To post a comment you must log in.
Revision history for this message
Po-Hsu Lin (cypressyew) wrote :

Thanks to Pierre's review, I have removed the redundant [config] section, which is for canonical-certification-server and added here accidentally, and I think it's deprecated now.

Revision history for this message
Sylvain Pineau (sylvain-pineau) wrote :

[config] section is still part of the launcher syntax :)

+1 for adopting the new format for the certification client launcher.

review: Approve

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.

Subscribers

People subscribed via source and target branches