Add optional docker php 8 environment

Bug #1955620 reported by Fergus Whyte
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Fix Released
High
Unassigned

Bug Description

To assist with Updating Mahara to php 8 it would be helpful if there was a php 8 environment

Revision history for this message
Kristina Hoeppner (kris-hoeppner) wrote :
Changed in mahara:
status: New → Confirmed
milestone: none → 22.04.0
importance: Undecided → High
status: Confirmed → In Progress
Changed in mahara:
milestone: 22.04.0 → 22.10.0
Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/c/mahara/+/12322
Committed: https://git.mahara.org/mahara/mahara/commit/7bfda9803d9db4b1e3212f7fc3d33ad3ba5b8cd1
Submitter: "Robert Lyon <email address hidden>"
Branch: main

commit 7bfda9803d9db4b1e3212f7fc3d33ad3ba5b8cd1
Author: Fergus Whyte <email address hidden>
Date: Thu Dec 23 17:55:37 2021 +1300

Bug 1955620: Adding options for development in PHP 7.4, 8, and 8.1 for Docker setup

tl;dr;

* Focal added for PHP7.4
* XDebug working in Focal
* Jammy added for PHP8.1

---

Added Focal for PHP 7.4. This is the minimum version of PHP needed for
Behat tests.

To use Focal set DOCKER_PHP_MAJOR_VERSION in the shell you are working
in.

export DOCKER_PHP_MAJOR_VERSION=7.4

XDebug is now available for Focal on port 9000. For VSCode configure
your launch.json file with path mapping set to the path inside the
container.

i.e.
{
"version": "0.2.0",
"configurations": [
{
"name": "Listen for XDebug in Docker",
"type": "php",
"request": "launch",
"port": 9000,
"pathMappings": {
"/mahara/": "${workspaceFolder}/",
},
"log": true
}
]
}

XDebug is also working for scripts running on the CLI inside
the container.

Added Jammy for PHP 8.1. This is the minimum version of PHP needed for
Behat tests.

To use Focal set DOCKER_PHP_MAJOR_VERSION in the shell you are working
in.

export DOCKER_PHP_MAJOR_VERSION=8.1

Change-Id: Ibd79677ce8ad6a9e9837a6f6049ad36e03fa248b

Revision history for this message
Robert Lyon (robertl-9) wrote :

We now can do docker for 7.2, 7.4, 8.0, and 8.1

Changed in mahara:
status: In Progress → Fix Committed
Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review
Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/c/mahara/+/12981
Committed: https://git.mahara.org/mahara/mahara/commit/6f1fa97f55cae2796ffb090883e78dc3bc4a039c
Submitter: "Gold <email address hidden>"
Branch: main

commit 6f1fa97f55cae2796ffb090883e78dc3bc4a039c
Author: Robert Lyon <email address hidden>
Date: Fri Jul 8 09:35:55 2022 +1200

Bug 1955620: Fixing issue with behat config check

It needs to be after the init step otherwise the config path will
always be false after a successful behat test

Change-Id: I79d58fecfeaf1c1a41961f1cb1930e179a8de994
Signed-off-by: Robert Lyon <email address hidden>

Doris Tam (doristam)
Changed in mahara:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.