aboutsummaryrefslogtreecommitdiff
path: root/libbuild2/scope.cxx
diff options
context:
space:
mode:
authorBoris Kolpackov <boris@codesynthesis.com>2022-10-24 08:46:25 +0200
committerBoris Kolpackov <boris@codesynthesis.com>2022-10-24 08:46:25 +0200
commit8bb6e9ab21f47e070a089e316c6795cc5b473b80 (patch)
tree0ec770a95f9a963f3265670e6a46db7a0cd350e8 /libbuild2/scope.cxx
parent27342e7a1066a3f1426a624a35e52c23bdedeed9 (diff)
Allow non-parallel absolute src/out to support imported target tagging
Also fix bug in out clearing.
Diffstat (limited to 'libbuild2/scope.cxx')
-rw-r--r--libbuild2/scope.cxx41
1 files changed, 32 insertions, 9 deletions
diff --git a/libbuild2/scope.cxx b/libbuild2/scope.cxx
index 0d952f0..51a1c25 100644
--- a/libbuild2/scope.cxx
+++ b/libbuild2/scope.cxx
@@ -811,12 +811,16 @@ namespace build2
const dir_path& sd (src_path ());
const dir_path& od (out_path ());
+ bool nabs (false);
+
if (dir.empty ())
- dir = src ? sd : od; // Already dormalized.
+ dir = src ? sd : od; // Already normalized.
else
{
if (dir.relative ())
dir = (src ? sd : od) / dir;
+ else if (src)
+ nabs = true;
dir.normalize ();
}
@@ -824,16 +828,30 @@ namespace build2
dir_path out;
if (src)
{
- out = o.dir.relative () ? od / o.dir : move (o.dir);
+ bool oabs (o.dir.absolute ());
+
+ out = oabs ? move (o.dir) : od / o.dir;
out.normalize ();
- // Make sure out and src are parallel.
+ // Make sure out and src are parallel unless both were specified as
+ // absolute. We make an exception for this case because out may be used
+ // to "tag" imported targets (see cc::search_library()). So it's sort of
+ // the "I know what I am doing" escape hatch (it would have been even
+ // better to verify such a target is outside any project but that won't
+ // be cheap).
//
// See similar code for prerequisites in parser::parse_dependency().
//
- if (out.sub (root_->out_path ()) &&
- dir.sub (root_->src_path ()) &&
- out.leaf (root_->out_path ()) == dir.leaf (root_->src_path ()))
+ if (nabs && oabs)
+ ;
+ else if (root_->out_eq_src ()
+ ? out == dir
+ //
+ // @@ PERF: could just compare leafs in place.
+ //
+ : (out.sub (root_->out_path ()) &&
+ dir.sub (root_->src_path ()) &&
+ out.leaf (root_->out_path ()) == dir.leaf (root_->src_path ())))
;
else
// @@ TMP change warn to fail after 0.16.0 release.
@@ -841,10 +859,15 @@ namespace build2
warn (loc) << "target output directory " << out
<< " must be parallel to source directory " << dir;
- // If in source build, then out must be empty.
+ // If this target is in this project, then out must be empty if this is
+ // in source build. We assume that if either src or out are relative,
+ // then it belongs to this project.
//
- if (sd == od)
- out.clear ();
+ if (root_->out_eq_src ())
+ {
+ if (!nabs || !oabs || out.sub (root_->out_path ()))
+ out.clear ();
+ }
}
o.dir = move (out); // Result.