Skip to content

Conversation

@ac000
Copy link
Owner

@ac000 ac000 commented Nov 1, 2025

We'll just follow the udev/systemd versioning scheme.

This replaces the version number that was generated via 'git describe' and will facilitate building itsa from an archive where there is no .git/ directory.

It is also trivial to override the version at compile time, e.g.

$ make ITSA_VER=$(git describe --long | tail -c +2)

Which will give a version like '001-0-g31cf44f'

We'll just follow the udev/systemd versioning scheme.

This replaces the version number that was generated via 'git describe'
and will facilitate building itsa from an archive where there is no
.git/ directory.

It is also trivial to override the version at compile time, e.g.

  $ make ITSA_VER=$(git describe --long | tail -c +2)

Which will give a version like '001-0-g31cf44f'

Signed-off-by: Andrew Clayton <ac@sigsegv.uk>
@ac000 ac000 requested a review from Copilot November 1, 2025 17:45
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR implements a static versioning system for the itsa application, replacing the dynamic git-based version generation. The version is now set to "001" following the udev/systemd versioning scheme, making it possible to build from source archives without a git repository while still allowing version overrides at compile time.

Key Changes:

  • Replaced dynamic GIT_VERSION with static ITSA_VER version identifier
  • Added version display to usage information and API info output
  • Updated build system to use the new versioning scheme

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/itsa.c Updated version references from GIT_VERSION to ITSA_VER and added version display in usage and API info functions
src/Makefile Replaced git describe command with static version "001" and updated compiler flags to define ITSA_VER instead of GIT_VERSION

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ac000 ac000 merged commit f9c2b93 into master Nov 1, 2025
8 checks passed
@ac000 ac000 deleted the vers branch November 1, 2025 17:55
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