Quoting Tycho Andersen (tycho.ander...@canonical.com): > criu version 1.3 has been tagged, which has the minimal set of patches to > allow > checkpointing and restoring containers. lxc-test-checkpoint-restore is now > skipped on any version of criu lower than 1.3. > > Signed-off-by: Tycho Andersen <tycho.ander...@canonical.com> > --- > src/tests/lxc-test-checkpoint-restore | 8 +++++++- > 1 file changed, 7 insertions(+), 1 deletion(-) > > diff --git a/src/tests/lxc-test-checkpoint-restore > b/src/tests/lxc-test-checkpoint-restore > index 43068ef..a6777d1 100755 > --- a/src/tests/lxc-test-checkpoint-restore > +++ b/src/tests/lxc-test-checkpoint-restore > @@ -15,7 +15,13 @@ if [ "$(id -u)" != "0" ]; then > exit 1 > fi > > -if [ "$(criu --version | head -n1 | cut -d' ' -f 2)" != "1.3-rc2" ]; then > +vergte() { > + ! [ "$1" = "$(echo "$1\n$2" | sort -V | tail -n1)" ]
A few things here: 1. anyone who has /bin/sh linked to /bin/bash will fail here, because the \n will be printed as '\n' 2. vergte sounds like 'vergion greater than or equal", but looking at how you use it you mean the inverse, verlte? > +} > + > +criu_version="$(criu --version | head -n1 | cut -d' ' -f 2)" > + > +if vergte "$criu_version" "1.3"; then > echo "SKIP: skipping test because no (or wrong) criu installed." > exit 0 > fi > -- > 1.9.1 > > _______________________________________________ > lxc-devel mailing list > lxc-devel@lists.linuxcontainers.org > http://lists.linuxcontainers.org/listinfo/lxc-devel _______________________________________________ lxc-devel mailing list lxc-devel@lists.linuxcontainers.org http://lists.linuxcontainers.org/listinfo/lxc-devel