From: Roger Peppe <rogpeppe@gmail.com>
Date: Wed, 2 Mar 2011 19:22:33 +0000 (-0500)
Subject: cgo: put temporary source files in _obj.
X-Git-Tag: weekly.2011-03-07~56
X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=9704d4ec3d42b5b02ae93ec2300add08f72e9449;p=gostls13.git

cgo: put temporary source files in _obj.
Fixes #1572.
Initially I tried changing things so all object
files get put in _obj, but it's too much - everything
needs changing. Perhaps some other time.

R=rsc
CC=golang-dev
https://golang.org/cl/4237050
---

diff --git a/src/Make.pkg b/src/Make.pkg
index 549936e53a..247a3291ec 100644
--- a/src/Make.pkg
+++ b/src/Make.pkg
@@ -36,7 +36,7 @@ INSTALLFILES+=$(pkgdir)/$(TARG).a
 # The rest of the cgo rules are below, but these variable updates
 # must be done here so they apply to the main rules.
 ifdef CGOFILES
-GOFILES+=$(patsubst %.go,%.cgo1.go,$(CGOFILES)) _cgo_gotypes.go
+GOFILES+=$(patsubst %.go,_obj/%.cgo1.go,$(CGOFILES)) _obj/_cgo_gotypes.go
 CGO_OFILES+=$(patsubst %.go,%.cgo2.o,$(CGOFILES)) _cgo_export.o
 OFILES+=_cgo_defun.$O _cgo_import.$O $(CGO_OFILES)
 endif
@@ -47,8 +47,6 @@ coverage:
 	gotest
 	6cov -g $(shell pwd) $O.out | grep -v '_test\.go:'
 
-CLEANFILES+=*.cgo1.go *.cgo2.c _cgo_defun.c _cgo_gotypes.go _cgo_export.*
-CLEANFILES+=_cgo_.c _cgo_import.c _cgo_main.c _cgo_flags _cgo_run _gcc_main.c
 CLEANFILES+=*.so _obj _test _testmain.go *.exe
 
 test:
@@ -61,7 +59,7 @@ nuke: clean
 	rm -f $(pkgdir)/$(TARG).a
 
 testpackage-clean:
-	rm -f _test/$(TARG).a _gotest_.$O
+	rm -f _test/$(TARG).a
 
 install: $(INSTALLFILES)
 
@@ -103,33 +101,33 @@ dir:
 # x.go and y.go.
 
 # Cgo translates each x.go file listed in $(CGOFILES) into a basic
-# translation of x.go, called x.cgo1.go. Additionally, three other
+# translation of x.go, called _obj/x.cgo1.go. Additionally, three other
 # files are created:
 #
-#	_cgo_gotypes.go	- declarations needed for all .go files in the package; imports "unsafe"
-#	_cgo_defun.c	- C trampoline code to be compiled with 6c and linked into the package
-#	x.cgo2.c	- C implementations compiled with gcc to create a dynamic library
+#	_obj/_cgo_gotypes.go	- declarations needed for all .go files in the package; imports "unsafe"
+#	_obj/_cgo_defun.c	- C trampoline code to be compiled with 6c and linked into the package
+#	_obj/x.cgo2.c	- C implementations compiled with gcc to create a dynamic library
 #
 
 ifdef CGOFILES
-_cgo_run: $(CGOFILES)
+_obj/_cgo_run: $(CGOFILES)
 	CGOPKGPATH=$(dir) cgo -- $(CGO_CFLAGS) $(CGOFILES)
-	touch _cgo_run
+	touch _obj/_cgo_run
 
 # _CGO_CFLAGS and _CGO_LDFLAGS are defined via the evaluation of _cgo_flags.
 # The include happens before the commands in the recipe run,
 # so it cannot be done in the same recipe that runs cgo.
-_load_cgo_flags: _cgo_run
-	$(eval include _cgo_flags)
+_obj/_load_cgo_flags: _obj/_cgo_run
+	$(eval include _obj/_cgo_flags)
 
 # Include any previous flags in case cgo files are up to date.
--include _cgo_flags
+-include _obj/_cgo_flags
 
 # Ugly but necessary - cgo writes these files too.
-_cgo_gotypes.go _cgo_export.c _cgo_export.h _cgo_main.c _cgo_defun.c: _load_cgo_flags
+_obj/_cgo_gotypes.go _obj/_cgo_export.c _obj/_cgo_export.h _obj/_cgo_main.c _obj/_cgo_defun.c: _obj/_load_cgo_flags
 	@true
 
-%.cgo1.go %.cgo2.c: _cgo_defun.c
+_obj/%.cgo1.go _obj/%.cgo2.c: _obj/_cgo_defun.c
 	@true
 endif
 
@@ -137,6 +135,9 @@ endif
 %.o: %.c
 	$(HOST_CC) $(_CGO_CFLAGS_$(GOARCH)) -g -fPIC -O2 -o $@ -c $(CGO_CFLAGS) $(_CGO_CFLAGS) $*.c
 
+%.o: _obj/%.c
+	$(HOST_CC) $(_CGO_CFLAGS_$(GOARCH)) -I . -g -fPIC -O2 -o $@ -c $(CGO_CFLAGS) $(_CGO_CFLAGS) $^
+
 # To find out which symbols are needed from external libraries
 # and which libraries are needed, we build a simple a.out that
 # links all the objects we just created and then use cgo -dynimport
@@ -145,14 +146,11 @@ endif
 # After main we have to define all the symbols that will be provided
 # by Go code.  That's crosscall2 and any exported symbols.
 
-_cgo_main.o: _cgo_main.c
-	$(HOST_CC) $(_CGO_CFLAGS_$(GOARCH)) -g -fPIC -O2 -o $@ -c $(CGO_CFLAGS) $(_CGO_CFLAGS) _cgo_main.c
-
 _cgo1_.o: _cgo_main.o $(CGO_OFILES)
 	$(HOST_CC) $(_CGO_CFLAGS_$(GOARCH)) -g -fPIC -O2 -o $@ $^ $(CGO_LDFLAGS) $(_CGO_LDFLAGS)
 
-_cgo_import.c: _cgo1_.o
-	cgo -dynimport _cgo1_.o >_$@ && mv -f _$@ $@
+_obj/_cgo_import.c: _cgo1_.o
+	cgo -dynimport _cgo1_.o >$@_ && mv -f $@_ $@
 
 # The rules above added x.cgo1.go and _cgo_gotypes.go to $(GOFILES),
 # added _cgo_defun.$O to $OFILES, and added the installed copy of
@@ -170,15 +168,17 @@ _CGO_LDFLAGS_windows=-shared -lm -mthreads
 RUNTIME_CFLAGS=-I$(pkgdir)
 
 # Compile _cgo_defun.c with 6c; needs access to the runtime headers.
-_cgo_defun.$O: _cgo_defun.c
-	$(CC) $(CFLAGS) $(RUNTIME_CFLAGS) _cgo_defun.c
+_cgo_defun.$O: _obj/_cgo_defun.c
+	$(CC) $(CFLAGS) $(RUNTIME_CFLAGS) -I . -o "$@" _obj/_cgo_defun.c
 
 # Generic build rules.
 # These come last so that the rules above can override them
 # for more specific file names.
 %.$O: %.c $(HFILES)
-	$(CC) $(CFLAGS) $*.c
+	$(CC) $(CFLAGS) -o "$@" $*.c
+
+%.$O: _obj/%.c $(HFILES)
+	$(CC) $(CFLAGS) -I . -o "$@" _obj/$*.c
 
 %.$O: %.s
 	$(AS) $*.s
-
diff --git a/src/cmd/cgo/gcc.go b/src/cmd/cgo/gcc.go
index e6ce21ed30..cc570f9cf3 100644
--- a/src/cmd/cgo/gcc.go
+++ b/src/cmd/cgo/gcc.go
@@ -599,7 +599,7 @@ func (p *Package) gccMachine() string {
 	return "-m32"
 }
 
-const gccTmp = "_cgo_.o"
+const gccTmp = "_obj/_cgo_.o"
 
 // gccCmd returns the gcc command line to use for compiling
 // the input.
diff --git a/src/cmd/cgo/main.go b/src/cmd/cgo/main.go
index b15d345278..dbf0664dce 100644
--- a/src/cmd/cgo/main.go
+++ b/src/cmd/cgo/main.go
@@ -215,6 +215,10 @@ func main() {
 		fs[i] = f
 	}
 
+	// make sure that _obj directory exists, so that we can write
+	// all the output files there.
+	os.Mkdir("_obj", 0777)
+
 	for i, input := range goFiles {
 		f := fs[i]
 		p.Translate(f)
diff --git a/src/cmd/cgo/out.go b/src/cmd/cgo/out.go
index ede8f57d85..279a9c15c7 100644
--- a/src/cmd/cgo/out.go
+++ b/src/cmd/cgo/out.go
@@ -20,20 +20,11 @@ import (
 // writeDefs creates output files to be compiled by 6g, 6c, and gcc.
 // (The comments here say 6g and 6c but the code applies to the 8 and 5 tools too.)
 func (p *Package) writeDefs() {
-	// The path for the shared object is slash-free so that ELF loaders
-	// will treat it as a relative path.  We rewrite slashes to underscores.
-	sopath := "cgo_" + strings.Map(slashToUnderscore, p.PackagePath)
-	soprefix := ""
-	if os.Getenv("GOOS") == "darwin" {
-		// OS X requires its own prefix for a relative path
-		soprefix = "@rpath/"
-	}
-
-	fgo2 := creat("_cgo_gotypes.go")
-	fc := creat("_cgo_defun.c")
-	fm := creat("_cgo_main.c")
+	fgo2 := creat("_obj/_cgo_gotypes.go")
+	fc := creat("_obj/_cgo_defun.c")
+	fm := creat("_obj/_cgo_main.c")
 
-	fflg := creat("_cgo_flags")
+	fflg := creat("_obj/_cgo_flags")
 	for k, v := range p.CgoFlags {
 		fmt.Fprintf(fflg, "_CGO_%s=%s\n", k, v)
 	}
@@ -94,7 +85,7 @@ func (p *Package) writeDefs() {
 
 	for _, n := range p.Name {
 		if n.FuncType != nil {
-			p.writeDefsFunc(fc, fgo2, n, soprefix, sopath)
+			p.writeDefsFunc(fc, fgo2, n)
 		}
 	}
 
@@ -195,7 +186,7 @@ func (p *Package) structType(n *Name) (string, int64) {
 	return buf.String(), off
 }
 
-func (p *Package) writeDefsFunc(fc, fgo2 *os.File, n *Name, soprefix, sopath string) {
+func (p *Package) writeDefsFunc(fc, fgo2 *os.File, n *Name) {
 	name := n.Go
 	gtype := n.FuncType.Go
 	if n.AddError {
@@ -271,8 +262,8 @@ func (p *Package) writeOutput(f *File, srcfile string) {
 		base = base[0 : len(base)-3]
 	}
 	base = strings.Map(slashToUnderscore, base)
-	fgo1 := creat(base + ".cgo1.go")
-	fgcc := creat(base + ".cgo2.c")
+	fgo1 := creat("_obj/" + base + ".cgo1.go")
+	fgcc := creat("_obj/" + base + ".cgo2.c")
 
 	p.GoFiles = append(p.GoFiles, base+".cgo1.go")
 	p.GccFiles = append(p.GccFiles, base+".cgo2.c")
@@ -340,7 +331,7 @@ func (p *Package) writeOutputFunc(fgcc *os.File, n *Name) {
 // Write out the various stubs we need to support functions exported
 // from Go so that they are callable from C.
 func (p *Package) writeExports(fgo2, fc, fm *os.File) {
-	fgcc := creat("_cgo_export.c")
+	fgcc := creat("_obj/_cgo_export.c")
 	fgcch := creat("_cgo_export.h")
 
 	fmt.Fprintf(fgcch, "/* Created by cgo - DO NOT EDIT. */\n")