← Back to issue list

add new "no-wrapper" property to apps

View original Github issue

Metadata

Project
snapcraft
Number
#1420
Type
pull request
State
merged
Author
mvo5
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #1420 redesigned app wrappers: `adapter: none` replaced the proposed `no-wrapper` property, resolving LP bug #1706575.

Resolution: closed resolved

Reason: The approved PR was merged. Commit 2a14e16ea added the app adapter property, allowing `adapter: none` to generate wrapper-free launches.

Evaluated by: mr-cal-bot

No scores available.

Issue body

A RFC branch to fix https://bugs.launchpad.net/snapcraft/+bug/1706575 This allows to specify that no shell wrapper for apps should be generated. This is useful for e.g. statically linked binaries that don't need this anyway. LP: 1706575

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #1420 redesigned app wrappers: `adapter: none` replaced the proposed `no-wrapper` property, resolving LP bug #1706575.
stealth/space-bunny-alpha — closed resolved Merged a redesigned app adapter: none property, replacing the proposed no-wrapper flag and enabling wrapper-free launches; commit 2a14e16ea addresses LP #1706575.
qwen/qwen3.6-35b-a3b — — Merged after addressing reviewer feedback. Adds an adapter: none property to apps, skipping shell wrapper generation for statically linked binaries. Finalized with a clean enum value and approved by two reviewers.
qwen3.6-35b-a3b-mtp-q6 — — Merged after incorporating review feedback. The PR introduces an adapter: none property for apps to skip generating shell wrappers, addressing bug 1706575 and supporting statically linked binaries.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#2262 schema: add "legacy" adapter type snapcraft merged Merged PR #2262 as 66c0cd0dd, adding the default "legacy" app adapter to preserve wrapper generation and resolve LP #1792018.
70%