]> Cypherpunks repositories - gostls13.git/commitdiff
cmd/link/internal/ld: re-enable tests on darwin
authorTobias Klauser <tklauser@distanz.ch>
Fri, 16 Apr 2021 08:17:34 +0000 (10:17 +0200)
committerTobias Klauser <tobias.klauser@gmail.com>
Fri, 16 Apr 2021 12:28:49 +0000 (12:28 +0000)
It looks like these are fixed again for darwin on current tip after CL
111258 marked them to be skipped.

Updates #23168

Change-Id: I4abecee1152ccd4f2d44d76d1acdecb0d6140981
Reviewed-on: https://go-review.googlesource.com/c/go/+/308994
Trust: Tobias Klauser <tobias.klauser@gmail.com>
Run-TryBot: Tobias Klauser <tobias.klauser@gmail.com>
TryBot-Result: Go Bot <gobot@golang.org>
Reviewed-by: Than McIntosh <thanm@google.com>
src/cmd/link/internal/ld/dwarf_test.go

index e845a95359a46df390e4f573427d3642a9c43b29..56dc4753b25c6b6ad96b7f53b28fb049c65b8bfe 100644 (file)
@@ -610,8 +610,8 @@ func TestInlinedRoutineRecords(t *testing.T) {
        if runtime.GOOS == "plan9" {
                t.Skip("skipping on plan9; no DWARF symbol table in executables")
        }
-       if runtime.GOOS == "solaris" || runtime.GOOS == "illumos" || runtime.GOOS == "darwin" {
-               t.Skip("skipping on solaris, illumos, and darwin, pending resolution of issue #23168")
+       if runtime.GOOS == "solaris" || runtime.GOOS == "illumos" {
+               t.Skip("skipping on solaris, illumos, pending resolution of issue #23168")
        }
 
        t.Parallel()
@@ -847,8 +847,8 @@ func TestAbstractOriginSanity(t *testing.T) {
        if runtime.GOOS == "plan9" {
                t.Skip("skipping on plan9; no DWARF symbol table in executables")
        }
-       if runtime.GOOS == "solaris" || runtime.GOOS == "illumos" || runtime.GOOS == "darwin" {
-               t.Skip("skipping on solaris, illumos, and darwin, pending resolution of issue #23168")
+       if runtime.GOOS == "solaris" || runtime.GOOS == "illumos" {
+               t.Skip("skipping on solaris, illumos, pending resolution of issue #23168")
        }
 
        if wd, err := os.Getwd(); err == nil {
@@ -865,8 +865,8 @@ func TestAbstractOriginSanityIssue25459(t *testing.T) {
        if runtime.GOOS == "plan9" {
                t.Skip("skipping on plan9; no DWARF symbol table in executables")
        }
-       if runtime.GOOS == "solaris" || runtime.GOOS == "illumos" || runtime.GOOS == "darwin" {
-               t.Skip("skipping on solaris, illumos, and darwin, pending resolution of issue #23168")
+       if runtime.GOOS == "solaris" || runtime.GOOS == "illumos" {
+               t.Skip("skipping on solaris, illumos, pending resolution of issue #23168")
        }
        if runtime.GOARCH != "amd64" && runtime.GOARCH != "386" {
                t.Skip("skipping on not-amd64 not-386; location lists not supported")
@@ -886,8 +886,8 @@ func TestAbstractOriginSanityIssue26237(t *testing.T) {
        if runtime.GOOS == "plan9" {
                t.Skip("skipping on plan9; no DWARF symbol table in executables")
        }
-       if runtime.GOOS == "solaris" || runtime.GOOS == "illumos" || runtime.GOOS == "darwin" {
-               t.Skip("skipping on solaris, illumos, and darwin, pending resolution of issue #23168")
+       if runtime.GOOS == "solaris" || runtime.GOOS == "illumos" {
+               t.Skip("skipping on solaris, illumos, pending resolution of issue #23168")
        }
        if wd, err := os.Getwd(); err == nil {
                gopathdir := filepath.Join(wd, "testdata", "issue26237")