aboutsummaryrefslogtreecommitdiff
path: root/src/plugin
diff options
context:
space:
mode:
authorAlex Brainman <alex.brainman@gmail.com>2018-11-24 18:02:29 +1100
committerAlex Brainman <alex.brainman@gmail.com>2018-11-27 08:52:43 +0000
commit41fd4c88ad900765716d17c14eb1b48b553589e2 (patch)
treea26fe86ac3bbf86641f543c42a75ebd9d4a07807 /src/plugin
parent440368da526b69fe9a500e29ce9cd84aa7cc6c35 (diff)
downloadgo-41fd4c88ad900765716d17c14eb1b48b553589e2.tar.gz
go-41fd4c88ad900765716d17c14eb1b48b553589e2.zip
cmd/link: do not use _GLOBAL_OFFSET_TABLE_ on windows/386
When building windows/386 executable that imports "plugin" package, cmd/link adds reference to DLL with blank name. Running objdump -x a.exe reports ... The Import Tables (interpreted .idata section contents) ... DLL Name: vma: Hint/Ord Member-Name Bound-To 25308a 0 _GLOBAL_OFFSET_TABLE_ ... So, obviously, executable cannot run, because Windows complains that it cannot find DLL when trying to run it. Stop using _GLOBAL_OFFSET_TABLE_ on windows/386. Fixes #28789 Change-Id: Idd489eafd998f6e329f40c5d90a2a8965ab1d873 Reviewed-on: https://go-review.googlesource.com/c/151139 Run-TryBot: Alex Brainman <alex.brainman@gmail.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Ian Lance Taylor <iant@golang.org>
Diffstat (limited to 'src/plugin')
-rw-r--r--src/plugin/plugin_test.go17
1 files changed, 17 insertions, 0 deletions
diff --git a/src/plugin/plugin_test.go b/src/plugin/plugin_test.go
new file mode 100644
index 0000000000..6dfe14854c
--- /dev/null
+++ b/src/plugin/plugin_test.go
@@ -0,0 +1,17 @@
+// Copyright 2018 The Go Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style
+// license that can be found in the LICENSE file.
+
+// +build !nacl
+
+package plugin_test
+
+import (
+ _ "plugin"
+ "testing"
+)
+
+func TestPlugin(t *testing.T) {
+ // This test makes sure that executable that imports plugin
+ // package can actually run. See issue #28789 for details.
+}