Skip to content

Commit 2d76085

Browse files
committed
Throw an error when trying to resolve a package with a registry and registry manager isn't configured
1 parent 737d328 commit 2d76085

File tree

1 file changed

+6
-5
lines changed

1 file changed

+6
-5
lines changed

Sources/Workspace/Workspace.swift

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1257,9 +1257,11 @@ extension Workspace: PackageContainerProvider {
12571257
}
12581258

12591259
// If it's a dependency from a package registry
1260-
if let registryManager = registryManager,
1261-
let _ = package.identity.scopeAndName
1262-
{
1260+
if let _ = package.identity.scopeAndName {
1261+
guard let registryManager = registryManager else {
1262+
return completion(.failure(InternalError("registry manager not initialized")))
1263+
}
1264+
12631265
let container = RegistryPackageContainer(
12641266
package: package,
12651267
identityResolver: identityResolver,
@@ -2693,8 +2695,7 @@ extension Workspace {
26932695
diagnostics.wrap {
26942696
switch state {
26952697
case .added(let state), .updated(let state):
2696-
if let _ = registryManager,
2697-
let _ = packageRef.identity.scopeAndName,
2698+
if let _ = packageRef.identity.scopeAndName,
26982699
case .version(let version) = state.requirement
26992700
{
27002701
_ = try downloadSourceArchive(for: packageRef, at: version)

0 commit comments

Comments
 (0)