← Back to issue list

providers: deprecate RockcraftBuilddBaseConfiguration

View original Github issue

Metadata

Project
rockcraft
Number
#72
Type
pull request
State
merged
Author
mr-cal
Labels
Created
Updated
Closed

Current evaluation

Merged deprecation of RockcraftBuilddBaseConfiguration, replacing it with BuilddBase using a compatibility tag parameter. Removes the custom class with no functional impact. Approved and passed CI.

Suggested action:

No scores available.

Issue body

- [X] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? ----- Use the compatibility tag parameter when defining a base. This was the last reason `rockcraft` was using `RockcraftBuilddBaseConfiguration`. Because the class `RockcraftBuilddBaseConfiguration` no longer is needed, its parent class `BuilddBase` in `craft-providers` is used instead. No functional changes. (CRAFT-1348)

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged deprecation of RockcraftBuilddBaseConfiguration, replacing it with BuilddBase using a compatibility tag parameter. Removes the custom class with no functional impact. Approved and passed CI.
qwen3.6-35b-a3b-mtp-q6 Merged deprecation of RockcraftBuilddBaseConfiguration. Replaced with the parent BuilddBase class from craft-providers via a compatibility tag parameter. No functional changes. Closes CRAFT-1348.
qwen3.6-35b-a3b-mtp-q6 Merged deprecation of RockcraftBuilddBaseConfiguration, replacing it with BuilddBase via a compatibility tag parameter. No functional changes. Resolves CRAFT-1348.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#3934 providers: deprecate SnapcraftBuilddBaseConfiguration snapcraft merged 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.
78%
#892 (internal) Remove CharmcraftBuilddBaseConfiguration and use BuilddBase directly (CRAFT-1278, CRAFT-1356). charmcraft merged Removed CharmcraftBuilddBaseConfiguration in favor of direct BuilddBase usage. Merged after single reviewer approval, modifying six files and resolving CRAFT-1278 and CRAFT-1356.
71%
#13 project: make build-base optional, defaults to base (CRAFT-590) rockcraft merged Merged after review and CI checks, making the build-base attribute in rockcraft.yaml optional and defaulting to base when unspecified. Implementation spans three files with 42 additions and 8 deletions.
70%