-
Notifications
You must be signed in to change notification settings - Fork 117
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
Detect Go binaries and skip some checks for them. #599
Conversation
Does this make sense when Go binaries are built with standard Go macros? @jcajka, could you verify? |
I don't think outright disabling the checks makes sense(as they from quick look on the python check script make sense, even for Go, to me). IIRC we use in fedora pie build mode of Go. That will generate pie binaries. In the linked issue example it should AFAIK in default mode produce static binaries(in C sense). I don't remember from top of my head how "static" pie (elf) C binaries looks like but I would assume that Go ones will look the same. I will double check. |
@Conan-Kudo BTW do you remember to what and where are the "gobuildflags" set nowadays? |
Other issue, uh... more factor, might be that the CGO got disabled for the build of the binaries, IMO the rpmlint can't realistically cater for such manual fine tuning, on other hand should probably handle it more gracefully. |
751c9bc
to
890758d
Compare
@Conan-Kudo What do you think about this pull request? |
I'm not sure it makes sense to incorporate this into rpmlint, given @jcajka's feedback. Actually, I wonder if we might be doing something wrong in openSUSE that we're not getting PIE and debuginfo+build-ids enabled on Go builds there... |
890758d
to
5e1a42a
Compare
Ok, so taking the provided example from the bug and my analysis: We report the following:
So either the provided Go shared library is self-contained (does not link to any shared library) and it's fine or the error message is correct and we can close the reported issue? What should I choose @jcajka ? |
Fixes #598.