Skip to content

fix: add linux build tag to hanwen handle_test.go - #451

Merged
javi11 merged 1 commit into
mainfrom
fix/hanwen-handle-test-linux-build-tag
Mar 28, 2026
Merged

fix: add linux build tag to hanwen handle_test.go#451
javi11 merged 1 commit into
mainfrom
fix/hanwen-handle-test-linux-build-tag

Conversation

@javi11

@javi11 javi11 commented Mar 28, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • handle.go has //go:build linux but handle_test.go was missing the matching build constraint
  • On macOS, handle.go is excluded from compilation but the test file was still compiled, causing undefined: NewHandle errors in go test ./...
  • Fix: add //go:build linux to handle_test.go

Test plan

  • Run go test ./... on macOS — no build failures
  • Run go test ./internal/fuse/backend/hanwen/... on Linux — tests compile and pass

🤖 Generated with Claude Code

handle.go has a //go:build linux constraint, but the test file was
missing the matching tag. On macOS the test compiled without the
source, causing "undefined: NewHandle" errors in go test ./...

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@javi11
javi11 merged commit a267f00 into main Mar 28, 2026
2 checks passed
yoshitaka420 pushed a commit to yoshitaka420/altmount that referenced this pull request Jun 1, 2026
Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
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.

1 participant