Skip to content

Some issues seem to have been broken in the build / install process. - #9

Merged
dpage merged 1 commit into
mainfrom
fix/build-process-fixes
Jan 30, 2026
Merged

Some issues seem to have been broken in the build / install process.#9
dpage merged 1 commit into
mainfrom
fix/build-process-fixes

Conversation

@bonesmoses

Copy link
Copy Markdown
Member
  • go.mod now specifies the point version of go to avoid toolchain errors.

    Without the full version spelled out, some versions of go can report a "toolchain not available" error during the make build step.

  • PREFIX can now be overridden by an environment variable.

    The PREFIX environment setting was being ignored, so installs always went to the default location of /usr/local.

* go.mod now specifies the point version of go to avoid toolchain errors.

  Without the full version spelled out, some versions of go can report a
  "toolchain not available" error during the `make build` step.

* PREFIX can now be overridden by an environment variable.

  The PREFIX environment setting was being ignored, so installs always
  went to the default location of /usr/local.
@bonesmoses
bonesmoses requested a review from dpage January 28, 2026 22:26
@coderabbitai

coderabbitai Bot commented Jan 28, 2026

Copy link
Copy Markdown
📝 Walkthrough

Walkthrough

This pull request modifies two files. The Makefile changes the PREFIX variable from a hard-coded assignment to a conditional assignment using the ?= operator, enabling external overrides via environment variables or command-line arguments while maintaining the /usr/local default. The go.mod file updates the Go language specification from version 1.23 to 1.23.0, adding the patch version component. No changes to dependencies or other build/install logic are included.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title mentions broken build/install issues, which aligns with both the Makefile PREFIX override fix and the go.mod version pinning changes described in the PR.
Description check ✅ Passed The description clearly relates to the changeset by explaining the rationale for both changes: go.mod version pinning to fix toolchain errors and Makefile PREFIX override to allow environment variable control.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@dpage
dpage merged commit e0ea433 into main Jan 30, 2026
6 checks passed
@bonesmoses
bonesmoses deleted the fix/build-process-fixes branch January 30, 2026 18:10
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.

2 participants