From d918ae0bde903a2041a4c9d9edef5c59c63925cf Mon Sep 17 00:00:00 2001 From: Massimiliano Culpo Date: Thu, 13 Apr 2023 17:09:34 +0200 Subject: containerize: strip binaries in a less aggressive way (#36683) --- lib/spack/spack/test/container/docker.py | 11 +++++++++++ lib/spack/spack/test/container/singularity.py | 11 +++++++++++ share/spack/templates/container/Dockerfile | 2 +- share/spack/templates/container/singularity.def | 2 +- 4 files changed, 24 insertions(+), 2 deletions(-) diff --git a/lib/spack/spack/test/container/docker.py b/lib/spack/spack/test/container/docker.py index 57c00dddde..4f65dfd983 100644 --- a/lib/spack/spack/test/container/docker.py +++ b/lib/spack/spack/test/container/docker.py @@ -116,3 +116,14 @@ def test_error_message_invalid_os(minimal_configuration): minimal_configuration["spack"]["container"]["images"]["os"] = "invalid:1" with pytest.raises(ValueError, match="invalid operating system"): writers.create(minimal_configuration) + + +@pytest.mark.regression("34629,18030") +def test_not_stripping_all_symbols(minimal_configuration): + """Tests that we are not stripping all symbols, so that libraries can still be + used for linking. + """ + minimal_configuration["spack"]["container"]["strip"] = True + content = writers.create(minimal_configuration)() + assert "xargs strip" in content + assert "xargs strip -s" not in content diff --git a/lib/spack/spack/test/container/singularity.py b/lib/spack/spack/test/container/singularity.py index d3cd2788e4..06aec57c8e 100644 --- a/lib/spack/spack/test/container/singularity.py +++ b/lib/spack/spack/test/container/singularity.py @@ -40,3 +40,14 @@ def test_singularity_specific_properties(properties, expected, singularity_confi writer = writers.create(singularity_configuration) for name, value in expected.items(): assert getattr(writer, name) == value + + +@pytest.mark.regression("34629,18030") +def test_not_stripping_all_symbols(singularity_configuration): + """Tests that we are not stripping all symbols, so that libraries can still be + used for linking. + """ + singularity_configuration["spack"]["container"]["strip"] = True + content = writers.create(singularity_configuration)() + assert "xargs strip" in content + assert "xargs strip -s" not in content diff --git a/share/spack/templates/container/Dockerfile b/share/spack/templates/container/Dockerfile index 9116590480..b4aee2ee78 100644 --- a/share/spack/templates/container/Dockerfile +++ b/share/spack/templates/container/Dockerfile @@ -32,7 +32,7 @@ RUN find -L {{ paths.view }}/* -type f -exec readlink -f '{}' \; | \ xargs file -i | \ grep 'charset=binary' | \ grep 'x-executable\|x-archive\|x-sharedlib' | \ - awk -F: '{print $1}' | xargs strip -s + awk -F: '{print $1}' | xargs strip {% endif %} # Modifications to the environment that are necessary to run diff --git a/share/spack/templates/container/singularity.def b/share/spack/templates/container/singularity.def index d5778f0d41..a67d25783d 100644 --- a/share/spack/templates/container/singularity.def +++ b/share/spack/templates/container/singularity.def @@ -37,7 +37,7 @@ EOF xargs file -i | \ grep 'charset=binary' | \ grep 'x-executable\|x-archive\|x-sharedlib' | \ - awk -F: '{print $1}' | xargs strip -s + awk -F: '{print $1}' | xargs strip {% endif %} {% if extra_instructions.build %} {{ extra_instructions.build }} -- cgit v1.2.3-60-g2f50