diff mbox series

mmc-utils: Change version string to git hash

Message ID 28623fa7390242e9b003a44016c0235f@hyperstone.com (mailing list archive)
State New, archived
Headers show
Series mmc-utils: Change version string to git hash | expand

Commit Message

Christian Loehle April 12, 2023, 3:04 p.m. UTC
Since mmc-utils versioning has always been 0.1 and changes are not
made frequently and formal releases do not happen, packaged versions
are just pulled from git anyway, using the git hash for version is
probably the best idea for now.

Signed-off-by: Christian Loehle <cloehle@hyperstone.com>
---
 Makefile | 3 ++-
 mmc.c    | 6 ++----
 2 files changed, 4 insertions(+), 5 deletions(-)

Comments

Ulf Hansson April 17, 2023, 8:30 a.m. UTC | #1
On Wed, 12 Apr 2023 at 17:04, Christian Loehle <CLoehle@hyperstone.com> wrote:
>
> Since mmc-utils versioning has always been 0.1 and changes are not
> made frequently and formal releases do not happen, packaged versions
> are just pulled from git anyway, using the git hash for version is
> probably the best idea for now.
>
> Signed-off-by: Christian Loehle <cloehle@hyperstone.com>

Seems reasonable to me, applied, thanks!

Kind regards
Uffe

> ---
>  Makefile | 3 ++-
>  mmc.c    | 6 ++----
>  2 files changed, 4 insertions(+), 5 deletions(-)
>
> diff --git a/Makefile b/Makefile
> index aa27ff2..d8d59a4 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -1,6 +1,7 @@
>  CC ?= gcc
> +GIT_VERSION := "$(shell git describe --abbrev=6 --always --tags)"
>  AM_CFLAGS = -D_FILE_OFFSET_BITS=64 -D_FORTIFY_SOURCE=2
> -CFLAGS ?= -g -O2
> +CFLAGS ?= -g -O2 -DVERSION=\"$(GIT_VERSION)\"
>  objects = \
>         mmc.o \
>         mmc_cmds.o \
> diff --git a/mmc.c b/mmc.c
> index b9aa478..795b4e3 100644
> --- a/mmc.c
> +++ b/mmc.c
> @@ -26,8 +26,6 @@
>
>  #include "mmc_cmds.h"
>
> -#define MMC_VERSION    "0.1"
> -
>  #define BASIC_HELP 0
>  #define ADVANCED_HELP 1
>
> @@ -319,7 +317,7 @@ static void help(char *np)
>
>         printf("\n\t%s help|--help|-h\n\t\tShow the help.\n",np);
>         printf("\n\t%s <cmd> --help\n\t\tShow detailed help for a command or subset of commands.\n",np);
> -       printf("\n%s\n", MMC_VERSION);
> +       printf("\n%s\n", VERSION);
>  }
>
>  static int split_command(char *cmd, char ***commands)
> @@ -500,7 +498,7 @@ static int parse_args(int argc, char **argv,
>         }
>
>         if(helprequested){
> -               printf("\n%s\n", MMC_VERSION);
> +               printf("\n%s\n", VERSION);
>                 return 0;
>         }
>
> --
> 2.37.3
>
>
> Hyperstone GmbH | Reichenaustr. 39a  | 78467 Konstanz
> Managing Director: Dr. Jan Peter Berns.
> Commercial register of local courts: Freiburg HRB381782
>
diff mbox series

Patch

diff --git a/Makefile b/Makefile
index aa27ff2..d8d59a4 100644
--- a/Makefile
+++ b/Makefile
@@ -1,6 +1,7 @@ 
 CC ?= gcc
+GIT_VERSION := "$(shell git describe --abbrev=6 --always --tags)"
 AM_CFLAGS = -D_FILE_OFFSET_BITS=64 -D_FORTIFY_SOURCE=2
-CFLAGS ?= -g -O2
+CFLAGS ?= -g -O2 -DVERSION=\"$(GIT_VERSION)\"
 objects = \
 	mmc.o \
 	mmc_cmds.o \
diff --git a/mmc.c b/mmc.c
index b9aa478..795b4e3 100644
--- a/mmc.c
+++ b/mmc.c
@@ -26,8 +26,6 @@ 
 
 #include "mmc_cmds.h"
 
-#define MMC_VERSION	"0.1"
-
 #define BASIC_HELP 0
 #define ADVANCED_HELP 1
 
@@ -319,7 +317,7 @@  static void help(char *np)
 
 	printf("\n\t%s help|--help|-h\n\t\tShow the help.\n",np);
 	printf("\n\t%s <cmd> --help\n\t\tShow detailed help for a command or subset of commands.\n",np);
-	printf("\n%s\n", MMC_VERSION);
+	printf("\n%s\n", VERSION);
 }
 
 static int split_command(char *cmd, char ***commands)
@@ -500,7 +498,7 @@  static int parse_args(int argc, char **argv,
 	}
 
 	if(helprequested){
-		printf("\n%s\n", MMC_VERSION);
+		printf("\n%s\n", VERSION);
 		return 0;
 	}