Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cmd/cgo: ldflags directive doesn't like empty spaces in paths #7906

Closed
gopherbot opened this issue Apr 30, 2014 · 7 comments

Comments

@gopherbot
Copy link

commented Apr 30, 2014

by camilo.aguilar:

In OSX it is very likely to have paths to dynamic libraries containing empty spaces.
IHMO, CGO should allow paths with spaces.

What does 'go version' print?
go version go1.2.1 darwin/amd64
@ianlancetaylor

This comment has been minimized.

Copy link
Contributor

commented Apr 30, 2014

Comment 1:

Labels changed: added repo-main, release-go1.4.

@rsc

This comment has been minimized.

Copy link
Contributor

commented Sep 15, 2014

Comment 3:

I don't understand why it is very likely. I am not sure I've ever seen a space in a path
to a dynamic library.
At the very least this bug needs an actual reproduction case. I don't know exactly what
part is even being talked about.

Labels changed: added release-none, removed release-go1.4.

Status changed to WaitingForReply.

@gopherbot

This comment has been minimized.

Copy link
Author

commented Sep 15, 2014

Comment 4 by camilo.aguilar:

One example is: /Applications/VMware\ Fusion.app/Contents/Frameworks/libvix.dylib
I agree it may not be very likely but it is still possible. If it is a hardship, then it
is not worth fixing. One can always work around.

@rsc rsc added this to the Unplanned milestone Apr 10, 2015

@rsc rsc removed release-none labels Apr 10, 2015

@cfstras

This comment has been minimized.

Copy link

commented Aug 17, 2015

This is still an issue, at least on Windows: many libraries, such as libVLC, are within "C:\Program Files". Escaping is not possible, since the go tool escapes those backslashes again.

/edit: Setting LIBRARY_PATH, however, will work.

@hydroflame

This comment has been minimized.

Copy link

commented Nov 14, 2015

Chromium Embedded Framework
has the same issue

@gopherbot

This comment has been minimized.

Copy link
Author

commented May 9, 2017

CL https://golang.org/cl/42972 mentions this issue.

@gopherbot

This comment has been minimized.

Copy link
Author

commented May 10, 2017

CL https://golang.org/cl/43072 mentions this issue.

@gopherbot gopherbot closed this in 1df777f May 10, 2017

@golang golang locked and limited conversation to collaborators May 10, 2018

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
6 participants
You can’t perform that action at this time.