Skip to content

Conversation

@julio-lopez
Copy link
Collaborator

@julio-lopez julio-lopez commented Nov 13, 2025

Rationale: move towards reproducible (hermetic) builds. The -trimpath flag removes absolute file system paths from the resulting executables.

Changes:

  • add -trimpath to KOPIA_BUILD_FLAGS
  • add -trimpath to goreleaser config
  • move KOPIA_BUILD_* var definitions to Makefile

Makefile Outdated
export KOPIA_VERSION_NO_PREFIX=$(KOPIA_VERSION:v%=%)

KOPIA_BUILD_TAGS=
KOPIA_BUILD_FLAGS=-ldflags "-trimpath -s -w -X github.com/kopia/kopia/repo.BuildVersion=$(KOPIA_VERSION_NO_PREFIX) -X github.com/kopia/kopia/repo.BuildInfo=$(shell git rev-parse HEAD) -X github.com/kopia/kopia/repo.BuildGitHubRepo=$(GITHUB_REPOSITORY)"
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

-trimpath is added here

@julio-lopez julio-lopez requested a review from Copilot November 13, 2025 07:31
@julio-lopez julio-lopez force-pushed the poc/ci-trimpath branch 3 times, most recently from a70ee0a to 1ebcf59 Compare November 13, 2025 07:46
@codecov
Copy link

codecov bot commented Nov 13, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.03%. Comparing base (cb455c6) to head (d031c54).
⚠️ Report is 754 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4983      +/-   ##
==========================================
+ Coverage   75.86%   78.03%   +2.16%     
==========================================
  Files         470      548      +78     
  Lines       37301    31417    -5884     
==========================================
- Hits        28299    24516    -3783     
+ Misses       7071     4852    -2219     
- Partials     1931     2049     +118     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@julio-lopez julio-lopez marked this pull request as ready for review November 13, 2025 18:28
@julio-lopez julio-lopez merged commit b7f811c into kopia:master Nov 13, 2025
30 of 31 checks passed
@julio-lopez julio-lopez deleted the poc/ci-trimpath branch November 13, 2025 18:31
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