Merge ~alfonsosanchezbeato/snappy-hwe-snaps/+git/network-manager:remove-link-test into ~snappy-hwe-team/snappy-hwe-snaps/+git/network-manager:network-manager/xenial/1.2.2

Proposed by Alfonso Sanchez-Beato
Status: Merged
Approved by: Simon Fels
Approved revision: 7ff975c272b12be073d11770203d291504c39ef0
Merged at revision: 7176580582dfec3433e0bbd0c85c9b89cb99e8f9
Proposed branch: ~alfonsosanchezbeato/snappy-hwe-snaps/+git/network-manager:remove-link-test
Merge into: ~snappy-hwe-team/snappy-hwe-snaps/+git/network-manager:network-manager/xenial/1.2.2
Diff against target: 42 lines (+24/-1)
2 files modified
run-tests.sh (+24/-0)
src/platform/tests/Makefile.am (+0/-1)
Reviewer Review Type Date Requested Status
System Enablement Bot continuous-integration Approve
Simon Fels Approve
Matteo Croce (community) Approve
Review via email: mp+319036@code.launchpad.net

Commit message

Do not run test-link-linux tests

Do not run test-link-linux, as it crashes in the environment used to
build snaps by launchpad.

Description of the change

Do not run test-link-linux tests

Do not run test-link-linux, as it crashes in the environment used to
build snaps by launchpad.

To post a comment you must log in.
Revision history for this message
System Enablement Bot (system-enablement-ci-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Matteo Croce (teknoraver) :
review: Approve
Revision history for this message
Simon Fels (morphis) wrote :

LGTM

review: Approve
Revision history for this message
System Enablement Bot (system-enablement-ci-bot) wrote :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/run-tests.sh b/run-tests.sh
0new file mode 1007550new file mode 100755
index 0000000..b05bd9c
--- /dev/null
+++ b/run-tests.sh
@@ -0,0 +1,24 @@
1#!/bin/sh -ex
2
3SUDO=
4if [ $(id -u) -ne 0 ]; then
5 SUDO=sudo
6fi
7
8# Install all necessary build dependencies in our bare build environment
9$SUDO apt update
10$SUDO apt install -y --force-yes \
11 autoconf autoconf-archive pkg-config \
12 libtool gcc g++ libc6-dev \
13 intltool gtk-doc-tools libdbus-glib-1-dev libdbus-1-dev libiw-dev \
14 libglib2.0-dev libnl-3-dev libnl-route-3-dev libnl-genl-3-dev \
15 libnss3-dev libgnutls28-dev libgcrypt11-dev uuid-dev systemd \
16 libsystemd-dev libudev-dev libgudev-1.0-dev libgirepository1.0-dev \
17 gobject-introspection libglib2.0-doc libmm-glib-dev libndp-dev \
18 libreadline-dev libnewt-dev dbus-test-runner isc-dhcp-client \
19 python-dbus python-gi iptables ppp-dev
20
21./autogen.sh
22./configure
23make -j$(nproc)
24make check
diff --git a/src/platform/tests/Makefile.am b/src/platform/tests/Makefile.am
index 5af4ce9..3878910 100644
--- a/src/platform/tests/Makefile.am
+++ b/src/platform/tests/Makefile.am
@@ -125,7 +125,6 @@ TESTS = \
125 test-cleanup-linux \125 test-cleanup-linux \
126 test-general \126 test-general \
127 test-link-fake \127 test-link-fake \
128 test-link-linux \
129 test-nmp-object \128 test-nmp-object \
130 test-route-fake \129 test-route-fake \
131 test-route-linux130 test-route-linux

Subscribers

People subscribed via source and target branches