← Back to issue list

Always set prefix in the autotools plugin

View original Github issue

Metadata

Project
snapcraft
Number
#536
Type
pull request
State
closed
Author
attente
Labels
—
Created
Updated
Closed

Current evaluation

Closed unmerged: hard-coding an autotools prefix would break existing builders; maintainers suggested a separate install-via option instead.

Resolution: closed not a bug

Reason: Maintainer sergiusens closed the PR on 2016-06-22 because the change would break the existing contract for many builders. No resolving merge or commit was recorded.

Evaluated by: mr-cal-bot

No scores available.

Issue body

Some codebases use prefix, libdir, datadir, etc. to locate installed resources at runtime. Since those paths are accessed at runtime, the prefix always needs to match the final mounted location of the snap.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed not a bug Closed unmerged: hard-coding an autotools prefix would break existing builders; maintainers suggested a separate install-via option instead.
stealth/space-bunny-alpha — closed not a bug Rejected unmerged: #536 was closed because always setting the autotools prefix would break existing builder contracts; maintainers suggested adding a separate install-via option.
qwen/qwen3.6-35b-a3b — — Closed as rejected. Always setting the autotools prefix would break existing projects by hardcoding paths. Maintainers declined the merge to preserve builder compatibility, suggesting a new install-via enum instead.
qwen3.6-35b-a3b-mtp-q6 — — Rejected and closed. The proposed prefix change would break existing projects and hardcode paths. Maintainers suggested a new install-via enum instead, but the pull request was closed to preserve existing builder contracts.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#1544630 CMake plugin should support installing via prefix instead of DESTDIR snapcraft (launchpad) closed Prefix-based installation landed for Autotools in merged PR #351; referenced PR #346 was closed unmerged after targeting the wrong plugin.
72%
#346 CMake plugin: Support installing via prefix. snapcraft closed Closed unmerged after the change was found to target CMake instead of Autotools; equivalent prefix-install support merged in snapcraft#351 via commit 72dbc3285.
72%
#1544629 Autotools plugin should support installing via prefix instead of DESTDIR snapcraft (launchpad) closed Prefix-based Autotools installation was merged in canonical/snapcraft#351, resolving Launchpad bug #1544629 in commit 72dbc3285.
72%
#351 Autotools plugin: Support installing via prefix. snapcraft merged Merged PR #351 by kyrofa adds an Autotools plugin config option to install via prefix instead of DESTDIR, fixing LP bug #1544629 (commit 72dbc3285). Reviewed and approved by sergiusens and come-maiz.
72%