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/dist: request for system CFLAGS/LDFLAGS where ever CC is invoked #6882

Closed
gopherbot opened this issue Dec 3, 2013 · 4 comments
Closed

cmd/dist: request for system CFLAGS/LDFLAGS where ever CC is invoked #6882

gopherbot opened this issue Dec 3, 2013 · 4 comments

Comments

@gopherbot
Copy link

@gopherbot gopherbot commented Dec 3, 2013

by vbatts:

From the question raised on golang-nuts:
https://groups.google.com/forum/#!topic/golang-nuts/SJn48BkCUqA

What steps will reproduce the problem?
1. CFLAGS=" -WILL " LDFLAGS=" -BREAK " ./all.bash
2.
3.


What is the expected output?

# Building C bootstrap tool.
cmd/dist
gcc: error: unrecognized command line option ‘-WILL’


What do you see instead?
# Building C bootstrap tool.
cmd/dist

# Building compilers and Go bootstrap tool for host, linux/amd64.
lib9
libbio
libmach
misc/pprof
cmd/addr2line
cmd/nm
cmd/objdump
cmd/pack
cmd/prof
cmd/cc
cmd/gc
[...]


Which compiler are you using (5g, 6g, 8g, gccgo)?
n/a


Which operating system are you using?
# cat /etc/*release
LSB_VERSION=base-4.0-amd64:base-4.0-noarch:core-4.0-amd64:core-4.0-noarch:graphics-4.0-amd64:graphics-4.0-noarch:printing-4.0-amd64:printing-4.0-noarch
Red Hat Enterprise Linux Workstation release 6.5 (Santiago)
Red Hat Enterprise Linux Workstation release 6.5 (Santiago)


Which version are you using?  (run 'go version')
go1.2.src.tar.gz


Please provide any additional information below.
Ideally these values would not only apply for the cmd/dist and bootstrapping, but
anywhere that CC is invoked on this system.

Since the CC value used for build of ./make.bash is persisted, it may be worth having an
optional CC_FINAL, in the case that CC was a build-time only explicit path.
@dsymonds
Copy link
Member

@dsymonds dsymonds commented Dec 4, 2013

Comment 1:

Labels changed: removed priority-triage.

@rsc
Copy link
Contributor

@rsc rsc commented Dec 4, 2013

Comment 2:

Labels changed: added release-none, removed go1.3maybe.

@rsc
Copy link
Contributor

@rsc rsc commented Dec 4, 2013

Comment 3:

Labels changed: added repo-main.

@gopherbot
Copy link
Author

@gopherbot gopherbot commented Mar 12, 2014

Comment 4:

This issue was closed by revision b87c772.

Status changed to Fixed.

@golang golang locked and limited conversation to collaborators Jun 25, 2016
This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.