fix(dnf): use str starts-with for record type check in install_pkgs (#572)

fromelicks · web-flow · commit a97bce3b687e · 2026-06-12T17:17:56.000-04:00
Nushell's describe returns the full type signature for records ("record<key: type, ...>"), not just "record". The equality check == 'record' is therefore always false, silently skipping all repo-specific package installs (terra, COPR, etc.). Fix matches the existing pattern already used in add_coprs. Closes #571
diff --git a/modules/dnf/dnf.nu b/modules/dnf/dnf.nu
@@ -517,7 +517,7 @@ def install_pkgs [install: record]: nothing -> nothing {
   # Get all the entries that have a repo and/or packages specified as an object.
   let object_install_list = $install.packages
     | where {|pkg|
-      ($pkg | describe) == 'record'
+      ($pkg | describe) | str starts-with 'record'
     }
 
   for $object_install in $object_install_list {

Original file line number	Diff line number	Diff line change
`@@ -517,7 +517,7 @@ def install_pkgs [install: record]: nothing -> nothing {`
`517`	`517`	`# Get all the entries that have a repo and/or packages specified as an object.`
`518`	`518`	`let object_install_list = $install.packages`
`519`	`519`	`\| where {\|pkg\|`
`520`		`- ($pkg \| describe) == 'record'`
	`520`	`+ ($pkg \| describe) \| str starts-with 'record'`
`521`	`521`	`}`
`522`	`522`
`523`	`523`	`for $object_install in $object_install_list {`