Skip to content

Architecture 32 bit#169

Merged
mengzhuo merged 3 commits intosipeed:mainfrom
PixelTux:architecture-32-bit
Feb 14, 2026
Merged

Architecture 32 bit#169
mengzhuo merged 3 commits intosipeed:mainfrom
PixelTux:architecture-32-bit

Conversation

@PixelTux
Copy link
Collaborator

Related Issues

Fixes:

As requested in #91 (comment) by @mengzhuo, this PR addresses the build failure on 32-bit architectures.

Problem:

../go/pkg/mod/github.com/larksuite/oapi-sdk-go/v3@v3.5.3/service/drive/v1/api_ext.go:30:13: cannot use math.MaxInt64 (untyped int constant 9223372036854775807) as int value in struct literal (overflows)
make: *** [Makefile:68: build] Error 1

pkg/channels/feishu.go depends on github.com/larksuite/oapi-sdk-go, which does not support 32-bit architectures due to usage of math.MaxInt64 where int overflows on 32-bit systems.
As a result, builds fail on 32-bit platforms (e.g., armv7l).

Solution:
Introduce architecture-specific files:

  • feishu_64.go – enabled on 64-bit platforms
  • feishu_32.go – excludes Feishu support on 32-bit platforms

This uses Go build constraints to skip Feishu on unsupported 32-bit architectures, allowing the project to compile successfully.

Testing:
✅ Built successfully on ARMv7 (32-bit) using a Tinker Board S
✅ Ran make build-all on a 64-bit system
✅ No build or installation errors observed

@mengzhuo mengzhuo merged commit 1cb690d into sipeed:main Feb 14, 2026
3 checks passed
emadomedher pushed a commit to emadomedher/picoclaw that referenced this pull request Feb 17, 2026
StarWindv referenced this pull request in StarWindv/PicoClaw-shou Feb 19, 2026
@Orgmar
Copy link
Contributor

Orgmar commented Feb 19, 2026

@PixelTux Thanks for tackling the 32 bit architecture support! That was blocking builds on a bunch of platforms, and fixing the underlying issue rather than just working around it is the right approach.

We have the PicoClaw Dev Group on Discord for contributors to connect. If you're interested, email support@sipeed.com with the subject [Join PicoClaw Dev Group] + Your GitHub account and we'll send you the invite link.

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