Skip to content

s3-aws: fix build for 386#4642

Merged
wy65701436 merged 1 commit intodistribution:mainfrom
ChenQi1989:fix-386-build
Aug 11, 2025
Merged

s3-aws: fix build for 386#4642
wy65701436 merged 1 commit intodistribution:mainfrom
ChenQi1989:fix-386-build

Conversation

@ChenQi1989
Copy link
Copy Markdown
Contributor

When building for 386, we got the following build error:

registry/storage/driver/s3-aws/s3.go:312:99: cannot use
maxChunkSize (untyped int constant 5368709120) as int value
in argument to getParameterAsInteger (overflows)

This is because the s3_64bit.go is used. Adjust the build tag matching in s3_32bit.go and s3_64bit.go to fix this issue.

When building for 386, we got the following build error:

  registry/storage/driver/s3-aws/s3.go:312:99: cannot use
  maxChunkSize (untyped int constant 5368709120) as int value
  in argument to getParameterAsInteger (overflows)

This is because the s3_64bit.go is used. Adjust the build tag matching
in s3_32bit.go and s3_64bit.go to fix this issue.

Signed-off-by: Chen Qi <Qi.Chen@windriver.com>
@milosgajdos milosgajdos requested review from joaodrp and thaJeztah May 29, 2025 14:07
@milosgajdos
Copy link
Copy Markdown
Member

PTAL @thaJeztah

@milosgajdos milosgajdos requested review from Jamstah, joaodrp and wy65701436 and removed request for joaodrp August 8, 2025 14:20
Copy link
Copy Markdown
Collaborator

@wy65701436 wy65701436 left a comment

Choose a reason for hiding this comment

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

lgtm

@wy65701436 wy65701436 merged commit c54bcb3 into distribution:main Aug 11, 2025
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants