diff options
author | Ulf Adams <ulfjack@google.com> | 2015-08-24 07:41:45 +0000 |
---|---|---|
committer | Han-Wen Nienhuys <hanwen@google.com> | 2015-08-24 14:02:37 +0000 |
commit | 52bcf5a5f03eba1e990006df589f351acbcd8692 (patch) | |
tree | b2cf84c58bd3c8d0f472d0e5e57b14be971cf0f0 /src/main/java | |
parent | 4217a8edf7911447013e061d0f29cb6303badf41 (diff) |
Change the skylark Java rules to use the configuration path separator.
The build-in rules also use it.
--
MOS_MIGRATED_REVID=101343109
Diffstat (limited to 'src/main/java')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/analysis/config/BuildConfiguration.java | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/analysis/config/BuildConfiguration.java b/src/main/java/com/google/devtools/build/lib/analysis/config/BuildConfiguration.java index 8b526ff75d..6cafb2a57a 100644 --- a/src/main/java/com/google/devtools/build/lib/analysis/config/BuildConfiguration.java +++ b/src/main/java/com/google/devtools/build/lib/analysis/config/BuildConfiguration.java @@ -1494,8 +1494,8 @@ public final class BuildConfiguration { @SkylarkCallable(name = "host_path_separator", structField = true, doc = "Returns the separator for PATH environment variable, which is ':' on Unix.") public String getHostPathSeparator() { - // TODO(bazel-team): This needs to change when we support Windows. - return ":"; + // TODO(bazel-team): Maybe do this in the constructor instead? This isn't serialization-safe. + return OS.getCurrent() == OS.WINDOWS ? ";" : ":"; } /** |