Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit afdd4061 authored by Paul Duffin's avatar Paul Duffin
Browse files

Rename OptionalPath.RelativeToPath -> RelativeToTop

Bug: 183650682
Test: m nothing
Change-Id: I95239c2273e715e8c4c15a4a02967e01f6913234
parent a71a67a4
Loading
Loading
Loading
Loading
+3 −3
Original line number Diff line number Diff line
@@ -286,9 +286,9 @@ func (p OptionalPath) Path() Path {
	return p.path
}

// RelativeToPath returns an OptionalPath with the path that was embedded having been replaced by
// the result of calling Path.RelativeToPath on it.
func (p OptionalPath) RelativeToPath() OptionalPath {
// RelativeToTop returns an OptionalPath with the path that was embedded having been replaced by the
// result of calling Path.RelativeToTop on it.
func (p OptionalPath) RelativeToTop() OptionalPath {
	if !p.valid {
		return p
	}
+1 −1
Original line number Diff line number Diff line
@@ -5066,7 +5066,7 @@ func TestApexWithApps(t *testing.T) {
	}
	// JNI libraries including transitive deps are
	for _, jni := range []string{"libjni", "libfoo"} {
		jniOutput := ctx.ModuleForTests(jni, "android_arm64_armv8-a_sdk_shared_apex10000").Module().(*cc.Module).OutputFile().RelativeToPath()
		jniOutput := ctx.ModuleForTests(jni, "android_arm64_armv8-a_sdk_shared_apex10000").Module().(*cc.Module).OutputFile().RelativeToTop()
		// ... embedded inside APK (jnilibs.zip)
		ensureListContains(t, appZipRule.Implicits.Strings(), jniOutput.String())
		// ... and not directly inside the APEX
+1 −1
Original line number Diff line number Diff line
@@ -61,7 +61,7 @@ func TestProto(t *testing.T) {
			t.Errorf("expected %q in %q", w, cmd)
		}

		foobarPath := foobar.Module().(android.HostToolProvider).HostToolPath().RelativeToPath().String()
		foobarPath := foobar.Module().(android.HostToolProvider).HostToolPath().RelativeToTop().String()

		if w := "--plugin=protoc-gen-foobar=" + foobarPath; !strings.Contains(cmd, w) {
			t.Errorf("expected %q in %q", w, cmd)