← Back to issue list

feat(deb): Don't run apt-get update as non-root user

View original Github issue

Metadata

Project
craft-parts
Number
#753
Type
pull request
State
closed
Author
thp-canonical
Labels
—
Created
Updated
Closed

Current evaluation

Duplicate PR closed unmerged: author confirmed it was superseded by merged PR #752 (commit 425a84f1), which made Debian packaging run apt-get update only as superuser.

Resolution: closed superseded

Reason: Closed unmerged after tigarmo asked whether it was superseded by #752 and thp-canonical replied "Yes, indeed :)". PR #752 was merged via commit 425a84f1, implementing the same apt-get update-as-root behaviour for deb parts.

Evaluated by: mr-cal-bot

No scores available.

Issue body

- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? ----- `apt-get update` will fail as non-root user. It is only needed when some build-dependencies are not installed yet. If all build dependencies are installed, craft-parts can be run without root permissions. This was previously only possible by setting the environment variable `CRAFT_PARTS_PACKAGE_REFRESH=0`.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed superseded Duplicate PR closed unmerged: author confirmed it was superseded by merged PR #752 (commit 425a84f1), which made Debian packaging run apt-get update only as superuser.
stealth/space-bunny-alpha — closed superseded Superseded by merged PR #752, which made Debian packaging run apt update only as superuser; the author confirmed the duplication.
qwen/qwen3.6-35b-a3b — — Superseded by pull request #752. The author confirmed the change was replaced, so this pull request was closed without merging.
qwen3.6-35b-a3b-mtp-q6 — — Superseded by pull request #752. The change to prevent running apt-get update as a non-root user was implemented in the referenced PR instead, leaving this pull request closed without merging.

Update history

No update history recorded yet.

Related work

  • Superseded By: canonical/craft-parts#752 (confidence 97%)

    Merged PR implementing the identical change (apt update only as superuser); author explicitly confirmed duplication in the comments

Related issues

Issue Project State Summary Similarity
#752 feat(deb): only run apt update as superuser craft-parts merged Merged PR #752 as commit 425a84f1, replacing an environment-variable check with os.geteuid() so apt-get update runs only as root.
85%