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

Jump to bottom

misc/cgo/testgodefs: failing on linux-arm-arm5spacemonkey #35536

Closed
bcmills opened this issue Nov 12, 2019 · 3 comments
Closed

misc/cgo/testgodefs: failing on linux-arm-arm5spacemonkey #35536

bcmills opened this issue Nov 12, 2019 · 3 comments
Labels
FrozenDueToAge NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one.
Milestone
Go1.14

Comments

@bcmills
Copy link
Contributor

bcmills commented Nov 12, 2019

Now that #32407 is resolved, on to the next failure!

This one also appears to be due to the read-only GOROOT (#30316), so before proceeding I plan to see if I can fix that on the other linux builders.

##### ../misc/cgo/testgodefs
./test.bash: line 17: anonunion_defs.go: Permission denied
./test.bash: line 17: issue8478_defs.go: Permission denied
./test.bash: line 17: fieldtypedef_defs.go: Permission denied
# misc/cgo/testgodefs/testdata
./main.go:8:8: undefined: T
./main.go:12:10: undefined: Issue8478
2019/11/12 10:47:19 Failed: exit status 2
2019/11/12 10:47:25 FAILED
@bcmills bcmills added the NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. label Nov 12, 2019
@bcmills bcmills added this to the Go1.14 milestone Nov 12, 2019
@mengzhuo
Copy link
Contributor

I changed builder user to "gopher" after I saw the "run by root" message but it failed linux/mips64le too.
https://build.golang.org/log/fa1c19c15a7a5ddf418d576cb2a5ba4c375c54c0

However it's fine when I run them manually.

+ FILE_PREFIXES='anonunion issue8478 fieldtypedef'
+ cd testdata
+ RM=
+ for FP in '$FILE_PREFIXES'
+ go tool cgo -godefs -srcdir . anonunion.go
+ RM=' anonunion_defs.go'
+ for FP in '$FILE_PREFIXES'
+ go tool cgo -godefs -srcdir . issue8478.go
+ RM=' anonunion_defs.go issue8478_defs.go'
+ for FP in '$FILE_PREFIXES'
+ go tool cgo -godefs -srcdir . fieldtypedef.go
+ RM=' anonunion_defs.go issue8478_defs.go fieldtypedef_defs.go'
+ go build -o testgodefs .
+ ./testgodefs
+ EXIT=0
+ rm -rf _obj testgodefs anonunion_defs.go issue8478_defs.go fieldtypedef_defs.go
+ exit 0

@bcmills
Copy link
Contributor Author

bcmills commented Nov 13, 2019

Yeah, there are currently some tests that fail with a read-only filesystem. I'm hoping to fix them this week.

@gopherbot
Copy link

Change https://golang.org/cl/206901 mentions this issue: misc/cgo/testgodefs: convert test from bash to Go

@golang golang locked and limited conversation to collaborators Nov 12, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
FrozenDueToAge NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one.
Projects
None yet
Milestone
Go1.14
Development

No branches or pull requests
3 participants
@mengzhuo @bcmills @gopherbot