Skip to content

Commit c44b74e

Browse files
committed
Update comments about Windows #cgo compiler directives
1 parent ebc7a9a commit c44b74e

File tree

2 files changed

+7
-1
lines changed

2 files changed

+7
-1
lines changed

README.md

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,9 @@ Some Windows capability is also included. One constraint in the cgo package is i
3535
for path names containing spaces and special characters, which makes it tricky to
3636
compile against a copy of MQ installed in the regular location. To build these packages I copied
3737
<mq install>/tools/c/include and <mq install>/bin64 to be under a temporary directory, shown
38-
in the CFLAGS and LDFLAGS directives.
38+
in the CFLAGS and LDFLAGS directives. This constraint seems to be removed
39+
in the Go 1.9 compiler; once that has reached a general availability level,
40+
I'll update this package to use standard MQ install paths.
3941

4042
## Getting started
4143

ibmmq/mqi.go

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,10 @@ the particular MQRC or MQCC values.
1717
The build directives for Windows assume the header and library files have
1818
been copied to a temporary location, because the default paths are not
1919
acceptable to Go (it does not like spaces or special characters like ~).
20+
Note: This problem appears to have been fixed in Go 1.9, and once that
21+
is fully available, the directives will be changed to a more reasonable
22+
path in this file. For example
23+
cgo windows CFLAGS -I"c:/Program Files/IBM/MQ/tools/c/include" -m64
2024
2125
The build directives for Linux assume the default MQ installation path
2226
in /opt/mqm. These would need to be changed in this file if you use a

0 commit comments

Comments
 (0)