-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Conversation
Signed-off-by: jefby <[email protected]>
@benbjohnson Hi, could you please check the code? Thanks very much. |
👍 thanks! |
add bolt_arm64.go for arm64
const maxMapSize = 0xFFFFFFFFFFFF // 256TB | ||
|
||
// maxAllocSize is the size used when creating array pointers. | ||
const maxAllocSize = 0x7FFFFFFF |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is causing build errors:
src/github.com/boltdb/bolt/bolt_arm64.go|4 error| maxMapSize redeclared in this block previous declaration at src/github.com/boltdb/bolt/bolt_amd64.go:4
src/github.com/boltdb/bolt/bolt_arm64.go|7 error| maxAllocSize redeclared in this block previous declaration at src/github.com/boltdb/bolt/bolt_amd64.go:7
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are you running Go 1.4.x?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll move this discussion to #416.
@benbjohnson Hi,it can work in go 1.5beta3/1.5/1.5rc1, maybe version 1.4.x is not supported arm64. |
It works in 1.5 but the problem is that 1.4 doesn't recognize arm64 as a GOARCH so it doesn't ignore the file. See the other PR for more details. |
Any current plans on merging the arm64 support in bolt? From the discussion, I understood that the build fails for Go 1.4.x. However, I am now trying to build Docker 1.9-dev on arm64 and this patch would be the missing piece. What's the current status? |
Oh and by the way - Go version is 1.5. |
@dodgerblue The issue is that Go 1.4 (and before) don't recognize |
@benbjohnson What about #417? Doesn't that fix the issue? I tested that on my machine and got no compile errors (albeit it's not an amd64, but a i386). So if I were to submit a PR, it would be consisted of #414 and #417. |
@dodgerblue Hmm, yeah, you're right. If you can submit the PR and test it against Go 1.4 & 1.5 then I'll merge it in. 👍 |
@benbjohnson I sent a new PR #435 , added build tag for arm64, I tested it against Go 1.4.2 & 1.5, it works fine. |
I checked other vendors, like https://github.com/kr/pty |
Support arm64 architecture
Signed-off-by: jefby [email protected]