Skip to content

Commit e71f00f

Browse files
committed
Merge branch 'jx/ci-ubuntu-fix' into maint-2.38
Adjust the GitHub CI to newer ubuntu release. * jx/ci-ubuntu-fix: ci: install python on ubuntu ci: use the same version of p4 on both Linux and macOS ci: remove the pipe after "p4 -V" to catch errors github-actions: run gcc-8 on ubuntu-20.04 image
2 parents ec9816c + 0d3507f commit e71f00f

File tree

3 files changed

+13
-17
lines changed

3 files changed

+13
-17
lines changed

.github/workflows/main.yml

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -227,17 +227,15 @@ jobs:
227227
pool: ubuntu-latest
228228
- jobname: linux-sha256
229229
cc: clang
230-
os: ubuntu
231230
pool: ubuntu-latest
232231
- jobname: linux-gcc
233232
cc: gcc
234233
cc_package: gcc-8
235-
pool: ubuntu-latest
234+
pool: ubuntu-20.04
236235
- jobname: linux-TEST-vars
237236
cc: gcc
238-
os: ubuntu
239237
cc_package: gcc-8
240-
pool: ubuntu-latest
238+
pool: ubuntu-20.04
241239
- jobname: osx-clang
242240
cc: clang
243241
pool: macos-12
@@ -280,7 +278,6 @@ jobs:
280278
- jobname: linux-musl
281279
image: alpine
282280
- jobname: linux32
283-
os: ubuntu32
284281
image: daald/ubuntu32:xenial
285282
- jobname: pedantic
286283
image: fedora

ci/install-dependencies.sh

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -5,17 +5,17 @@
55

66
. ${0%/*}/lib.sh
77

8-
P4WHENCE=https://cdist2.perforce.com/perforce/r$LINUX_P4_VERSION
8+
P4WHENCE=https://cdist2.perforce.com/perforce/r21.2
99
LFSWHENCE=https://github.com/github/git-lfs/releases/download/v$LINUX_GIT_LFS_VERSION
1010
UBUNTU_COMMON_PKGS="make libssl-dev libcurl4-openssl-dev libexpat-dev
1111
tcl tk gettext zlib1g-dev perl-modules liberror-perl libauthen-sasl-perl
1212
libemail-valid-perl libio-socket-ssl-perl libnet-smtp-ssl-perl"
1313

1414
case "$runs_on_pool" in
15-
ubuntu-latest)
15+
ubuntu-*)
1616
sudo apt-get -q update
1717
sudo apt-get -q -y install language-pack-is libsvn-perl apache2 \
18-
$UBUNTU_COMMON_PKGS $CC_PACKAGE
18+
$UBUNTU_COMMON_PKGS $CC_PACKAGE $PYTHON_PACKAGE
1919
mkdir --parents "$P4_PATH"
2020
pushd "$P4_PATH"
2121
wget --quiet "$P4WHENCE/bin.linux26x86_64/p4d"
@@ -40,7 +40,7 @@ macos-*)
4040
mkdir -p $HOME/bin
4141
(
4242
cd $HOME/bin
43-
wget -q "https://cdist2.perforce.com/perforce/r21.2/bin.macosx1015x86_64/helix-core-server.tgz" &&
43+
wget -q "$P4WHENCE/bin.macosx1015x86_64/helix-core-server.tgz" &&
4444
tar -xf helix-core-server.tgz &&
4545
sudo xattr -d com.apple.quarantine p4 p4d 2>/dev/null || true
4646
)
@@ -83,9 +83,9 @@ esac
8383
if type p4d >/dev/null 2>&1 && type p4 >/dev/null 2>&1
8484
then
8585
echo "$(tput setaf 6)Perforce Server Version$(tput sgr0)"
86-
p4d -V | grep Rev.
86+
p4d -V
8787
echo "$(tput setaf 6)Perforce Client Version$(tput sgr0)"
88-
p4 -V | grep Rev.
88+
p4 -V
8989
else
9090
echo >&2 "WARNING: perforce wasn't installed, see above for clues why"
9191
fi

ci/lib.sh

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -226,18 +226,18 @@ export GIT_TEST_CLONE_2GB=true
226226
export SKIP_DASHED_BUILT_INS=YesPlease
227227

228228
case "$runs_on_pool" in
229-
ubuntu-latest)
229+
ubuntu-*)
230230
if test "$jobname" = "linux-gcc-default"
231231
then
232232
break
233233
fi
234234

235-
if [ "$jobname" = linux-gcc ]
235+
PYTHON_PACKAGE=python2
236+
if test "$jobname" = linux-gcc
236237
then
237-
MAKEFLAGS="$MAKEFLAGS PYTHON_PATH=/usr/bin/python3"
238-
else
239-
MAKEFLAGS="$MAKEFLAGS PYTHON_PATH=/usr/bin/python2"
238+
PYTHON_PACKAGE=python3
240239
fi
240+
MAKEFLAGS="$MAKEFLAGS PYTHON_PATH=/usr/bin/$PYTHON_PACKAGE"
241241

242242
export GIT_TEST_HTTPD=true
243243

@@ -246,7 +246,6 @@ ubuntu-latest)
246246
# were recorded in the Homebrew database upon creating the OS X
247247
# image.
248248
# Keep that in mind when you encounter a broken OS X build!
249-
export LINUX_P4_VERSION="16.2"
250249
export LINUX_GIT_LFS_VERSION="1.5.2"
251250

252251
P4_PATH="$HOME/custom/p4"

0 commit comments

Comments
 (0)