diff options
author | Tobias Thierer <tobiast@google.com> | 2020-01-10 14:50:07 +0000 |
---|---|---|
committer | Tobias Thierer <tobiast@google.com> | 2020-01-10 15:48:51 +0000 |
commit | 5b440706eb3d2eafe0c8a90d66b9b434bbc3b2ac (patch) | |
tree | f5ff9d6f4c74b209964a7026e7a0fb1e2f47d5b6 /tools | |
parent | 26cbd59b476e5abe20a6edde94b82598adb31389 (diff) |
StandardRepositories: Add OpenJDK 9+181.
This affects the behavior of the CompareUpstreams and CopyUpstreamFiles tools.
I haven't re-run those tools because they don't currently work for another
reason (I haven't managed to rebuild OpenJDK 8) but it should this change
should be correct.
CompareUpstreams.java's toplevel documentation already lists
instructions for how to check out OpenJDK 9+181.
This CL is a prerequisite for being able to refer to OpenJDK 9+181 as a
potential reference upstream for particular files.
I also renamed referenceUpstreamAsOfAndroidP() -> referenceUpstream(),
since the semantics of that method have changed over time rather than
being tied specifically to Android P.
Test: Nothing.
Bug: 129875045
Change-Id: I66b3768eb0ab807f2fe58fb7cfff4bee4d1e3583
Diffstat (limited to 'tools')
3 files changed, 5 insertions, 5 deletions
diff --git a/tools/upstream/src/main/java/libcore/CompareUpstreams.java b/tools/upstream/src/main/java/libcore/CompareUpstreams.java index bdd23645d6..0f4de2146b 100644 --- a/tools/upstream/src/main/java/libcore/CompareUpstreams.java +++ b/tools/upstream/src/main/java/libcore/CompareUpstreams.java @@ -170,8 +170,7 @@ public class CompareUpstreams { : null; for (Path relPath : relPaths) { - Repository expectedUpstream = standardRepositories.referenceUpstreamAsOfAndroidP( - relPath); + Repository expectedUpstream = standardRepositories.referenceUpstream(relPath); out.print(relPath + "\t"); Path ojluniFile = standardRepositories.ojluni().absolutePath(relPath); List<String> linesB = Util.readLines(ojluniFile); diff --git a/tools/upstream/src/main/java/libcore/CopyUpstreamFiles.java b/tools/upstream/src/main/java/libcore/CopyUpstreamFiles.java index 713786170e..807607fe4b 100644 --- a/tools/upstream/src/main/java/libcore/CopyUpstreamFiles.java +++ b/tools/upstream/src/main/java/libcore/CopyUpstreamFiles.java @@ -45,8 +45,7 @@ public class CopyUpstreamFiles { } } for (Path relPath : relPaths) { - Repository expectedUpstream = standardRepositories.referenceUpstreamAsOfAndroidP( - relPath); + Repository expectedUpstream = standardRepositories.referenceUpstream(relPath); for (Repository upstream : standardRepositories.upstreams()) { Path upstreamFile = upstream.absolutePath(relPath); if (upstreamFile != null) { diff --git a/tools/upstream/src/main/java/libcore/StandardRepositories.java b/tools/upstream/src/main/java/libcore/StandardRepositories.java index 31efe94d7a..b85533c78c 100644 --- a/tools/upstream/src/main/java/libcore/StandardRepositories.java +++ b/tools/upstream/src/main/java/libcore/StandardRepositories.java @@ -37,6 +37,7 @@ public class StandardRepositories { private final List<Repository> historicUpstreams; private final Repository openJdk8u121; private final Repository openJdk9b113; + private final Repository openJdk9p181; private final Repository openJdk7u40; private final OjluniRepository ojluni; @@ -46,6 +47,7 @@ public class StandardRepositories { allUpstreams.add(openJdk9(upstreamRoot, "9+181")); this.openJdk9b113 = addAndReturn(allUpstreams, openJdk9(upstreamRoot, "9b113+")); this.openJdk8u121 = addAndReturn(allUpstreams, openJdkLegacy(upstreamRoot, "8u121-b13")); + this.openJdk9p181 = addAndReturn(allUpstreams, openJdk9(upstreamRoot, "9+181")); Repository openJdk8u60 = addAndReturn(allUpstreams, openJdkLegacy(upstreamRoot, "8u60")); this.openJdk7u40 = addAndReturn(allUpstreams, openJdkLegacy(upstreamRoot, "7u40")); this.allUpstreams = Collections.unmodifiableList(new ArrayList<>(allUpstreams)); @@ -109,7 +111,7 @@ public class StandardRepositories { return result; } - public Repository referenceUpstreamAsOfAndroidP(Path relPath) { + public Repository referenceUpstream(Path relPath) { boolean isJsr166 = isJsr166(relPath); if (isJsr166) { return openJdk9b113; |