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

Gosu 1.8 binary release doesnt match architecture #19

Closed
0rax opened this issue May 11, 2016 · 13 comments
Closed

Gosu 1.8 binary release doesnt match architecture #19

0rax opened this issue May 11, 2016 · 13 comments

Comments

@0rax
Copy link

0rax commented May 11, 2016

Just realised when upgrade the gosu binary in the Gogs Docker Container that the current binary release for the armhf architecture is broken. It seems that the current gosu-armhf binary is exactly the same binary as gosu-amd64.

gosu binary file report
gosu armor check

0rax added a commit to gogs/gogs that referenced this issue May 11, 2016
- drop gosu version back to 1.7 as gosu binary for armhf is broken
- see tianon/gosu#19
@tianon
Copy link
Owner

tianon commented May 11, 2016

Oh, troubling indeed 😰

$ file gosu-*
gosu-amd64:       ELF 64-bit LSB executable, x86-64, version 1 (SYSV), statically linked, stripped
gosu-amd64.asc:   data
gosu-arm64:       ELF 64-bit LSB executable, x86-64, version 1 (SYSV), statically linked, stripped
gosu-arm64.asc:   data
gosu-armel:       ELF 64-bit LSB executable, x86-64, version 1 (SYSV), statically linked, stripped
gosu-armel.asc:   data
gosu-armhf:       ELF 64-bit LSB executable, x86-64, version 1 (SYSV), statically linked, stripped
gosu-armhf.asc:   data
gosu-i386:        ELF 64-bit LSB executable, x86-64, version 1 (SYSV), statically linked, stripped
gosu-i386.asc:    data
gosu-ppc64:       ELF 64-bit LSB executable, x86-64, version 1 (SYSV), statically linked, stripped
gosu-ppc64.asc:   data
gosu-ppc64el:     ELF 64-bit LSB executable, x86-64, version 1 (SYSV), statically linked, stripped
gosu-ppc64el.asc: data

@0rax
Copy link
Author

0rax commented May 11, 2016

Oh yeah :/ Hopefully not a lot of people are relying on anything but the amd64 build :/

@tianon
Copy link
Owner

tianon commented May 11, 2016

I've deleted the binaries and added a note: https://github.com/tianon/gosu/releases/tag/1.8

Now to figure out what's happened 😢

@tianon
Copy link
Owner

tianon commented May 11, 2016

Here's the set of changes: 1.7...1.8#diff-3254677a7917c6c01f55212f86c57fbf

So far my hunch is this one:

-ENV BUILD_FLAGS -v -ldflags -d
+ENV BUILD_FLAGS="-v -ldflags '-d -s -w'"

@tianon
Copy link
Owner

tianon commented May 11, 2016

Nope, still no good after reverting that. Might be the switch to Alpine, or strange interaction WRT eval. 😕

@0rax
Copy link
Author

0rax commented May 11, 2016

Weird, will take a quick look at it, keep you informed if I find anything weird

@tianon
Copy link
Owner

tianon commented May 11, 2016

Doh, definitely eval being silly. Will have a fix shortly.

@tianon tianon closed this as completed in 57bff5b May 11, 2016
@tianon
Copy link
Owner

tianon commented May 11, 2016

Much better:

+ file gosu-amd64 gosu-arm64 gosu-armel gosu-armhf gosu-i386 gosu-ppc64 gosu-ppc64el
gosu-amd64:   ELF 64-bit LSB executable, x86-64, version 1 (SYSV), statically linked, stripped
gosu-arm64:   ELF 64-bit LSB executable, ARM aarch64, version 1 (SYSV), statically linked, stripped
gosu-armel:   ELF 32-bit LSB executable, ARM, EABI5 version 1 (SYSV), statically linked, stripped
gosu-armhf:   ELF 32-bit LSB executable, ARM, EABI5 version 1 (SYSV), statically linked, stripped
gosu-i386:    ELF 32-bit LSB executable, Intel 80386, version 1 (SYSV), statically linked, stripped
gosu-ppc64:   ELF 64-bit MSB executable, 64-bit PowerPC or cisco 7500, version 1 (SYSV), statically linked, stripped
gosu-ppc64el: ELF 64-bit LSB executable, 64-bit PowerPC or cisco 7500, version 1 (SYSV), statically linked, stripped

@0rax
Copy link
Author

0rax commented May 11, 2016

Thanks, that's perfect.

I was just gonna suggest you to do that :)

@tianon
Copy link
Owner

tianon commented May 11, 2016

I find it really amusing that it took 22 days for anyone to notice, myself included! Especially since the build.sh output includes the contents of the SHA256SUMS file, which were all exactly alike (and should've set off alarm bells in my head).

@tianon
Copy link
Owner

tianon commented May 11, 2016

@0rax
Copy link
Author

0rax commented May 11, 2016

Well yes, that's what surprised me.

To be fair in my case, I try to check if gosu has been updated or not whenever I update somethings related to the Gogs Docker container without following the release themselves. I think a lot of people are not updating it right away.

Thats perfect thanks, will update the version in Gogs !

@tianon
Copy link
Owner

tianon commented May 11, 2016

Yeah, I definitely don't upgrade gosu in all my own images every single time I update here. 😄

unknwon pushed a commit to gogs/gogs that referenced this issue May 11, 2016
* docker: update build script to use glide + make

- docker/build.sh will now use glide to fetch dependencies
- glide is built from source to keep compatibility with arm
  (no pre-prebuilt binary for arm)
- docker/build.sh will also now use the provided Makefile
  It will generate an error when trying to get git build has as we do
  not ship the 88mo .git directory during the build (should not cause
  any problem as the variable it sets was not set previously)

* docker: fix docker arm build

- drop gosu version back to 1.7 as gosu binary for armhf is broken
- see tianon/gosu#19

* docker: update gosu to 1.9

Signed-off-by: Jean-Philippe Roemer <[email protected]>
encodeering added a commit to encodeering/docker-postgres that referenced this issue May 21, 2016
cniweb added a commit to openhab/openhab-docker that referenced this issue Jan 10, 2017
See tianon/gosu#19

Signed-off-by: Christian Häussler <[email protected]> (github: @cniweb)
cniweb added a commit to openhab/openhab-docker that referenced this issue Jan 10, 2017
See tianon/gosu#19

Signed-off-by: Christian Häussler <[email protected]> (github: @cniweb)
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