FAQ
Hello dave@cheney.net (cc: golang-dev@googlegroups.com),

I'd like you to review this change to
https://code.google.com/p/go/


https://codereview.appspot.com/6946078/

Search Discussions

  • Dave at Dec 18, 2012 at 10:32 pm

    On 2012/12/18 21:14:11, dfc wrote:
    Fair enough, great job, I will test today.
    On 19/12/2012, at 0:03, minux wrote:


    On Tue, Dec 18, 2012 at 8:53 PM, minux
    wrote:
    PTAL. I think make.bash approach is better.
    scratch that. GOHOSTARCH is not set by default on ARM builds.
    LGTM on ARMv7.

    pandaboard(~) % echo $GOARM

    pandaboard(~) % go tool dist env
    GOROOT="/home/dfc/go"
    GOBIN="/home/dfc/go/bin"
    GOARCH="arm"
    GOOS="linux"
    GOHOSTARCH="arm"
    GOHOSTOS="linux"
    GOTOOLDIR="/home/dfc/go/pkg/tool/linux_arm"
    GOCHAR="5"
    GOARM="7"

    Will test on the Pi this afternoon.

    https://codereview.appspot.com/6946078/
  • Dave at Dec 20, 2012 at 1:17 am
  • Rsc at Dec 20, 2012 at 7:31 pm
  • Minux Ma at Dec 21, 2012 at 6:40 pm
    *** Submitted as
    https://code.google.com/p/go/source/detail?r=bff02defeb7a ***

    cmd/dist: make GOARM detection better compatible with thumb toolchain
    Fixes issue 4557.

    R=dave, rsc
    CC=golang-dev
    https://codereview.appspot.com/6946078


    https://codereview.appspot.com/6946078/

Related Discussions

Discussion Navigation
viewthread | post
Discussion Overview
groupgolang-dev @
categoriesgo
postedDec 18, '12 at 3:20p
activeDec 21, '12 at 6:40p
posts5
users3
websitegolang.org

3 users in discussion

Dave: 2 posts Minux Ma: 2 posts Rsc: 1 post

People

Translate

site design / logo © 2022 Grokbase