Merge ~athos-ribeiro/ubuntu/+source/php-symfony-polyfill:merge-lp1973074-kinetic into ubuntu/+source/php-symfony-polyfill:debian/sid

Proposed by Athos Ribeiro
Status: Merged
Merge reported by: Athos Ribeiro
Merged at revision: a3433ef98c68f52ec014fb76f29f3f8a8022f412
Proposed branch: ~athos-ribeiro/ubuntu/+source/php-symfony-polyfill:merge-lp1973074-kinetic
Merge into: ubuntu/+source/php-symfony-polyfill:debian/sid
Diff against target: 160 lines (+120/-1)
4 files modified
debian/changelog (+29/-0)
debian/control (+2/-1)
debian/patches/0003-Fix-QQQQQ-date-format.patch (+88/-0)
debian/patches/series (+1/-0)
Reviewer Review Type Date Requested Status
Bryce Harrington (community) Approve
Canonical Server Pending
git-ubuntu import Pending
Review via email: mp+422221@code.launchpad.net

Description of the change

Merge for php-symfony-polyfill as per LP: #1973074

A PPA with the proposed merge is available at https://launchpad.net/~athos-ribeiro/+archive/ubuntu/lp1973074-merge-php-symfony-polyfill-kinetic/+packages

Local result summary for the autopkgtest suite run:

autopkgtest [11:02:34]: @@@@@@@@@@@@@@@@@@@@ summary
command1 PASS

To post a comment you must log in.
Revision history for this message
Bryce Harrington (bryce) wrote :

LGTM, +1

The tests impacted by libicu changes struck me as a bit fiddly, so not at all surprised it needs more updating.

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

Thanks Bryce!

Uploaded

$ dput ubuntu ../php-symfony-polyfill_1.25.0-2ubuntu1_source.changes
D: Setting host argument.
Checking signature on .changes
gpg: ../php-symfony-polyfill_1.25.0-2ubuntu1_source.changes: Valid signature from 033C4CA276024834
Checking signature on .dsc
gpg: ../php-symfony-polyfill_1.25.0-2ubuntu1.dsc: Valid signature from 033C4CA276024834
Package includes an .orig.tar.gz file although the debian revision suggests
that it might not be required. Multiple uploads of the .orig.tar.gz may be
rejected by the upload queue management software.
Uploading to ubuntu (via ftp to upload.ubuntu.com):
  Uploading php-symfony-polyfill_1.25.0-2ubuntu1.dsc: done.
  Uploading php-symfony-polyfill_1.25.0.orig.tar.gz: done.
  Uploading php-symfony-polyfill_1.25.0-2ubuntu1.debian.tar.xz: done.
  Uploading php-symfony-polyfill_1.25.0-2ubuntu1_source.buildinfo: done.
  Uploading php-symfony-polyfill_1.25.0-2ubuntu1_source.changes: done.
Successfully uploaded packages.

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 1f6166c..4f1fc0f 100644
3--- a/debian/changelog
4+++ b/debian/changelog
5@@ -1,3 +1,16 @@
6+php-symfony-polyfill (1.25.0-2ubuntu1) kinetic; urgency=medium
7+
8+ * Merge with Debian unstable (LP: #1973074). Remaining changes:
9+ - d/p/0003-Fix-QQQQQ-date-format.patch: add support to libicu70.
10+ LP #1964709
11+ * Dropped changes:
12+ - d/p/revert-update-intl-currencies-data.patch:
13+ Add patch to revert upstream's commit for
14+ updating intl currencies data.
15+ [ Removed due to LP #1964709 ]
16+
17+ -- Athos Ribeiro <athos.ribeiro@canonical.com> Thu, 12 May 2022 10:47:21 -0300
18+
19 php-symfony-polyfill (1.25.0-2) unstable; urgency=medium
20
21 * Team upload.
22@@ -15,6 +28,22 @@ php-symfony-polyfill (1.25.0-1) unstable; urgency=medium
23
24 -- David Prévot <taffit@debian.org> Mon, 07 Mar 2022 00:32:13 +0100
25
26+php-symfony-polyfill (1.24.0-1ubuntu2) jammy; urgency=medium
27+
28+ * d/patches: libicu70 and PHP 8.1 adjustments. (LP: #1964709)
29+ - Drop Ubuntu patch to allow PHP 8.1 symfony support.
30+ - d/p/0002-Fix-QQQQQ-date-format.patch: add support to libicu70.
31+
32+ -- Athos Ribeiro <athos.ribeiro@canonical.com> Thu, 10 Mar 2022 22:17:19 -0300
33+
34+php-symfony-polyfill (1.24.0-1ubuntu1) jammy; urgency=medium
35+
36+ * d/p/revert-update-intl-currencies-data.patch:
37+ Add patch to revert upstream's commit for
38+ updating intl currencies data.
39+
40+ -- Utkarsh Gupta <utkarsh@ubuntu.com> Fri, 21 Jan 2022 04:56:16 +0530
41+
42 php-symfony-polyfill (1.24.0-1) unstable; urgency=medium
43
44 [ Nicolas Grekas ]
45diff --git a/debian/control b/debian/control
46index 207c2f8..b0ad20c 100644
47--- a/debian/control
48+++ b/debian/control
49@@ -1,7 +1,8 @@
50 Source: php-symfony-polyfill
51 Section: php
52 Priority: optional
53-Maintainer: Debian PHP PEAR Maintainers <pkg-php-pear@lists.alioth.debian.org>
54+Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
55+XSBC-Original-Maintainer: Debian PHP PEAR Maintainers <pkg-php-pear@lists.alioth.debian.org>
56 Uploaders: Daniel Beyer <dabe@deb.ymc.ch>, David Prévot <taffit@debian.org>
57 Build-Depends: debhelper-compat (= 13),
58 dh-sequence-phpcomposer,
59diff --git a/debian/patches/0003-Fix-QQQQQ-date-format.patch b/debian/patches/0003-Fix-QQQQQ-date-format.patch
60new file mode 100644
61index 0000000..0f3876a
62--- /dev/null
63+++ b/debian/patches/0003-Fix-QQQQQ-date-format.patch
64@@ -0,0 +1,88 @@
65+From ade8d6d02c5ca9860b02fac04918b73117cf4d2e Mon Sep 17 00:00:00 2001
66+From: Athos Ribeiro <athos.ribeiro@canonical.com>
67+Date: Sat, 12 Mar 2022 14:48:08 -0300
68+Subject: [PATCH] Fix QQQQQ date format
69+
70+As per [1], the QQQQQ and qqqqq quarter patterns are supposed to convert
71+quarter into numbers (1, 2, 3, or 4). Until vertion 70.1, ICU carried a
72+bug [2] which would make those patterns to convert into the wrong string,
73+mimicking the behavior of the QQQQ pattern. The issue has been fixed
74+upstream [3] and released in ICU 70.1.
75+
76+This patch adjusts the pattern resolution according to the ICU version
77+being used by intl.
78+
79+[1] https://unicode-org.github.io/icu/userguide/format_parse/datetime/#date-field-symbol-table
80+[2] https://unicode-org.atlassian.net/browse/ICU-21647
81+[3] https://github.com/unicode-org/icu/commit/dcfdaca46c1e3cc08764ad1d85f321ad0c27ae16
82+
83+Signed-off-by: Athos Ribeiro <athos.ribeiro@canonical.com>
84+
85+Bug-Ubuntu: https://bugs.launchpad.net/ubuntu/+source/php-symfony-polyfill/+bug/1964709
86+Forwarded: https://github.com/symfony/polyfill/pull/399
87+Last-Update: 2022-03-13
88+---
89+ src/Intl/Icu/DateFormat/QuarterTransformer.php | 10 +++++++++-
90+ tests/Intl/Icu/AbstractIntlDateFormatterTest.php | 10 ++++++++--
91+ 2 files changed, 17 insertions(+), 3 deletions(-)
92+
93+diff --git a/src/Intl/Icu/DateFormat/QuarterTransformer.php b/src/Intl/Icu/DateFormat/QuarterTransformer.php
94+index 4291a72..76d37cf 100644
95+--- a/src/Intl/Icu/DateFormat/QuarterTransformer.php
96++++ b/src/Intl/Icu/DateFormat/QuarterTransformer.php
97+@@ -33,10 +33,18 @@ class QuarterTransformer extends Transformer
98+ return $this->padLeft($quarter, $length);
99+ case 3:
100+ return 'Q'.$quarter;
101+- default:
102++ case 4:
103+ $map = [1 => '1st quarter', 2 => '2nd quarter', 3 => '3rd quarter', 4 => '4th quarter'];
104+
105+ return $map[$quarter];
106++ default:
107++ if (\defined('INTL_ICU_VERSION') && version_compare(\INTL_ICU_VERSION, '70.1', '<')) {
108++ $map = [1 => '1st quarter', 2 => '2nd quarter', 3 => '3rd quarter', 4 => '4th quarter'];
109++
110++ return $map[$quarter];
111++ } else {
112++ return $quarter;
113++ }
114+ }
115+ }
116+
117+diff --git a/tests/Intl/Icu/AbstractIntlDateFormatterTest.php b/tests/Intl/Icu/AbstractIntlDateFormatterTest.php
118+index b23a960..a0a51f8 100644
119+--- a/tests/Intl/Icu/AbstractIntlDateFormatterTest.php
120++++ b/tests/Intl/Icu/AbstractIntlDateFormatterTest.php
121+@@ -81,6 +81,12 @@ abstract class AbstractIntlDateFormatterTest extends TestCase
122+ $dateTime = new \DateTime('@0');
123+ $dateTimeImmutable = new \DateTimeImmutable('@0');
124+
125++ /* https://unicode-org.atlassian.net/browse/ICU-21647 */
126++ $expectedQuarterX5 = '1';
127++ if (\defined('INTL_ICU_VERSION') && version_compare(\INTL_ICU_VERSION, '70.1', '<')) {
128++ $expectedQuarterX5 = '1st quarter';
129++ }
130++
131+ $formatData = [
132+ /* general */
133+ ['y-M-d', 0, '1970-1-1'],
134+@@ -129,13 +135,13 @@ abstract class AbstractIntlDateFormatterTest extends TestCase
135+ ['QQ', 0, '01'],
136+ ['QQQ', 0, 'Q1'],
137+ ['QQQQ', 0, '1st quarter'],
138+- ['QQQQQ', 0, '1st quarter'],
139++ ['QQQQQ', 0, $expectedQuarterX5],
140+
141+ ['q', 0, '1'],
142+ ['qq', 0, '01'],
143+ ['qqq', 0, 'Q1'],
144+ ['qqqq', 0, '1st quarter'],
145+- ['qqqqq', 0, '1st quarter'],
146++ ['qqqqq', 0, $expectedQuarterX5],
147+
148+ // 4 months
149+ ['Q', 7776000, '2'],
150+--
151+2.34.1
152+
153diff --git a/debian/patches/series b/debian/patches/series
154index 4e623ec..08f2f3d 100644
155--- a/debian/patches/series
156+++ b/debian/patches/series
157@@ -1,2 +1,3 @@
158 0001-Adapt-to-installed-path.patch
159 0002-Revert-Indicates-that-polyfills-provides-correspondi.patch
160+0003-Fix-QQQQQ-date-format.patch

Subscribers

People subscribed via source and target branches