Reduce usage of __DATE__

To increase the reproducibility of builds
we shouldn't use __DATE__. However, for
the development builds there is some demand
for leaving this in.

So as suggested by Gert Doering go for a
compromise where we only use __DATE__ if
we also include the git information. This
will remove this information from release
builds, but not from builds done directly
from the git checkout.

Signed-off-by: Frank Lichtenheld <frank@lichtenheld.com>
Acked-by: Gert Doering <gert@greenie.muc.de>
Message-Id: <20220804150301.62856-2-frank@lichtenheld.com>
URL: https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg24807.html
Signed-off-by: Gert Doering <gert@greenie.muc.de>

diff --git a/src/openvpn/options.c b/src/openvpn/options.c
index 3eae14a7524743b5dc7b375c25f7b8a0ef9606cc..0ce3158bf1de34eb2edbfa3e03b9712dd458961e 100644 (file)
--- a/src/openvpn/options.c
+++ b/src/openvpn/options.c
@@ -110,7 +110,9 @@ const char title_string[] =
 #ifdef ENABLE_DCO
     " [DCO]"
 #endif
+#ifdef CONFIGURE_GIT_REVISION
     " built on " __DATE__
+#endif
 ;
 
 #ifndef ENABLE_SMALL

diff --git a/src/tapctl/main.c b/src/tapctl/main.c
index df73a965a07e45a0af68f2e2134b84ceb1099305..a98cc6a828792b44a1eb9424b6a8cebed8837dc6 100644 (file)
--- a/src/tapctl/main.c
+++ b/src/tapctl/main.c
@@ -44,7 +44,6 @@
 
 const TCHAR title_string[] =
     TEXT(PACKAGE_NAME) TEXT(" ") TEXT(PACKAGE_VERSION)
-    TEXT(" built on ") TEXT(__DATE__)
 ;
 
 static const TCHAR usage_message[] =