Merge ~mkukri/ubuntu/+source/phpmyadmin:merge into ubuntu/+source/phpmyadmin:debian/sid

Proposed by Mate Kukri
Status: Merged
Merge reported by: Mate Kukri
Merged at revision: 7d68c02eb8cf6bcd972b607f038c2828ed53ed1a
Proposed branch: ~mkukri/ubuntu/+source/phpmyadmin:merge
Merge into: ubuntu/+source/phpmyadmin:debian/sid
Diff against target: 157 lines (+118/-1)
4 files modified
debian/changelog (+13/-0)
debian/control (+2/-1)
debian/patches/Require-PHP-8.0.patch (+102/-0)
debian/patches/series (+1/-0)
Reviewer Review Type Date Requested Status
William Desportes (community) Approve
Simon Quigley (community) Approve
Review via email: mp+456374@code.launchpad.net

Commit message

Merge with Debian

To post a comment you must log in.
Revision history for this message
William Desportes (williamdes) wrote :

As far as I know normally all patches from symfony are dropped from Ubuntu
Said in an other way: if symfony has no Ubuntu diff, phpMyAdmin should not have any in Ubuntu

review: Disapprove
Revision history for this message
Mate Kukri (mkukri) wrote :

> As far as I know normally all patches from symfony are dropped from Ubuntu
> Said in an other way: if symfony has no Ubuntu diff, phpMyAdmin should not
> have any in Ubuntu

Should this be just synced from Debian as is?

Revision history for this message
Mate Kukri (mkukri) wrote :

See LP: #2044817

Revision history for this message
Mate Kukri (mkukri) wrote :

#ubuntu-devel discussion led to this being uploaded, there is still desire to eventually have this synced.

Revision history for this message
Simon Quigley (tsimonq2) :
review: Approve
Revision history for this message
Athos Ribeiro (athos-ribeiro) wrote :

Hi William,

I believe we still carry the PSR stack which causes the issues that originally led to this delta being added.

This will only be fixed once the symfony transition is over in debian and gets sync'd back in Ubuntu. Hopefully we will be able to drop this delta for noble though :)

Revision history for this message
William Desportes (williamdes) wrote (last edit ):

Thank you for clearing this up for me, you can go ahead then

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/debian/changelog b/debian/changelog
index b871191..9f9482d 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,10 @@
1phpmyadmin (4:5.2.1+dfsg-2ubuntu1) noble; urgency=medium
2
3 * Merge with Debian unstable. Remaining changes:
4 - Require PHP >= 8.0 (LP #2016016)
5
6 -- Mate Kukri <mate.kukri@canonical.com> Mon, 27 Nov 2023 15:44:19 +0000
7
1phpmyadmin (4:5.2.1+dfsg-2) unstable; urgency=medium8phpmyadmin (4:5.2.1+dfsg-2) unstable; urgency=medium
29
3 * Allow symfony 610 * Allow symfony 6
@@ -14,6 +21,12 @@ phpmyadmin (4:5.2.1+dfsg-2) unstable; urgency=medium
1421
15 -- William Desportes <williamdes@wdes.fr> Wed, 18 Oct 2023 21:55:32 +020022 -- William Desportes <williamdes@wdes.fr> Wed, 18 Oct 2023 21:55:32 +0200
1623
24phpmyadmin (4:5.2.1+dfsg-1ubuntu1) mantic; urgency=medium
25
26 * Require PHP >= 8.0 (LP: #2016016)
27
28 -- William Desportes <williamdes@wdes.fr> Fri, 14 Apr 2023 18:11:15 +0200
29
17phpmyadmin (4:5.2.1+dfsg-1) unstable; urgency=medium30phpmyadmin (4:5.2.1+dfsg-1) unstable; urgency=medium
1831
19 * Add a d/pkg-php-tools-overrides to force the PHP 8.0 polyfill dep32 * Add a d/pkg-php-tools-overrides to force the PHP 8.0 polyfill dep
diff --git a/debian/control b/debian/control
index 2c0fedb..3400cf0 100644
--- a/debian/control
+++ b/debian/control
@@ -1,5 +1,6 @@
1Source: phpmyadmin1Source: phpmyadmin
2Maintainer: phpMyAdmin Packaging Team <team+phpmyadmin@tracker.debian.org>2Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
3XSBC-Original-Maintainer: phpMyAdmin Packaging Team <team+phpmyadmin@tracker.debian.org>
3Uploaders: Felipe Sateler <fsateler@debian.org>,4Uploaders: Felipe Sateler <fsateler@debian.org>,
4 Matthias Blümel <debian@blaimi.de>,5 Matthias Blümel <debian@blaimi.de>,
5 William Desportes <williamdes@wdes.fr>6 William Desportes <williamdes@wdes.fr>
diff --git a/debian/patches/Require-PHP-8.0.patch b/debian/patches/Require-PHP-8.0.patch
6new file mode 1006447new file mode 100644
index 0000000..b81b7e3
--- /dev/null
+++ b/debian/patches/Require-PHP-8.0.patch
@@ -0,0 +1,102 @@
1From: William Desportes <williamdes@wdes.fr>
2Date: Fri, 14 Apr 2023 18:06:39 +0200
3Subject: Require PHP >= 8.0
4
5Due to the delta carried in the Ubuntu PSR stack, phpmyadmin does not run with
6PHP versions < 8. This patch makes it fail gracefully instead of allowing
7phpmyadmin to abruptly crash, allowing us to provide useful error messages so
8users understand what their next steps should be towards getting a functional
9phpmyadmin deployment.
10
11Bug: https://github.com/phpmyadmin/phpmyadmin/issues/17503
12Bug-Ubuntu: https://bugs.launchpad.net/bugs/2016016
13Origin: vendor
14Forwarded: not-needed
15Reviewed-by: Athos Ribeiro <athos.ribeiro@canonical.com>
16---
17 index.php | 9 +++++++--
18 js/messages.php | 9 +++++++--
19 setup/lib/common.inc.php | 9 +++++++--
20 url.php | 9 +++++++--
21 4 files changed, 28 insertions(+), 8 deletions(-)
22
23diff --git a/index.php b/index.php
24index 26fcabf..8831d47 100644
25--- a/index.php
26+++ b/index.php
27@@ -11,8 +11,13 @@ if (! defined('ROOT_PATH')) {
28 // phpcs:enable
29 }
30
31-if (PHP_VERSION_ID < 70205) {
32- die('<p>PHP 7.2.5+ is required.</p><p>Currently installed version is: ' . PHP_VERSION . '</p>');
33+if (PHP_VERSION_ID < 80000) {
34+ die(
35+ '<p>PHP 8.0.0+ is required ('
36+ . '<a href="https://bugs.launchpad.net/ubuntu/+source/symfony/+bug/1975892">Ubuntu Launchpad bug #1975892</a>'
37+ . '&nbsp;and&nbsp;<a href="https://github.com/phpmyadmin/phpmyadmin/issues/17503">phpMyAdmin issue #17503</a>'
38+ . ').</p><p>Currently installed version is: ' . PHP_VERSION . '</p>'
39+ );
40 }
41
42 // phpcs:disable PSR1.Files.SideEffects
43diff --git a/js/messages.php b/js/messages.php
44index 927226b..1e2b249 100644
45--- a/js/messages.php
46+++ b/js/messages.php
47@@ -15,8 +15,13 @@ if (! defined('ROOT_PATH')) {
48 // phpcs:enable
49 }
50
51-if (PHP_VERSION_ID < 70205) {
52- die('<p>PHP 7.2.5+ is required.</p><p>Currently installed version is: ' . PHP_VERSION . '</p>');
53+if (PHP_VERSION_ID < 80000) {
54+ die(
55+ '<p>PHP 8.0.0+ is required ('
56+ . '<a href="https://bugs.launchpad.net/ubuntu/+source/symfony/+bug/1975892">Ubuntu Launchpad bug #1975892</a>'
57+ . '&nbsp;and&nbsp;<a href="https://github.com/phpmyadmin/phpmyadmin/issues/17503">phpMyAdmin issue #17503</a>'
58+ . ').</p><p>Currently installed version is: ' . PHP_VERSION . '</p>'
59+ );
60 }
61
62 // phpcs:disable PSR1.Files.SideEffects
63diff --git a/setup/lib/common.inc.php b/setup/lib/common.inc.php
64index 17adc52..b681566 100644
65--- a/setup/lib/common.inc.php
66+++ b/setup/lib/common.inc.php
67@@ -6,8 +6,13 @@ use PhpMyAdmin\Common;
68 use PhpMyAdmin\Config\ConfigFile;
69 use PhpMyAdmin\DatabaseInterface;
70
71-if (PHP_VERSION_ID < 70205) {
72- die('<p>PHP 7.2.5+ is required.</p><p>Currently installed version is: ' . PHP_VERSION . '</p>');
73+if (PHP_VERSION_ID < 80000) {
74+ die(
75+ '<p>PHP 8.0.0+ is required ('
76+ . '<a href="https://bugs.launchpad.net/ubuntu/+source/symfony/+bug/1975892">Ubuntu Launchpad bug #1975892</a>'
77+ . '&nbsp;and&nbsp;<a href="https://github.com/phpmyadmin/phpmyadmin/issues/17503">phpMyAdmin issue #17503</a>'
78+ . ').</p><p>Currently installed version is: ' . PHP_VERSION . '</p>'
79+ );
80 }
81
82 if (! defined('PHPMYADMIN')) {
83diff --git a/url.php b/url.php
84index 0274481..ebf69df 100644
85--- a/url.php
86+++ b/url.php
87@@ -11,8 +11,13 @@ if (! defined('ROOT_PATH')) {
88 // phpcs:enable
89 }
90
91-if (PHP_VERSION_ID < 70205) {
92- die('<p>PHP 7.2.5+ is required.</p><p>Currently installed version is: ' . PHP_VERSION . '</p>');
93+if (PHP_VERSION_ID < 80000) {
94+ die(
95+ '<p>PHP 8.0.0+ is required ('
96+ . '<a href="https://bugs.launchpad.net/ubuntu/+source/symfony/+bug/1975892">Ubuntu Launchpad bug #1975892</a>'
97+ . '&nbsp;and&nbsp;<a href="https://github.com/phpmyadmin/phpmyadmin/issues/17503">phpMyAdmin issue #17503</a>'
98+ . ').</p><p>Currently installed version is: ' . PHP_VERSION . '</p>'
99+ );
100 }
101
102 // phpcs:disable PSR1.Files.SideEffects
diff --git a/debian/patches/series b/debian/patches/series
index 941b5c1..c152c80 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -17,3 +17,4 @@ Fix-a-case-where-a-fatal-error-message-was-not-displayed.patch
17Fix-sql-parser-tests-for-5.8.patch17Fix-sql-parser-tests-for-5.8.patch
18Fixes-for-sql-parser-5.8.patch18Fixes-for-sql-parser-5.8.patch
19Force-depends-on-sql-parser-5.8.patch19Force-depends-on-sql-parser-5.8.patch
20Require-PHP-8.0.patch

Subscribers

People subscribed via source and target branches