Fix validation of man page extension. (#749)

If the extension does not exist or is not a number, the error message is
not raised because the assumptions cause a different exception.

3 files changed, 8 insertions, 1 deletions

diff --git a/mesonbuild/interpreter.py b/mesonbuild/interpreter.py
index abbeaf7..ad381b6 100644
--- a/mesonbuild/interpreter.py
+++ b/mesonbuild/interpreter.py
@@ -454,7 +454,10 @@ class Man(InterpreterObject):
 
     def validate_sources(self):
         for s in self.sources:
-            num = int(s.split('.')[-1])
+            try:
+                num = int(s.split('.')[-1])
+            except (IndexError, ValueError):
+                num = 0
             if num < 1 or num > 8:
                 raise InvalidArguments('Man file must have a file extension of a number between 1 and 8')
 
diff --git a/test cases/failing/31 invalid man extension/meson.build b/test cases/failing/31 invalid man extension/meson.build
new file mode 100644
index 0000000..45eddca
--- /dev/null
+++ b/test cases/failing/31 invalid man extension/meson.build
@@ -0,0 +1,2 @@
+project('man install', 'c')
+m1 = install_man('foo.a1')
diff --git a/test cases/failing/32 no man extension/meson.build b/test cases/failing/32 no man extension/meson.build
new file mode 100644
index 0000000..bf83571
--- /dev/null
+++ b/test cases/failing/32 no man extension/meson.build
@@ -0,0 +1,2 @@
+project('man install', 'c')
+m1 = install_man('foo')