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

Subscribers

People subscribed via source and target branches