From 1d8e20d744bae14ab3814125e6d9ac28b9fc97e3 Mon Sep 17 00:00:00 2001 From: Matthew Maurer Date: Mon, 20 Nov 2023 21:18:12 +0000 Subject: rust: internalize srcPathFromModuleSrcs This was frequently misused (for example, in the prebuilts module, it was used as a complex "assert(len(srcs))==1"), and can be superceded by getCrateRoot anywhere it was used. It's now only called from compiler.go, and can drop the second return parameter, as it was only actually used by the prebuilt assert misuse. Bug: 309943184 Test: m nothing Change-Id: I6c92580bc8f0ecb7586c544056b5409e6dd280e7 --- rust/compiler_test.go | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'rust/compiler_test.go') diff --git a/rust/compiler_test.go b/rust/compiler_test.go index ec6829a1b..89f4d1abf 100644 --- a/rust/compiler_test.go +++ b/rust/compiler_test.go @@ -67,6 +67,7 @@ func TestCfgsToFlags(t *testing.T) { func TestEnforceSingleSourceFile(t *testing.T) { singleSrcError := "srcs can only contain one path for a rust file and source providers prefixed by \":\"" + prebuiltSingleSrcError := "prebuilt libraries can only have one entry in srcs" // Test libraries testRustError(t, singleSrcError, ` @@ -90,7 +91,7 @@ func TestEnforceSingleSourceFile(t *testing.T) { }`) // Test prebuilts - testRustError(t, singleSrcError, ` + testRustError(t, prebuiltSingleSrcError, ` rust_prebuilt_dylib { name: "foo-bar-prebuilt", srcs: ["liby.so", "libz.so"], -- cgit v1.2.3-59-g8ed1b