From 1580b9fe0ab2b242227cdd7601766bdfce5493e2 Mon Sep 17 00:00:00 2001
From: Boris Kolpackov <boris@codesynthesis.com>
Date: Tue, 17 Jul 2018 14:49:04 +0200
Subject: Diagnose improper project/package arrangements in new command

---
 bdep/project.cxx | 27 ++++++++++-----------------
 1 file changed, 10 insertions(+), 17 deletions(-)

(limited to 'bdep/project.cxx')

diff --git a/bdep/project.cxx b/bdep/project.cxx
index ba8a412..795e365 100644
--- a/bdep/project.cxx
+++ b/bdep/project.cxx
@@ -125,20 +125,8 @@ namespace bdep
     return r;
   }
 
-  // Given a directory which can be a project root, a package root, or one of
-  // their subdirectories, return the absolute project (first) and relative
-  // package (second) directories. The package directory may be absent if the
-  // given directory is not within a package root or empty if the project and
-  // package roots are the same.
-  //
-  struct project_package
-  {
-    dir_path           project;
-    optional<dir_path> package;
-  };
-
-  static project_package
-  find_project_packages (const dir_path& start)
+  project_package
+  find_project_package (const dir_path& start, bool ignore_nf)
   {
     dir_path prj;
     optional<dir_path> pkg;
@@ -168,7 +156,7 @@ namespace bdep
       }
 
       // Check for the database file first since an (initialized) simple
-      // project mosl likely won't have any *.manifest files.
+      // project most likely won't have any *.manifest files.
       //
       if (exists (d / bdep_file, true)     ||
           exists (d / packages_file, true) ||
@@ -188,7 +176,12 @@ namespace bdep
     if (prj.empty ())
     {
       if (!pkg)
+      {
+        if (ignore_nf)
+          return project_package ();
+
         fail << start << " is not a (sub)directory of a package or project";
+      }
 
       // Project and package are the same.
       //
@@ -277,7 +270,7 @@ namespace bdep
     {
       for (const dir_path& d: po.directory ())
       {
-        project_package p (find_project_packages (d));
+        project_package p (find_project_package (d));
 
         // We only work on one project at a time.
         //
@@ -313,7 +306,7 @@ namespace bdep
     }
     else
     {
-      project_package p (find_project_packages (path::current_directory ()));
+      project_package p (find_project_package (path::current_directory ()));
 
       r.project = move (p.project);
 
-- 
cgit v1.1