gn: Support build directories outside the source tree.

tools/gn/path_output.cc

 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "tools/gn/path_output.h"
 
+#include "base/strings/string_util.h"
 #include "build/build_config.h"
 #include "tools/gn/filesystem_utils.h"
 #include "tools/gn/output_file.h"
 #include "tools/gn/string_utils.h"
 
-PathOutput::PathOutput(const SourceDir& current_dir, EscapingMode escaping)
+PathOutput::PathOutput(const SourceDir& current_dir,
+                       const base::StringPiece& source_root,
+                       EscapingMode escaping)
     : current_dir_(current_dir) {
-  CHECK(current_dir.is_source_absolute())
-      << "Currently this only supports writing to output directories inside "
-         "the source root. There needs to be some tweaks to PathOutput to make "
-         "doing this work correctly.";
-  inverse_current_dir_ = InvertDir(current_dir_);
-
+  inverse_current_dir_ =
+      RebaseSourceAbsolutePath("//", current_dir, source_root);
+  if (!EndsWithSlash(inverse_current_dir_))

[brettw, 2014/11/05 19:59:49]

I would hope that the above operation is always the same with regard to the
output ending in a slash or not. Can we remove this and do an assertion instead?
Or is there a complicated case it can't tell?

[zeuthen, 2014/11/07 19:24:12]

I tried adding a DHCECK() and it fails is in the PathOutput.BasicInRoot test
case:

  SourceDir build_dir("//");
  base::StringPiece source_root("/source/root");
  PathOutput writer(build_dir, source_root, ESCAPE_NONE);

where inverse_current_dir_ ends up being a ".".

+    inverse_current_dir_.push_back('/');
   options_.mode = escaping;
 }
 
 PathOutput::~PathOutput() {
 }
 
 void PathOutput::WriteFile(std::ostream& out, const SourceFile& file) const {
   WritePathStr(out, file.value());
 }
 
@@ skipping 125 matching lines @@
     // it's system-absolute.
 #if defined(OS_WIN)
     // On Windows, trim the leading slash, since the input for absolute
     // paths will look like "/C:/foo/bar.txt".
     EscapeStringToStream(out, str.substr(1), options_);
 #else
     EscapeStringToStream(out, str, options_);
 #endif
   }
 }