Skip to content

Remove android/arm from 1.4/cross since it's nontrivial to get the -marm compiler backend for it #30

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

Merged
merged 1 commit into from
Nov 19, 2014

Conversation

tianon
Copy link
Member

@tianon tianon commented Nov 19, 2014

I think we can safely revisit this later. I'm really keen on getting this working, since it's awesome, but getting over that compiler hump is going to be fun.

For the curious, I was playing with variations on the following to get this to work:

RUN echo 'deb http://http.debian.net/debian sid main' > /etc/apt/sources.list.d/sid.list \
    && dpkg --add-architecture armel \
    && apt-get update && apt-get install -y gcc-arm-linux-gnueabi \
    && rm -rf /var/lib/apt/lists/* \
    && rm /etc/apt/sources.list.d/sid.list

(see also https://packages.debian.org/sid/gcc-arm-linux-gnueabi)

@yosifkit
Copy link
Member

LGTM

@@ -17,6 +16,9 @@ ENV GOLANG_CROSSPLATFORMS \
solaris/amd64 \
windows/386 windows/amd64

# TODO gcc: error: unrecognized command line option '-marm'
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should we add this on top instead?

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh it makes sense when seeing the full diff. LGTM

yosifkit added a commit that referenced this pull request Nov 19, 2014
Remove android/arm from 1.4/cross since it's nontrivial to get the -marm compiler backend for it
@yosifkit yosifkit merged commit a384506 into docker-library:master Nov 19, 2014
@yosifkit yosifkit deleted the remove-android branch November 19, 2014 18:33
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.

3 participants