Skip to content

fix(build): restore static native RE2 for full artifacts - #87

Merged
M09Ic merged 2 commits into
masterfrom
fix/re2-static-release
Aug 7, 2026
Merged

fix(build): restore static native RE2 for full artifacts#87
M09Ic merged 2 commits into
masterfrom
fix/re2-static-release

Conversation

@M09Ic

@M09Ic M09Ic commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Summary

  • upgrade the chainreactors/go-re2 replacement to the revision with static native RE2 support
  • build every aiscan-full release target and GoReleaser artifact with re2_cgo re2_static
  • exercise the Windows native packages and full CLI with the same release tags
  • give standard Windows artifacts an explicit .exe suffix and verify both Windows outputs exist
  • add an embedded nginx fingerprint smoke test to catch RE2/resource regressions

Why

Full release artifacts should use the native RE2 backend without requiring a shared RE2 library at runtime. The CI checks keep the release tags aligned with the tested Windows build and make Windows artifact names deterministic.

Verification

  • go mod tidy -diff
  • go test -count=1 ./core/resources
  • go test -count=1 ./cmd/aiscan
  • CGO_ENABLED=1 go test -count=1 -tags "re2_cgo re2_static" ./agent/... ./pkg/runner/... ./pkg/web/...
  • CGO_ENABLED=1 go test -count=1 -tags "forceposix emptytemplates noembed osusergo netgo full sqlite re2_cgo re2_static" ./cmd/aiscan

@M09Ic
M09Ic merged commit 45d5bfa into master Aug 7, 2026
12 of 13 checks passed
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