← Back to issue list

providers: deprecate SnapcraftBuilddBaseConfiguration

View original Github issue

Metadata

Project
snapcraft
Number
#3934
Type
pull request
State
merged
Author
mr-cal
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #3934 (commit f6abc2efd), replacing the custom buildd configuration with craft-providers BuilddBase and moving compatibility-tag setup to its constructor.

Resolution: closed resolved

Reason: Merged PR snapcraft#3934 landed as commit f6abc2efd, removing SnapcraftBuilddBaseConfiguration and returning BuilddBase directly with the compatibility tag passed to its constructor.

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] Have you successfully run `make lint`? - [X] Have you successfully run `pytest tests/unit`? ----- The class `SnapcraftBuilddBaseConfiguration` no longer is needed, so the parent class (`BuilddBase`) in `craft-providers` is used instead. Additionally, the compatibility tag is now set via the `BuilddBase` constructor. Similar [PR in rockcraft](https://github.com/canonical/rockcraft/pull/72). (CRAFT-1282, CRAFT-1355)

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #3934 (commit f6abc2efd), replacing the custom buildd configuration with craft-providers BuilddBase and moving compatibility-tag setup to its constructor.
stealth/space-bunny-alpha — closed resolved Merged canonical/snapcraft#3934 replaced SnapcraftBuilddBaseConfiguration with craft-providers’ BuilddBase and moved compatibility-tag setup into the base constructor.
qwen/qwen3.6-35b-a3b — — Merged. Deprecated SnapcraftBuilddBaseConfiguration, replacing it with BuilddBase from craft-providers. Compatibility tagging now uses the BuilddBase constructor. Approved by one reviewer and passed CI checks.
qwen3.6-35b-a3b-mtp-q6 — — Merged. Deprecated SnapcraftBuilddBaseConfiguration, replacing it with BuilddBase from craft-providers. Compatibility tags are now set via the BuilddBase constructor, aligning with upstream updates and addressing CRAFT-1282 and CRAFT-1355.

Update history

No update history recorded yet.

Related work

Related issues

Issue Project State Summary Similarity
#72 providers: deprecate RockcraftBuilddBaseConfiguration rockcraft merged Merged PR #72 in commit 7747a03d, replacing RockcraftBuilddBaseConfiguration with craft-providers BuilddBase and compatibility tags; no functional changes.
86%
#4204 providers: remove deprecated `build_base` parameter snapcraft merged Merged PR #4204 via commit d77b51446, removing the deprecated build_base argument from craft-providers launched_environment() calls.
74%
#892 (internal) Remove CharmcraftBuilddBaseConfiguration and use BuilddBase directly (CRAFT-1278, CRAFT-1356). charmcraft merged Merged #892, removing CharmcraftBuilddBaseConfiguration and using BuilddBase directly; commit 57313bc3 was merged as 834a4fd4.
73%
#3105 build provider: clean incompatible build-environments snapcraft merged Merged #3105 as f1119fd03, rebuilding build instances when base metadata is missing or the instance came from an incompatible or legacy Snapcraft version; compatibility tests were updated.
72%
#3583 build providers: snapcraft's new base is core20 (CRAFT-544) snapcraft merged Merged PR #3583 via 2b2d7dbcf, replacing build providers' injected core18 base with core20 to match Snapcraft's new base.
72%
#1019 build: update craft-providers rockcraft merged Merged PR canonical/rockcraft#1019 (commit 841f7432, "build: update craft-providers"), a 3-line dependency bump to craft-providers that fixed "EOL devel" base errors in spread tests; snap-tests failures were confirmed expected.
72%
#835 build(deps): use dev craft-providers craft-application closed PR #835 was closed unmerged; its underlying provider test failure was fixed by merged PR #838 in commit 6b237f12.
72%
#4505 chore: remove deprecated parameter `build-base` snapcraft merged PR #4505 merged as 4aa5f63cb, dropping the deprecated build_base arg passed to craft-providers to silence its deprecation warning; the same cleanup had already landed on main via PR #4204 (d77b51466).
71%
#143 build(deps): bump craft-providers to 1.20.0 craft-application merged Merged PR #143 as commit f0349036, updating craft-providers to 1.20.0 so host-injected snaps also receive their base snap.
70%
#4530 build(deps): bump craft-providers to 1.21.0 snapcraft merged Merged as commit ba4a2fb18: bumped craft-providers to 1.21.0 (base-v6 compat, Ubuntu 24.04 Noble support, dropped Mantic) as a prerequisite for snapcraft#4519; follow-up support landed in #4532.
70%