Skip to content

Commit

Permalink
Remove inconsistent error on missing subpackage path
Browse files Browse the repository at this point in the history
The behavior of the code was slightly puzzling: if the package path did
not exist, we could log an error and continue, but if it existed but
was empty, we would error out. To be consistent, simply always error out.
  • Loading branch information
Geod24 committed Dec 18, 2023
1 parent b77bc6a commit 45b5870
Showing 1 changed file with 0 additions and 4 deletions.
4 changes: 0 additions & 4 deletions source/dub/packagemanager.d
Original file line number Diff line number Diff line change
Expand Up @@ -994,10 +994,6 @@ symlink_exit:
p.normalize();
enforce(!p.absolute, "Sub package paths must be sub paths of the parent package.");
auto path = pack.path ~ p;
if (!existsFile(path)) {
logError("Package %s declared a sub-package, definition file is missing: %s", pack.name, path.toNativeString());
continue;
}
sp = Package.load(path, NativePath.init, pack);
} else sp = new Package(spr.recipe, pack.path, pack);

Expand Down

0 comments on commit 45b5870

Please sign in to comment.