Merge ~athos-ribeiro/ubuntu/+source/php-klogger:merge-lp1995965-kinetic into ubuntu/+source/php-klogger:debian/sid

Proposed by Athos Ribeiro
Status: Merged
Merge reported by: git-ubuntu bot
Merged at revision: 0325d4304114b995e6725098d52ff0a05b1c7c39
Proposed branch: ~athos-ribeiro/ubuntu/+source/php-klogger:merge-lp1995965-kinetic
Merge into: ubuntu/+source/php-klogger:debian/sid
Diff against target: 98 lines (+58/-1)
4 files modified
debian/changelog (+21/-0)
debian/control (+2/-1)
debian/patches/0005-psr-logger-compliance.patch (+34/-0)
debian/patches/series (+1/-0)
Reviewer Review Type Date Requested Status
Bryce Harrington (community) Approve
Canonical Server Reporter Pending
git-ubuntu import Pending
Review via email: mp+432699@code.launchpad.net

Description of the change

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

LGTM, +1

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

Thanks Bryce!

Uploaded. I just realized that the branch name says kinetic, but that is due to my scripts fetching the name of the host series (I usually do this in a ubuntu+1 machine) and the fact that I did not upgraded my laptop to lunar yet.

$ dput ubuntu ../php-klogger_1.2.2-1ubuntu1_source.changes
D: Setting host argument.
Checking signature on .changes
gpg: ../php-klogger_1.2.2-1ubuntu1_source.changes: Valid signature from 033C4CA276024834
Checking signature on .dsc
gpg: ../php-klogger_1.2.2-1ubuntu1.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-klogger_1.2.2-1ubuntu1.dsc: done.
  Uploading php-klogger_1.2.2.orig.tar.gz: done.
  Uploading php-klogger_1.2.2-1ubuntu1.debian.tar.xz: done.
  Uploading php-klogger_1.2.2-1ubuntu1_source.buildinfo: done.
  Uploading php-klogger_1.2.2-1ubuntu1_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 a798d03..0db8a1d 100644
3--- a/debian/changelog
4+++ b/debian/changelog
5@@ -1,3 +1,11 @@
6+php-klogger (1.2.2-1ubuntu1) lunar; urgency=medium
7+
8+ * Merge with Debian unstable (LP: #1995965). Remaining changes:
9+ - d/p/0005-psr-logger-compliance.patch: comply with php-psr-log 3.
10+ (LP #1967796)
11+
12+ -- Athos Ribeiro <athos.ribeiro@canonical.com> Tue, 08 Nov 2022 13:19:14 -0300
13+
14 php-klogger (1.2.2-1) unstable; urgency=medium
15
16 * New upstream version 1.2.2
17@@ -13,6 +21,19 @@ php-klogger (1.2.2-1) unstable; urgency=medium
18
19 -- James Valleroy <jvalleroy@mailbox.org> Mon, 05 Sep 2022 10:39:08 -0400
20
21+php-klogger (1.2.1-4ubuntu1) jammy; urgency=medium
22+
23+ * d/p/0005-psr-logger-compliance.patch: comply with php-psr-log 3.
24+ (LP: #1967796)
25+
26+ -- Athos Ribeiro <athos.ribeiro@canonical.com> Mon, 04 Apr 2022 11:39:42 -0300
27+
28+php-klogger (1.2.1-4build1) jammy; urgency=medium
29+
30+ * No change rebuild for php-psr-log 3
31+
32+ -- Bryce Harrington <bryce@canonical.com> Fri, 25 Feb 2022 06:27:22 +0000
33+
34 php-klogger (1.2.1-4) unstable; urgency=medium
35
36 * d/control: Move to Debian PHP PEAR Maintainers
37diff --git a/debian/control b/debian/control
38index 9f8080c..64d793b 100644
39--- a/debian/control
40+++ b/debian/control
41@@ -1,7 +1,8 @@
42 Source: php-klogger
43 Section: php
44 Priority: optional
45-Maintainer: Debian PHP PEAR Maintainers <pkg-php-pear@lists.alioth.debian.org>
46+Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
47+XSBC-Original-Maintainer: Debian PHP PEAR Maintainers <pkg-php-pear@lists.alioth.debian.org>
48 Uploaders:
49 James Valleroy <jvalleroy@mailbox.org>,
50 Sunil Mohan Adapa <sunil@medhas.org>
51diff --git a/debian/patches/0005-psr-logger-compliance.patch b/debian/patches/0005-psr-logger-compliance.patch
52new file mode 100644
53index 0000000..d6a8a2e
54--- /dev/null
55+++ b/debian/patches/0005-psr-logger-compliance.patch
56@@ -0,0 +1,34 @@
57+Description: Comply with psr/log 3
58+Author: Athos Ribeiro <athos.ribeiro@canonical.com>
59+Forwarded: https://github.com/katzgrau/KLogger/pull/104
60+Bug-Ubuntu: https://bugs.launchpad.net/ubuntu/+source/php-klogger/+bug/1967796
61+Last-Update: 2022-11-08
62+
63+--- a/src/Logger.php
64++++ b/src/Logger.php
65+@@ -204,11 +204,11 @@
66+ * Logs with an arbitrary level.
67+ *
68+ * @param mixed $level
69+- * @param string $message
70++ * @param Stringable|string $message
71+ * @param array $context
72+- * @return null
73++ * @return void
74+ */
75+- public function log($level, $message, array $context = array())
76++ public function log($level, \Stringable|string $message, array $context = array()): void
77+ {
78+ if ($this->logLevels[$this->logLevelThreshold] < $this->logLevels[$level]) {
79+ return;
80+--- a/composer.json
81++++ b/composer.json
82+@@ -5,7 +5,7 @@
83+ "keywords": ["logging"],
84+ "require": {
85+ "php": ">=5.3",
86+- "psr/log": "^1.0.0"
87++ "psr/log": "^3.0.0"
88+ },
89+ "require-dev": {
90+ "phpunit/phpunit": "^6.0.0"
91diff --git a/debian/patches/series b/debian/patches/series
92index 534c9d6..a52a5e7 100644
93--- a/debian/patches/series
94+++ b/debian/patches/series
95@@ -1,2 +1,3 @@
96 0001-Load-psr-log-directly.patch
97 0004-Support-phpunit-v8.patch
98+0005-psr-logger-compliance.patch

Subscribers

People subscribed via source and target branches