Skip to content

Cannot build for GOARCH=386 #663

@mamachanko

Description

@mamachanko

What steps did you take:

Build for, say, windows/386:

GOOS=windows GOARCH=386 go build ./...

What happened:

Int overflow. 🤖 💀

# carvel.dev/imgpkg/pkg/imgpkg/registry
pkg/imgpkg/registry/registry.go:202:36: cannot use 9999999999 (untyped int constant) as int value in argument to rand.Intn (overflows)

What did you expect:

It builds w/o problems.

Anything else you would like to add:

imgpkg is not distributed for */386. That's not something we need to change. However, since carvel.dev/imgpkg can be consumed as a library, it should be compatible.

Environment:

  • imgpkg version (use imgpkg --version): n/a
  • Docker registry used (e.g. Docker HUB): n/a
  • OS: windows/386

Vote on this request

This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.

👍 "I would like to see this addressed as soon as possible"
👎 "There are other more important things to focus on right now"

We are also happy to receive and review Pull Requests if you want to help working on this issue.

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugThis issue describes a defect or unexpected behavior

    Type

    No type

    Projects

    Status

    Closed

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions