← Back to issue list

extensions: fix install path

View original Github issue

Metadata

Project
snapcraft
Number
#2236
Type
pull request
State
merged
Author
kyrofa
Labels
Created
Updated
Closed

Current evaluation

Merged fix for extension installation path, changing directory from extensions-data to extensions. Resolves Launchpad bug #1790197. Approved, passed CI, and merged with a single line change.

Suggested action:

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 tests/unit`? ----- Extensions are currently being installed into "extensions-data" instead of the expected "extensions". This PR resolves [LP: #1790197](https://bugs.launchpad.net/snapcraft/+bug/1790197) by installing extension parts into "extensions".

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged fix for extension installation path, changing directory from extensions-data to extensions. Resolves Launchpad bug #1790197. Approved, passed CI, and merged with a single line change.
qwen3.6-35b-a3b-mtp-q6 Merged into master by maintainer kyrofa, fixing the extension installation path from extensions-data to extensions. Resolves Launchpad bug #1790197. All unit and static tests passed, with negligible coverage impact.
qwen3.6-35b-a3b-mtp-q6 Merged into master. Corrected extension installation path from extensions-data to extensions, resolving Launchpad bug #1790197. Author kyrofa. Unit and static tests passed with negligible coverage increase.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#3768 extensions: Fix install command snapcraft merged Merged a fix for the extensions install command. The update corrects the destination path to include the full filename, resolving a directory creation issue with install -D. The change modifies two files, passes CI checks, and was merged without review comments.
80%
#3810 extensions: method to get data dir snapcraft merged Merged an update adding a method to retrieve the extension data directory. After brief discussion on test paths, the reviewer approved the changes, CI passed, and the code was integrated.
72%
#2355 extensions: cleanup and generic tests snapcraft merged Resolved LP: #1791172 by ensuring extensions are properly installable and adding generic smoke tests for future extensions. Merged after approval and passing CI, modifying 10 files (+133/-17).
72%