From 18e666bad75362ef7d031ebf557effdc42dd290f Mon Sep 17 00:00:00 2001 From: "Bryan C. Mills" Date: Wed, 5 May 2021 11:24:24 -0400 Subject: [PATCH] cmd/go: skip TestExecInDeletedDir on aix and solaris-based platforms They empirically do not seem to support deleting the current working directory. Updates #34499 Change-Id: I705cdf8b545fbb837d5b75d2904a7c2a89276fa5 Reviewed-on: https://go-review.googlesource.com/c/go/+/317249 Trust: Bryan C. Mills Run-TryBot: Bryan C. Mills Reviewed-by: Robert Findley --- src/cmd/go/go_test.go | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/src/cmd/go/go_test.go b/src/cmd/go/go_test.go index ff397a1995..a059a6dd90 100644 --- a/src/cmd/go/go_test.go +++ b/src/cmd/go/go_test.go @@ -2836,9 +2836,11 @@ func TestCoverpkgTestOnly(t *testing.T) { // Regression test for golang.org/issue/34499: version command should not crash // when executed in a deleted directory on Linux. func TestExecInDeletedDir(t *testing.T) { - // The crash has only been reproduced on Linux. - if runtime.GOOS == "windows" || runtime.GOOS == "plan9" { - t.Skip() + switch runtime.GOOS { + case "windows", "plan9", + "aix", // Fails with "device busy". + "solaris", "illumos": // Fails with "invalid argument". + t.Skipf("%v does not support removing the current working directory", runtime.GOOS) } tg := testgo(t) defer tg.cleanup() -- 2.48.1