Skip to content

Commit e0c4108

Browse files
authored
Fixes an issue where the old code path falls into recursive deps and causes unneeded deps to be present (#1423)
* Fixes an issue where the old code path falls into recursive deps and causes unneeded deps to be present * wip * another approach
1 parent 37f763d commit e0c4108

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/nimble.nim

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -176,7 +176,7 @@ proc processFreeDependenciesSAT(rootPkgInfo: PackageInfo, options: Options): Has
176176
versionRange = upgradeVersions[name]
177177
let resolvedDep = ((name: name, ver: versionRange)).resolveAlias(options)
178178
let (packages, _) = install(@[resolvedDep], options,
179-
doPrompt = false, first = false, fromLockFile = false, preferredPackages = @[])
179+
doPrompt = false, first = false, fromLockFile = false, preferredPackages = result.toSeq())
180180
for pkg in packages:
181181
if pkg in result:
182182
# If the result already contains the newly tried to install package

0 commit comments

Comments
 (0)