From ea73e713f503c85b7de6af372f2702e20838ff3f Mon Sep 17 00:00:00 2001 From: mesh-admin Date: Tue, 13 Oct 2026 11:59:31 +0000 Subject: [PATCH] i3status-rust: name the pacman-contrib package apart from the update script, and expect it in the test (issue 628) The package took the id `updates`, which the update script already holds, so the node-engine refused two resources with one identity. --- modules/i3status-rust/cmd/i3status-rust-tools/manifest_test.go | 3 ++- modules/i3status-rust/module.json | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/modules/i3status-rust/cmd/i3status-rust-tools/manifest_test.go b/modules/i3status-rust/cmd/i3status-rust-tools/manifest_test.go index ec41d8db..14e41b62 100644 --- a/modules/i3status-rust/cmd/i3status-rust-tools/manifest_test.go +++ b/modules/i3status-rust/cmd/i3status-rust-tools/manifest_test.go @@ -23,7 +23,8 @@ func TestItClaimsTheBarSeatAndRequiresTheXDisplay(t *testing.T) { if !reflect.DeepEqual(m.Requires, []string{"x11-display"}) { t.Fatalf("requires: %v", m.Requires) } - if present, absent := m.packages(); !reflect.DeepEqual(present, []string{"i3status-rust"}) || absent != nil { + // pacman-contrib for the update block's checkupdates, declared here though the pacman module declares it too (ADR 0303). + if present, absent := m.packages(); !reflect.DeepEqual(present, []string{"i3status-rust", "pacman-contrib"}) || absent != nil { t.Fatalf("packages: %v, absent %v", present, absent) } } diff --git a/modules/i3status-rust/module.json b/modules/i3status-rust/module.json index acedff96..1edf68cf 100644 --- a/modules/i3status-rust/module.json +++ b/modules/i3status-rust/module.json @@ -36,7 +36,7 @@ "package": "i3status-rust" }, { - "id": "updates", + "id": "checkupdates", "type": "package", "package": "pacman-contrib" },