Skip to content

fix: Remove hardcoded test version from makefile#205

Merged
DeJayDev merged 3 commits intorunpod:mainfrom
acamq:main
Nov 18, 2025
Merged

fix: Remove hardcoded test version from makefile#205
DeJayDev merged 3 commits intorunpod:mainfrom
acamq:main

Conversation

@acamq
Copy link
Copy Markdown
Contributor

@acamq acamq commented Oct 26, 2025

The makefile 'version' target was writing "1.0.0-test" to a file called "VERSION" (uppercase), which caused issues on case-insensitive filesystems (macOS/Windows) where it would overwrite the git-tracked "version" file (lowercase) that contains the actual version.

The version file is now tracked in git and maintained through version control, so the makefile target now simply validates that the file exists rather than generating it.

This fixes the issue where runpodctl would report version "1.0.0-test" instead of the correct version from the tracked version file. runpodctl version now prints runpodctl v1.14.11 as intended.

Built and tested on:
Ubuntu 24.04.3 LTS with Linux kernel 4.4.0, x86_64
Zorin OS 18 with Linux kernel 6.14.0-33-generic, x86_64
Windows 10 Pro 22H2 x86-64

claude and others added 2 commits October 26, 2025 15:26
The makefile 'version' target was writing "1.0.0-test" to a file
called "VERSION" (uppercase), which caused issues on case-insensitive
filesystems (macOS/Windows) where it would overwrite the git-tracked
"version" file (lowercase) that contains the actual version.

The version file is now tracked in git and maintained through version
control, so the makefile target now simply validates that the file
exists rather than generating it.

This fixes the issue where runpodctl would report version "1.0.0-test"
instead of the correct version from the tracked version file.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
…-011CUW2dr3CvFSkpAkeB9ePh

fix: Remove hardcoded test version from makefile
@acamq acamq changed the title fix: fixed version command fix: Remove hardcoded test version from makefile Oct 26, 2025
@DeJayDev DeJayDev self-assigned this Nov 8, 2025
Copy link
Copy Markdown

@DeJayDev DeJayDev left a comment

Choose a reason for hiding this comment

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

Thank you!

@DeJayDev DeJayDev merged commit 7e26379 into runpod:main Nov 18, 2025
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