Skip to content

Conversation

@NicoHaase
Copy link
Contributor

No description provided.

@NicoHaase NicoHaase marked this pull request as draft November 29, 2024 13:10
@NicoHaase NicoHaase force-pushed the patch-1 branch 2 times, most recently from a102e30 to afb4de6 Compare November 29, 2024 13:17
run: |
PHP_VERSION_DOCKERSUFFIX=''
if [ "${{ matrix.php-version }}" = '8.1' ]; then
if [ "${{ matrix.php-version }}" = '8.4' ]; then
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PHP 8.4 is officially available, no need to add the -rc suffix

@mlocati
Copy link
Contributor

mlocati commented Nov 29, 2024

To solve the issue with apt-get, I'd create the file /docker/fix-apt with this content:

#!/bin/sh

if grep -q 'VERSION="8 (jessie)"' /etc/os-release; then
    distro=jessie
elif grep -q 'VERSION="9 (stretch)"' /etc/os-release; then
    distro=stretch
else
    return
fi

deb_archive=http://archive.kernel.org/debian-archive
deb_archive_security=http://archive.kernel.org/debian-archive/debian-security

sed -ri "s;^(\s*deb\s+http://(httpredir|deb).debian.org/debian\s+$distro-updates\b.*);#\1;" /etc/apt/sources.list
sed -ri "s;^(\s*deb\s+)http://(httpredir|deb).debian.org;\1$deb_archive;" /etc/apt/sources.list
sed -ri "s;^(\s*deb\s+)http://security.debian.org/debian-security;\1$deb_archive_security;" /etc/apt/sources.list
sed -ri "s;^(\s*deb\s+)http://security.debian.org;\1$deb_archive_security;" /etc/apt/sources.list
tmp="$(mktemp)"
DEBIAN_FRONTEND=noninteractive apt-get update -q 2>"$tmp"
if grep -qE ' KEYEXPIRED [0-9]' "$tmp"; then
    printf 'APT::Get::AllowUnauthenticated "true";\n' >/etc/apt/apt.conf.d/99unauthenticated
fi
rm "$tmp"

Then I'd make it executable, copied to the Docker image, and executed before the "Update apt cache" step.

@mlocati
Copy link
Contributor

mlocati commented Dec 2, 2024

Superseded by #5

@mlocati mlocati closed this Dec 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants