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

Jump to bottom

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

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

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

gopherbot opened this issue Apr 30, 2014 · 7 comments
Labels
FrozenDueToAge NeedsDecision Feedback is required from experts, contributors, and/or the community before a change can be made.
Milestone
Go1.9

Comments

@gopherbot
Copy link

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
Copy link
Contributor

Comment 1:

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

@rsc
Copy link
Contributor

rsc 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
Copy link
Author

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
@cfstras
Copy link

cfstras 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.

@ghost
Copy link

ghost commented Nov 14, 2015

Chromium Embedded Framework
has the same issue

@rsc rsc modified the milestones: Go1.7, Unplanned Jan 4, 2016
@rsc rsc modified the milestones: Go1.8, Go1.7 May 17, 2016
@0xmohit 0xmohit mentioned this issue Jul 21, 2016
Closed
@aronatkins aronatkins mentioned this issue Sep 1, 2016
Closed
@quentinmit quentinmit added the NeedsFix The path to resolution is known, but the work has not been done. label Oct 5, 2016
@rsc rsc modified the milestones: Go1.9, Go1.8 Oct 21, 2016
@rsc rsc added NeedsDecision Feedback is required from experts, contributors, and/or the community before a change can be made. and removed NeedsFix The path to resolution is known, but the work has not been done. labels Oct 21, 2016
@gopherbot
Copy link
Author

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

@gopherbot
Copy link
Author

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

@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.
Assignees
No one assigned
Labels
FrozenDueToAge NeedsDecision Feedback is required from experts, contributors, and/or the community before a change can be made.
Projects
None yet
Milestone
Go1.9
Development

No branches or pull requests
5 participants
@rsc @quentinmit @cfstras @ianlancetaylor @gopherbot