Skip to content

Conversation

ccicchitelli
Copy link

@ccicchitelli ccicchitelli commented Jun 7, 2016

The test.go file currently will not run, this update fixes that.

@ccicchitelli ccicchitelli changed the title Updated test.go with changes from Eric Updated test.go to match recent changes Jun 7, 2016
@jpoirier
Copy link

jpoirier commented Jun 8, 2016

fyi - if you remove those flags mpu.go won't build. mpu.go is a Go language wrapper around a library file that was built using C. Go and C have two different ABIs and Go uses a tool called cgo for its cffi (foreign function interface), cgo needs the ld flags so it knows what lib to look for.

@ccicchitelli
Copy link
Author

Yeah I figured that out...i had removed them to try to generate a new error, which eventually it did.

@jpoirier
Copy link

jpoirier commented Jun 8, 2016

yup, saw the updated commit but thought I'd explain why the magical commands are there.

@ccicchitelli
Copy link
Author

thx!!

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

Successfully merging this pull request may close these issues.

2 participants