Make Autotools respect tools.build:install_strip

Make Autotools strip libraries if requested, consistent with CMake and
Meson, by using the 'install-strip' target instead of 'install'.

Author: pbs3141

conan/tools/gnu/autotools.py

@@ -84,9 +84,9 @@ def make(self, target=None, args=None, makefile=None):
         command = join_arguments([make_program, str_makefile, target, str_args, str_extra_args, jobs])
         self._conanfile.run(command)
 
-    def install(self, args=None, target="install", makefile=None):
+    def install(self, args=None, target=None, makefile=None):
         """
-        This is just an "alias" of ``self.make(target="install")``
+        This is just an "alias" of ``self.make(target="install")`` or ``self.make(target="install-strip")``
 
         :param args: (Optional, Defaulted to ``None``): List of arguments to use for the
                      ``make`` call. By default an argument ``DESTDIR=unix_path(self.package_folder)``
@@ -95,6 +95,11 @@ def install(self, args=None, target="install", makefile=None):
         :param target: (Optional, Defaulted to ``None``): Choose which target to install.
         :param makefile: (Optional, Defaulted to ``None``): Allow specifying a custom makefile to use instead of default "Makefile"
         """
+        if target is None:
+            target = "install"
+            do_strip = self._conanfile.conf.get("tools.build:install_strip", check_type=bool)
+            if do_strip:
+                target += "-strip"
         args = args if args else []
         str_args = " ".join(args)
         if "DESTDIR=" not in str_args:

test/unittests/tools/gnu/autotools_test.py

@@ -33,3 +33,28 @@ def test_source_folder_works(chdir_mock):
     autotools.autoreconf()
     chdir_mock.assert_called_with(autotools, os.path.normpath("/path/to/sources"))
     assert conanfile.command == 'autoreconf -bar foo'
+
+@pytest.mark.parametrize("install_strip", [False, True])
+def test_install_strip(install_strip):
+    """
+    When the configuration `tools.build:install_strip` is set to True,
+    the Autotools install command should invoke the `install-strip` target.
+    """
+
+    settings = MockSettings({"build_type": "Release",
+                             "compiler": "gcc",
+                             "compiler.version": "7",
+                             "os": "Linux",
+                             "arch": "x86_64"})
+    conanfile = ConanFileMock()
+    conanfile.settings = settings
+    conanfile.conf = Conf()
+    conanfile.conf["tools.build:install_strip"] = install_strip
+    conanfile.folders.generators = "."
+    conanfile.folders.set_base_generators(temp_folder())
+    conanfile.folders.set_base_package(temp_folder())
+
+    autotools = Autotools(conanfile)
+    autotools.install()
+
+    assert ('install-strip' in str(conanfile.command)) == install_strip