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

fixCompilationOutput breaks compilation output #357

Closed
benmoss opened this issue May 31, 2017 · 3 comments
Closed

fixCompilationOutput breaks compilation output #357

benmoss opened this issue May 31, 2017 · 3 comments

Comments

@benmoss
Copy link
Contributor

benmoss commented May 31, 2017

Here's what go build reports:

$ go build ./api
# github.com/pivotal-cf-experimental/davos/database
database/client.go:113: notice.ProductNotices undefined (type ProductNotice has no field or method ProductNotices)

And here's what Ginkgo reports:

$ ginkgo ./api
Failed to compile api:

# github.com/pivotal-cf-experimental/davos/database
api/database/client.go:113: notice.ProductNotices undefined (type ProductNotice has no field or method ProductNotices)

It looks like the problem being solved by the fixCompilationOutput function is causing this problem in turn.

@onsi
Copy link
Owner

onsi commented Jun 1, 2017

Hey @benmoss - what's the problem specifically? fixCompilationOutput exists so that when you run ginkgo -r you get clear locations of all your compilation failures. It basically tries to get you a full relative path from CWD into each failure (i.e. api/database/client.go instead of database/client.go)

@onsi onsi added this to To Do in Is this thing on? Jun 1, 2017
@onsi onsi removed this from To Do in Is this thing on? Jun 1, 2017
@benmoss
Copy link
Contributor Author

benmoss commented Jun 2, 2017

Yeah, our code doesn't live in api/database/client.go, it's just in database/client.go. It seems that Ginkgo is prepending api to the front because the API package is the one I ran, but the compilation failure was in the database package, which api depends on.

@onsi
Copy link
Owner

onsi commented Jun 3, 2017

ok I'll take a look

@onsi onsi closed this as completed in ed94a3a Jun 27, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants