mirror of https://github.com/xemu-project/xemu.git
tests/docker/docker.py: support --include-executable
When passed the path to a binary we copy it and any linked libraries (if it is dynamically linked) into the docker build context. These can then be included by a dockerfile with the line: # Copy all of context into container ADD . / This is mainly intended for setting up foreign architecture docker images which use qemu-$arch to do cross-architecture linux-user execution. It also relies on the host and guest file-system following reasonable multi-arch layouts so the copied libraries don't clash with the guest ones. Signed-off-by: Alex Bennée <alex.bennee@linaro.org> Message-id: 1468934445-32183-3-git-send-email-famz@redhat.com Signed-off-by: Fam Zheng <famz@redhat.com>
This commit is contained in:
parent
a9f8d03891
commit
504ca3c208
|
@ -20,6 +20,7 @@ import atexit
|
||||||
import uuid
|
import uuid
|
||||||
import argparse
|
import argparse
|
||||||
import tempfile
|
import tempfile
|
||||||
|
import re
|
||||||
from shutil import copy, rmtree
|
from shutil import copy, rmtree
|
||||||
|
|
||||||
def _text_checksum(text):
|
def _text_checksum(text):
|
||||||
|
@ -38,6 +39,54 @@ def _guess_docker_command():
|
||||||
raise Exception("Cannot find working docker command. Tried:\n%s" % \
|
raise Exception("Cannot find working docker command. Tried:\n%s" % \
|
||||||
commands_txt)
|
commands_txt)
|
||||||
|
|
||||||
|
def _copy_with_mkdir(src, root_dir, sub_path):
|
||||||
|
"""Copy src into root_dir, creating sub_path as needed."""
|
||||||
|
dest_dir = os.path.normpath("%s/%s" % (root_dir, sub_path))
|
||||||
|
try:
|
||||||
|
os.makedirs(dest_dir)
|
||||||
|
except OSError:
|
||||||
|
# we can safely ignore already created directories
|
||||||
|
pass
|
||||||
|
|
||||||
|
dest_file = "%s/%s" % (dest_dir, os.path.basename(src))
|
||||||
|
copy(src, dest_file)
|
||||||
|
|
||||||
|
|
||||||
|
def _get_so_libs(executable):
|
||||||
|
"""Return a list of libraries associated with an executable.
|
||||||
|
|
||||||
|
The paths may be symbolic links which would need to be resolved to
|
||||||
|
ensure theright data is copied."""
|
||||||
|
|
||||||
|
libs = []
|
||||||
|
ldd_re = re.compile(r"(/.*/)(\S*)")
|
||||||
|
try:
|
||||||
|
ldd_output = subprocess.check_output(["ldd", executable])
|
||||||
|
for line in ldd_output.split("\n"):
|
||||||
|
search = ldd_re.search(line)
|
||||||
|
if search and len(search.groups()) == 2:
|
||||||
|
so_path = search.groups()[0]
|
||||||
|
so_lib = search.groups()[1]
|
||||||
|
libs.append("%s/%s" % (so_path, so_lib))
|
||||||
|
except subprocess.CalledProcessError:
|
||||||
|
print "%s had no associated libraries (static build?)" % (executable)
|
||||||
|
|
||||||
|
return libs
|
||||||
|
|
||||||
|
def _copy_binary_with_libs(src, dest_dir):
|
||||||
|
"""Copy a binary executable and all its dependant libraries.
|
||||||
|
|
||||||
|
This does rely on the host file-system being fairly multi-arch
|
||||||
|
aware so the file don't clash with the guests layout."""
|
||||||
|
|
||||||
|
_copy_with_mkdir(src, dest_dir, "/usr/bin")
|
||||||
|
|
||||||
|
libs = _get_so_libs(src)
|
||||||
|
if libs:
|
||||||
|
for l in libs:
|
||||||
|
so_path = os.path.dirname(l)
|
||||||
|
_copy_with_mkdir(l , dest_dir, so_path)
|
||||||
|
|
||||||
class Docker(object):
|
class Docker(object):
|
||||||
""" Running Docker commands """
|
""" Running Docker commands """
|
||||||
def __init__(self):
|
def __init__(self):
|
||||||
|
@ -151,6 +200,10 @@ class BuildCommand(SubCommand):
|
||||||
""" Build docker image out of a dockerfile. Arguments: <tag> <dockerfile>"""
|
""" Build docker image out of a dockerfile. Arguments: <tag> <dockerfile>"""
|
||||||
name = "build"
|
name = "build"
|
||||||
def args(self, parser):
|
def args(self, parser):
|
||||||
|
parser.add_argument("--include-executable", "-e",
|
||||||
|
help="""Specify a binary that will be copied to the
|
||||||
|
container together with all its dependent
|
||||||
|
libraries""")
|
||||||
parser.add_argument("tag",
|
parser.add_argument("tag",
|
||||||
help="Image Tag")
|
help="Image Tag")
|
||||||
parser.add_argument("dockerfile",
|
parser.add_argument("dockerfile",
|
||||||
|
@ -168,6 +221,11 @@ class BuildCommand(SubCommand):
|
||||||
# Create a docker context directory for the build
|
# Create a docker context directory for the build
|
||||||
docker_dir = tempfile.mkdtemp(prefix="docker_build")
|
docker_dir = tempfile.mkdtemp(prefix="docker_build")
|
||||||
|
|
||||||
|
# Do we include a extra binary?
|
||||||
|
if args.include_executable:
|
||||||
|
_copy_binary_with_libs(args.include_executable,
|
||||||
|
docker_dir)
|
||||||
|
|
||||||
dkr.build_image(tag, docker_dir, dockerfile,
|
dkr.build_image(tag, docker_dir, dockerfile,
|
||||||
quiet=args.quiet, argv=argv)
|
quiet=args.quiet, argv=argv)
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue