No plainbox
Metadata
Current evaluation
Merged PR #1931 (b7d3662a4), replacing obsolete Plainbox app and test parts with checkbox-cli/checkbox-ng and removing Plainbox from provider tests.
Resolution: closed resolved
Reason: PR #1931 merged as commit b7d3662a4. It updates provider tests to use checkbox-cli/checkbox-ng after Plainbox was folded into Checkbox.
Evaluated by: mr-cal-bot
No scores available.
Issue body
- [X] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [X] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [X] If this is a bugfix. Have you checked that there is a bug report open for the issue you are trying to fix on [bug reports](https://bugs.launchpad.net/snapcraft)?
- [X] If this is a new feature. Have you discussed the design on the [forum](https://forum.snapcraft.io)?
- [X] Have you successfully run `./runtests.sh static`?
- [X] Have you successfully run `./runtests.sh unit`?
-----
A follow-up to joc's https://github.com/snapcore/snapcraft/pull/1889
The plainbox library is being merged in to the checkbox
application. This new set of patches removes all plainbox-local parts and plainbox as an application (replaced by checkbox-cli). Tests (static/unit) run ok using https://pypi.python.org/pypi/checkbox-ng/1.0.0
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #1931 (b7d3662a4), replacing obsolete Plainbox app and test parts with checkbox-cli/checkbox-ng and removing Plainbox from provider tests. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged #1931 (b7d3662a4), removing the obsolete plainbox app and local parts, switching to checkbox-cli/checkbox-ng, and dropping plainbox from provider tests. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged. Removed plainbox dependencies and local application components, replacing them with checkbox-cli and checkbox-ng. Static and unit tests passed, review was approved, and changes were integrated. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Replaced plainbox dependencies and local components with checkbox-cli and checkbox-ng. Follows up on PR #1889. Static and unit tests pass successfully. |
Update history
No update history recorded yet.
Related work
-
Related To:
snapcraft#1889
(confidence 99%)
Predecessor PR #1889 first removed the Plainbox part from plugin tests.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1796000 plainbox test failure | snapcraft (launchpad) | closed | Plainbox/checkbox exporter failure was fixed by updating the checkbox-ng dependency in the plainbox spread run (commit bcf9ba69d); the maintainer closed the issue as already fixed. | |
| #1889 tests: remove plainbox part from plugin tests | snapcraft | merged | Merged #1889 (commit df9f7f795), removing the obsolete plainbox part from plugin tests after Plainbox was folded into Checkbox, fixing integration-test failures. | |
| #2140 lp 1768233 workaround for plainbox_provider plugin | snapcraft | merged | Merged PR #2140, restoring plainbox-provider Checkbox builds by adding pip, setuptools and wheel as stage-packages, with an integration test (commit d932f0fc6). | |
| #1552369 Add a plugin for Plainbox Providers | snapcraft (launchpad) | closed | Upstream Plainbox Provider support was merged via canonical/snapcraft#609 (commit 2b8ace75b), adding the plugin and Checkbox integration tests. | |
| #2799 Plainbox test updates | snapcraft | merged | Merged PR #2799 updated Plainbox tests across three files (+2/-2), completing the approved test-maintenance change. | |
| #609 Create Plainbox Provider plugin | snapcraft | merged | Merged as commit 2b8ace75b, adding the Plainbox Provider plugin, Checkbox integration tests, and a missing Python dependency in the demo snap. | |
| #2321 tests: add spread suite for plainbox plugin (#2317) | snapcraft | merged | Merged PR #2321 via commit d2b3aca84, adding Spread coverage for Plainbox plugin and snapd integration, pinning checkbox-ng, and removing a Plainbox-specific test. |