defer os.RemoveAll(dir)
dst := filepath.Join(dir, strings.TrimSuffix(file, ".go"))
- cmd := exec.Command("go", "build", "-o="+dst, path(file))
+ cmd := exec.Command("go", "build", "-gcflags=-L", "-o="+dst, path(file)) // TODO(gri) no need for -gcflags=-L if go tool is adjusted
out, err := cmd.CombinedOutput()
if err == nil {
t.Errorf("expected cgo to fail but it succeeded")
"text/scanner"
"cmd/asm/internal/flags"
+ "cmd/internal/objabi"
"cmd/internal/src"
)
in.Error("unexpected token at end of #line: ", tok)
}
pos := src.MakePos(in.Base(), uint(in.Line()), uint(in.Col()))
- in.Stack.SetBase(src.NewLinePragmaBase(pos, file, uint(line)))
+ in.Stack.SetBase(src.NewLinePragmaBase(pos, file, objabi.AbsFile(objabi.WorkingDir(), file, *flags.TrimPath), uint(line)))
}
// #undef processing
objabi.Flagcount("E", "debug symbol export", &Debug['E'])
objabi.Flagfn1("I", "add `directory` to import search path", addidir)
objabi.Flagcount("K", "debug missing line numbers", &Debug['K'])
+ objabi.Flagcount("L", "show full file names in error messages", &Debug['L'])
objabi.Flagcount("N", "disable optimizations", &Debug['N'])
flag.BoolVar(&Debug_asm, "S", false, "print assembly listing")
objabi.AddVersionFlag() // -V
}
func linestr(pos src.XPos) string {
- return Ctxt.OutermostPos(pos).Format(Debug['C'] == 0)
+ return Ctxt.OutermostPos(pos).Format(Debug['C'] == 0, Debug['L'] == 1)
}
// lasterror keeps track of the most recently issued error.
p.error_at(p.pos_at(line, col+uint(i+1)), "invalid line number: "+nstr)
return
}
- absFile := text[:i]
+ filename := text[:i]
+ absFilename := filename
if p.fileh != nil {
- absFile = p.fileh(absFile)
+ absFilename = p.fileh(filename)
}
- p.base = src.NewLinePragmaBase(src.MakePos(p.base.Pos().Base(), line, col), absFile, uint(n))
+ p.base = src.NewLinePragmaBase(src.MakePos(p.base.Pos().Base(), line, col), filename, absFilename, uint(n))
}
func (p *parser) got(tok token) bool {
if msg := perr.Msg; msg != test.msg {
t.Errorf("%s: got msg = %q; want %q", test.src, msg, test.msg)
}
- if filename := perr.Pos.RelFilename(); filename != test.filename {
+ if filename := perr.Pos.AbsFilename(); filename != test.filename {
t.Errorf("%s: got filename = %q; want %q", test.src, filename, test.filename)
}
if line := perr.Pos.RelLine(); line != test.line+linebase {
afile := src.NewFileBase("a.go", "a.go")
bfile := src.NewFileBase("b.go", "/foo/bar/b.go")
- lfile := src.NewLinePragmaBase(src.MakePos(afile, 7, 0), "linedir", 100)
+ lfile := src.NewLinePragmaBase(src.MakePos(afile, 7, 0), "linedir", "linedir", 100)
var tests = []struct {
pos src.Pos
// Line returns a string containing the filename and line number for p
func (p *Prog) Line() string {
- return p.Ctxt.OutermostPos(p.Pos).Format(false)
+ return p.Ctxt.OutermostPos(p.Pos).Format(false, true)
}
// LineNumber returns a string containing the line number for p's position
func (p Pos) SymFilename() string { return p.base.SymFilename() }
func (p Pos) String() string {
- return p.Format(true)
+ return p.Format(true, true)
}
// Format formats a position as "filename:line" or "filename:line:column",
-// controlled by the showCol flag.
-// If the position is relative to a line directive, the original position
-// is appended in square brackets without column (since the column doesn't
-// change).
-func (p Pos) Format(showCol bool) string {
+// controlled by the showCol flag. A position relative to a line directive
+// is always formatted without column information. In that case, if showOrig
+// is set, the original position (again controlled by showCol) is appended
+// in square brackets: "filename:line[origfile:origline:origcolumn]".
+func (p Pos) Format(showCol, showOrig bool) string {
if !p.IsKnown() {
return "<unknown line number>"
}
// that's provided via a line directive).
// TODO(gri) This may not be true if we have an inlining base.
// We may want to differentiate at some point.
- return format(p.RelFilename(), p.RelLine(), 0, false) +
- "[" + format(p.Filename(), p.Line(), p.Col(), showCol) + "]"
+ s := format(p.RelFilename(), p.RelLine(), 0, false)
+ if showOrig {
+ s += "[" + format(p.Filename(), p.Line(), p.Col(), showCol) + "]"
+ }
+ return s
}
// format formats a (filename, line, col) tuple as "filename:line" (showCol
// NewLinePragmaBase returns a new *PosBase for a line pragma of the form
// //line filename:line
// at position pos.
-func NewLinePragmaBase(pos Pos, filename string, line uint) *PosBase {
- return &PosBase{pos, filename, filename, FileSymPrefix + filename, line - 1, -1}
+func NewLinePragmaBase(pos Pos, filename, absFilename string, line uint) *PosBase {
+ return &PosBase{pos, filename, absFilename, FileSymPrefix + absFilename, line - 1, -1}
}
// NewInliningBase returns a copy of the old PosBase with the given inlining
func TestPos(t *testing.T) {
f0 := NewFileBase("", "")
f1 := NewFileBase("f1", "f1")
- f2 := NewLinePragmaBase(Pos{}, "f2", 10)
- f3 := NewLinePragmaBase(MakePos(f1, 10, 1), "f3", 100)
- f4 := NewLinePragmaBase(MakePos(f3, 10, 1), "f4", 100)
+ f2 := NewLinePragmaBase(Pos{}, "f2", "f2", 10)
+ f3 := NewLinePragmaBase(MakePos(f1, 10, 1), "f3", "f3", 100)
+ f4 := NewLinePragmaBase(MakePos(f3, 10, 1), "f4", "f4", 100)
// line directives from issue #19392
fp := NewFileBase("p.go", "p.go")
- fc := NewLinePragmaBase(MakePos(fp, 3, 0), "c.go", 10)
- ft := NewLinePragmaBase(MakePos(fp, 6, 0), "t.go", 20)
- fv := NewLinePragmaBase(MakePos(fp, 9, 0), "v.go", 30)
- ff := NewLinePragmaBase(MakePos(fp, 12, 0), "f.go", 40)
+ fc := NewLinePragmaBase(MakePos(fp, 3, 0), "c.go", "c.go", 10)
+ ft := NewLinePragmaBase(MakePos(fp, 6, 0), "t.go", "t.go", 20)
+ fv := NewLinePragmaBase(MakePos(fp, 9, 0), "v.go", "v.go", 30)
+ ff := NewLinePragmaBase(MakePos(fp, 12, 0), "f.go", "f.go", 40)
for _, test := range []struct {
pos Pos
func TestConversion(t *testing.T) {
b1 := NewFileBase("b1", "b1")
b2 := NewFileBase("b2", "b2")
- b3 := NewLinePragmaBase(MakePos(b1, 10, 0), "b3", 123)
+ b3 := NewLinePragmaBase(MakePos(b1, 10, 0), "b3", "b3", 123)
var tab PosTable
for _, want := range []Pos{
--- /dev/null
+// run
+
+// Copyright 2017 The Go Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style
+// license that can be found in the LICENSE file.
+
+package main
+
+import (
+ "bytes"
+ "fmt"
+ "io/ioutil"
+ "log"
+ "os"
+ "os/exec"
+ "path/filepath"
+ "runtime"
+ "strings"
+)
+
+func main() {
+ if runtime.GOOS == "nacl" {
+ return // no file system available on builders
+ }
+
+ f, err := ioutil.TempFile("", "issue22660.go")
+ if err != nil {
+ log.Fatal(err)
+ }
+ f.Close()
+ defer os.Remove(f.Name())
+
+ // path must appear in error messages even if we strip them with -trimpath
+ path := filepath.Join("users", "xxx", "go")
+ var src bytes.Buffer
+ fmt.Fprintf(&src, "//line %s:1\n", filepath.Join(path, "foo.go"))
+
+ if err := ioutil.WriteFile(f.Name(), src.Bytes(), 0660); err != nil {
+ log.Fatal(err)
+ }
+
+ out, err := exec.Command("go", "tool", "compile", fmt.Sprintf("-trimpath=%s", path), f.Name()).CombinedOutput()
+ if err == nil {
+ log.Fatalf("expected compiling %s to fail", f.Name())
+ }
+
+ if !strings.HasPrefix(string(out), path) {
+ log.Fatalf("expected full path (%s) in error message, got:\n%s", path, out)
+ }
+}