From ff145bca17e782e654e859862acca7ccaf399db2 Mon Sep 17 00:00:00 2001
From: Michael Wiebusch <m.wiebusch@gsi.de>
Date: Mon, 9 Feb 2015 17:10:59 +0100
Subject: [PATCH] Fixed bug and improved compile time feature: Only time of
 last code change is recorded as compile time, not every 'make program'

---
 .gitignore              |  1 +
 firmware/src/Makefile   | 16 ++++++++--------
 firmware/src/main.c     | 10 ++--------
 firmware/src/version.c  |  1 -
 firmware/src/version.h  | 14 +++++++++-----
 firmware/src/version.sh | 14 +++++++++-----
 6 files changed, 29 insertions(+), 27 deletions(-)

diff --git a/.gitignore b/.gitignore
index 9ce83ac..6874f33 100644
--- a/.gitignore
+++ b/.gitignore
@@ -13,3 +13,4 @@
 *.info_symbol
 
 .kateproject*
+*.kate-swp
diff --git a/firmware/src/Makefile b/firmware/src/Makefile
index 00c6d12..a4a4fe5 100644
--- a/firmware/src/Makefile
+++ b/firmware/src/Makefile
@@ -11,22 +11,22 @@ OBJS+=spi2.o
 OBJS+=spi3.o
 OBJS+=dac.o
 OBJS+=misc_utils.o
-OBJS+=version.o
+# OBJS+=version.o
 
 # OBJS+=keypins.o
 
 
 all: src
 
-src: app.a 
+src: version.h app.a 
 
-app.a: version $(OBJS)
-	$(AR) cr app.a $(OBJS) 
+app.a: $(OBJS) version.o
+	$(AR) cr app.a $(OBJS) version.o
 
-.PHONY: src clean 
 clean: 
-	rm -f app.a $(OBJS)
+	rm -f app.a $(OBJS) version.o
 	
-.PHONY:
-version:
+version.h: $(OBJS)
 	./version.sh
+
+.PHONY: src clean
\ No newline at end of file
diff --git a/firmware/src/main.c b/firmware/src/main.c
index 310b138..14bb7f6 100644
--- a/firmware/src/main.c
+++ b/firmware/src/main.c
@@ -25,14 +25,12 @@
 
 
 
-
 extern uint16_t SPI_DMA_Buffer[];
 extern uint16_t SPI_DMA_Buffer_In[];
 extern uint16_t uC_regs[];
 
 
 
-
 // prototypes for functions that are included in main.c
 
 // void SysTick_Handler(void);
@@ -44,7 +42,6 @@ extern uint16_t uC_regs[];
 void USART1_IRQHandler(void);
 
 
-
 // MAIN ---------------------------------------------------------------------
 int main(int argc, char *argv[]) {
   
@@ -65,7 +62,6 @@ int main(int argc, char *argv[]) {
   init_SPI2();
   init_SPI3();
 
-
   init_USART1();
   init_USART3();
   
@@ -90,10 +86,8 @@ int main(int argc, char *argv[]) {
     }
     
   }
-
-	return 0;
-
-
+  
+  return 0;
 }
 
 
diff --git a/firmware/src/version.c b/firmware/src/version.c
index 41e687e..b4e97c4 100644
--- a/firmware/src/version.c
+++ b/firmware/src/version.c
@@ -24,4 +24,3 @@ void write_timestamp(void){
   uC_regs[0x10] = (COMPILE_HOUR<<6) | (COMPILE_MINUTE);
   
 }
-
diff --git a/firmware/src/version.h b/firmware/src/version.h
index f3ef0b9..3cb6fde 100644
--- a/firmware/src/version.h
+++ b/firmware/src/version.h
@@ -7,11 +7,15 @@
  ********************************************************
  */
 
-#define COMPILE_YEAR   14
-#define COMPILE_MONTH  06
-#define COMPILE_DAY    10
-#define COMPILE_HOUR   11
-#define COMPILE_MINUTE 42
+// Do not use these constants directly in other source files
+// than "version.c". They would not be up to date
+// (due to the makefile structure)
+
+#define COMPILE_YEAR   15
+#define COMPILE_MONTH  2
+#define COMPILE_DAY    9
+#define COMPILE_HOUR   17
+#define COMPILE_MINUTE 7
 
 void write_timestamp(void);
 
diff --git a/firmware/src/version.sh b/firmware/src/version.sh
index ecde1c7..b729f8f 100755
--- a/firmware/src/version.sh
+++ b/firmware/src/version.sh
@@ -14,11 +14,15 @@ cat <<EOF > version.h
  ********************************************************
  */
 
-#define COMPILE_YEAR   $(date '+%y')
-#define COMPILE_MONTH  $(date '+%m')
-#define COMPILE_DAY    $(date '+%d')
-#define COMPILE_HOUR   $(date '+%H')
-#define COMPILE_MINUTE $(date '+%M')
+// Do not use these constants directly in other source files
+// than "version.c". They would not be up to date
+// (due to the makefile structure)
+
+#define COMPILE_YEAR   $(date '+%y' | sed 's/^0\+//g')
+#define COMPILE_MONTH  $(date '+%m' | sed 's/^0\+//g')
+#define COMPILE_DAY    $(date '+%d' | sed 's/^0\+//g')
+#define COMPILE_HOUR   $(date '+%H' | sed 's/^0\+//g')
+#define COMPILE_MINUTE $(date '+%M' | sed 's/^0\+//g')
 
 void write_timestamp(void);
 
-- 
2.43.0