bsp: drop /etc/apt/preferences.d/armbian — desktop pin moves to armbian-config#9664
bsp: drop /etc/apt/preferences.d/armbian — desktop pin moves to armbian-config#9664igorpecovnik wants to merge 1 commit intomainfrom
Conversation
…an-config The pin block in this file (firefox / firefox-esr / thunderbird at o=Armbian:1001, o=Ubuntu:50) is moving to armbian-config, which will write /etc/apt/preferences.d/armbian-desktops on every `module_desktops install` (see armbian/configng#850). The build framework will install desktops via armbian-config going forward, so the runtime helper covers both image-build time and post-install upgrades. That gives the pin a single source of truth in the configng repo (next to the YAML that lists the packages the pin protects), and dodges the dpkg conffile preservation trap that left this file commented-out and inert on systems built between May 2023 and Feb 2026. Existing systems with this file already deployed will keep it as an orphan conffile — it does no harm because the priorities match what armbian-config emits to the new -desktops file. The legacy file can be cleaned up by users (or a later BSP postrm) if needed, but does not need active removal.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
💤 Files with no reviewable changes (1)
📝 WalkthroughWalkthroughA single APT pinning configuration file was deleted that previously enforced Firefox, Firefox-ESR, and Thunderbird packages to be sourced from the Armbian repository while deprioritizing Ubuntu versions, thereby removing the repository preference constraints. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Summary
The pin block in packages/bsp/common/etc/apt/preferences.d/armbian (
firefox/firefox-esr/thunderbirdato=Armbianpriority 1001,o=Ubuntupriority 50) is moving to armbian-config, where it will be written to/etc/apt/preferences.d/armbian-desktopson everymodule_desktops install— see armbian/configng#850.Why move it
The BSP file is a dpkg conffile. From May 2023 → Feb 2026 it shipped fully commented-out (commit
bc5329e— "It should not be needed anymore since we re-pack those packages"). When commitf169f2c2re-enabled it in Feb 2026, dpkg's conffile preservation rule meant existing systems kept the commented version — the new active pin only landed on freshly-installed systems. We just hit this on a real test box where thunderbird installed as snap because the local pin was inert.By moving ownership to armbian-config and writing to a distinct path (
armbian-desktopsinstead ofarmbian), dpkg conffile preservation becomes moot. The runtime helper rewrites the file on every desktop install, so the pin is always current.Why this is safe to delete from BSP now
module_desktops installgoing forward, so the pin lands at image build time too — no gap./etc/apt/preferences.d/armbiankeep it as an orphan conffile. Its priorities match what armbian-config now writes toarmbian-desktops, so they compose harmoniously — no conflict, no double-pinning.Stack
This PR's effect is fully realized only after armbian/configng#850 lands. If the build framework still does its own desktop install path on some boards, those would temporarily get no pin (snap-shim could win) until configng's runtime helper takes over.
Test plan
/etc/apt/preferences.d/armbianis absent and/etc/apt/preferences.d/armbian-desktopsis present with the broader pin block (chromium / firefox / thunderbird / chrome / code / edge)apt-cache policy thunderbirdshows Armbian = 1001, Ubuntu = 50Summary by CodeRabbit