summaryrefslogtreecommitdiff
path: root/lib
diff options
context:
space:
mode:
authorMassimiliano Culpo <massimiliano.culpo@gmail.com>2020-10-13 18:15:48 +0200
committerGitHub <noreply@github.com>2020-10-13 09:15:48 -0700
commitb84812256dae57d4bccbe46d7f7956585f57c0d6 (patch)
tree280568c1c2bbeef9c996c841539baf09513774f7 /lib
parentc40de7c895aca85075dae7b18391928fd46ff5e8 (diff)
downloadspack-b84812256dae57d4bccbe46d7f7956585f57c0d6.tar.gz
spack-b84812256dae57d4bccbe46d7f7956585f57c0d6.tar.bz2
spack-b84812256dae57d4bccbe46d7f7956585f57c0d6.tar.xz
spack-b84812256dae57d4bccbe46d7f7956585f57c0d6.zip
autotools: add attribute to delete libtool archives .la files (#18850)
* autotools: add attribute to delete libtool archives .la files According to Autotools Mythbuster (https://autotools.io/libtool/lafiles.html) libtool archive files are mostly vestigial, but they might create issues when relocating binary packages as shown in #18694. For GCC specifically, most distributions remove these files with explicit commands: https://git.stg.centos.org/rpms/gcc/blob/master/f/gcc.spec#_1303 Considered all of that, this commit adds an easy way for each AutotoolsPackage to remove every .la file that has been installed. The default, for the time being, is to maintain them - to be consistent with what Spack was doing previously. * autotools: delete libtool archive files by default Following review this commit changes the default for libtool archive files deletion and adds test to verify the behavior.
Diffstat (limited to 'lib')
-rw-r--r--lib/spack/spack/build_systems/autotools.py35
-rw-r--r--lib/spack/spack/test/build_systems.py40
2 files changed, 67 insertions, 8 deletions
diff --git a/lib/spack/spack/build_systems/autotools.py b/lib/spack/spack/build_systems/autotools.py
index e4b9f566ae..21ac535b32 100644
--- a/lib/spack/spack/build_systems/autotools.py
+++ b/lib/spack/spack/build_systems/autotools.py
@@ -2,8 +2,6 @@
# Spack Project Developers. See the top-level COPYRIGHT file for details.
#
# SPDX-License-Identifier: (Apache-2.0 OR MIT)
-
-
import inspect
import os
import os.path
@@ -80,10 +78,25 @@ class AutotoolsPackage(PackageBase):
#: Options to be passed to autoreconf when using the default implementation
autoreconf_extra_args = []
+ #: If False deletes all the .la files in the prefix folder
+ #: after the installation. If True instead it installs them.
+ install_libtool_archives = False
+
+ @property
+ def _removed_la_files_log(self):
+ """File containing the list of remove libtool archives"""
+ build_dir = self.build_directory
+ if not os.path.isabs(self.build_directory):
+ build_dir = os.path.join(self.stage.path, build_dir)
+ return os.path.join(build_dir, 'removed_la_files.txt')
+
@property
def archive_files(self):
"""Files to archive for packages based on autotools"""
- return [os.path.join(self.build_directory, 'config.log')]
+ files = [os.path.join(self.build_directory, 'config.log')]
+ if not self.install_libtool_archives:
+ files.append(self._removed_la_files_log)
+ return files
@run_after('autoreconf')
def _do_patch_config_files(self):
@@ -534,3 +547,19 @@ class AutotoolsPackage(PackageBase):
# Check that self.prefix is there after installation
run_after('install')(PackageBase.sanity_check_prefix)
+
+ @run_after('install')
+ def remove_libtool_archives(self):
+ """Remove all .la files in prefix sub-folders if the package sets
+ ``install_libtool_archives`` to be False.
+ """
+ # If .la files are to be installed there's nothing to do
+ if self.install_libtool_archives:
+ return
+
+ # Remove the files and create a log of what was removed
+ libtool_files = fs.find(str(self.prefix), '*.la', recursive=True)
+ with fs.safe_remove(*libtool_files):
+ fs.mkdirp(os.path.dirname(self._removed_la_files_log))
+ with open(self._removed_la_files_log, mode='w') as f:
+ f.write('\n'.join(libtool_files))
diff --git a/lib/spack/spack/test/build_systems.py b/lib/spack/spack/test/build_systems.py
index 367faeef66..1606ba8ebd 100644
--- a/lib/spack/spack/test/build_systems.py
+++ b/lib/spack/spack/test/build_systems.py
@@ -7,8 +7,8 @@ import glob
import os
import pytest
+import llnl.util.filesystem as fs
import spack.repo
-from llnl.util.filesystem import working_dir
from spack.build_environment import get_std_cmake_args, setup_package
from spack.spec import Spec
from spack.util.executable import which
@@ -30,7 +30,7 @@ def test_affirmative_make_check(directory, config, mock_packages, working_env):
pkg = spack.repo.get(s)
setup_package(pkg, False)
- with working_dir(directory):
+ with fs.working_dir(directory):
assert pkg._has_make_target('check')
pkg._if_make_target_execute('check')
@@ -50,7 +50,7 @@ def test_negative_make_check(directory, config, mock_packages, working_env):
pkg = spack.repo.get(s)
setup_package(pkg, False)
- with working_dir(directory):
+ with fs.working_dir(directory):
assert not pkg._has_make_target('check')
pkg._if_make_target_execute('check')
@@ -71,7 +71,7 @@ def test_affirmative_ninja_check(
pkg = spack.repo.get(s)
setup_package(pkg, False)
- with working_dir(directory):
+ with fs.working_dir(directory):
assert pkg._has_ninja_target('check')
pkg._if_ninja_target_execute('check')
@@ -96,7 +96,7 @@ def test_negative_ninja_check(directory, config, mock_packages, working_env):
pkg = spack.repo.get(s)
setup_package(pkg, False)
- with working_dir(directory):
+ with fs.working_dir(directory):
assert not pkg._has_ninja_target('check')
pkg._if_ninja_target_execute('check')
@@ -182,6 +182,36 @@ class TestAutotoolsPackage(object):
assert '--without-baz' in options
assert '--no-fee' in options
+ def test_libtool_archive_files_are_deleted_by_default(
+ self, mutable_database
+ ):
+ # Install a package that creates a mock libtool archive
+ s = spack.spec.Spec('libtool-deletion')
+ s.concretize()
+ s.package.do_install(explicit=True)
+
+ # Assert the libtool archive is not there and we have
+ # a log of removed files
+ assert not os.path.exists(s.package.libtool_archive_file)
+ search_directory = os.path.join(s.prefix, '.spack')
+ libtool_deletion_log = fs.find(
+ search_directory, 'removed_la_files.txt', recursive=True
+ )
+ assert libtool_deletion_log
+
+ def test_libtool_archive_files_might_be_installed_on_demand(
+ self, mutable_database, monkeypatch
+ ):
+ # Install a package that creates a mock libtool archive,
+ # patch its package to preserve the installation
+ s = spack.spec.Spec('libtool-deletion')
+ s.concretize()
+ monkeypatch.setattr(s.package, 'install_libtool_archives', True)
+ s.package.do_install(explicit=True)
+
+ # Assert libtool archives are installed
+ assert os.path.exists(s.package.libtool_archive_file)
+
@pytest.mark.usefixtures('config', 'mock_packages')
class TestCMakePackage(object):