From 8897b5181458af0f4237fbc0344c1ba9ceb9ddcd Mon Sep 17 00:00:00 2001
From: Petr Viktorin <pviktori@redhat.com>
Date: May 14 2013 16:49:05 +0000
Subject: Remove leading zero from IPA_NUM_VERSION


The numeric IPA_NUM_VERSION contained a leading zero, so it was treated
as octal value in Python code instead of decimal.

https://fedorahosted.org/freeipa/ticket/3622

---

diff --git a/Makefile b/Makefile
index 8f4053b..b6f4fa2 100644
--- a/Makefile
+++ b/Makefile
@@ -10,7 +10,7 @@ TARGET ?= master
 
 SUPPORTED_PLATFORM ?= redhat
 
-IPA_NUM_VERSION ?= $(shell printf %02d%02d%02d $(IPA_VERSION_MAJOR) $(IPA_VERSION_MINOR) $(IPA_VERSION_RELEASE))
+IPA_NUM_VERSION ?= $(shell printf %d%02d%02d $(IPA_VERSION_MAJOR) $(IPA_VERSION_MINOR) $(IPA_VERSION_RELEASE))
 
 # After updating the version in VERSION you should run the version-update
 # target.
diff --git a/ipapython/version.py.in b/ipapython/version.py.in
index 9cf8ddb..04cf5f8 100644
--- a/ipapython/version.py.in
+++ b/ipapython/version.py.in
@@ -20,9 +20,26 @@
 # The full version including strings
 VERSION="__VERSION__"
 
+
 # Just the numeric portion of the version so one can do direct numeric
 # comparisons to see if the API is compatible.
+#
+# How NUM_VERSION was generated changed over time:
+# Before IPA 3.1.3, it was simply concatenated decimal numbers:
+#   IPA 2.2.2:  NUM_VERSION=222
+#   IPA 2.2.99: NUM_VERSION=2299 (development version)
+#   IPA 3.1.0:  NUM_VERSION=310
+#   IPA 3.1.3:  NUM_VERSION=313
+# In IPA 3.1.4 and 3.2.0, the version was taken as an octal number due to a bug
+# (https://fedorahosted.org/freeipa/ticket/3622):
+#   IPA 3.1.4:  NUM_VERSION=12356 (octal 030104)
+#   IPA 3.2.0:  NUM_VERSION=12416 (octal 030200)
+# After IPA 3.2.0, it is decimal number where each part has two digits:
+#   IPA 3.2.1:  NUM_VERSION=30201
+#   IPA 3.2.99: NUM_VERSION=30299 (development version)
+#   IPA 3.3.0:  NUM_VERSION=30300
 NUM_VERSION=__NUM_VERSION__
 
+
 # The version of the API.
 API_VERSION=u'__API_VERSION__'