Merge ~ahasenack/ubuntu/+source/uwsgi-plugin-php:focal-uwsgi-plugin-php-ftbfs into ubuntu/+source/uwsgi-plugin-php:ubuntu/devel

Proposed by Andreas Hasenack
Status: Merged
Approved by: Andreas Hasenack
Approved revision: 9c16524904e563eee8c8ccdc62a6e82eeea85e4f
Merged at revision: 9c16524904e563eee8c8ccdc62a6e82eeea85e4f
Proposed branch: ~ahasenack/ubuntu/+source/uwsgi-plugin-php:focal-uwsgi-plugin-php-ftbfs
Merge into: ubuntu/+source/uwsgi-plugin-php:ubuntu/devel
Diff against target: 42 lines (+10/-2)
3 files modified
debian/changelog (+7/-0)
debian/control (+2/-1)
debian/rules (+1/-1)
Reviewer Review Type Date Requested Status
Bryce Harrington (community) Approve
Canonical Server MOTU reviewers Pending
Review via email: mp+380440@code.launchpad.net

Description of the change

uwsgi --build-plugin ends up calling a python script to build the plugin. This script comes from the uwsgi-src source. I checked and I believe it's python3 compatible, specially because it has checks like these in it (uwsgiconfig.py):
"""
try:
    import ConfigParser
except:
    import configparser as ConfigParser

PY3 = sys.version_info[0] == 3
"""

See the linked bug for my troubleshooting on this issue.

I didn't add a build dependency on python3 because uwsgi-dev, a b-d of this package, already has it.

PPA: https://launchpad.net/~ahasenack/+archive/ubuntu/uwsgi-plugin-php-ftbfs
sudo add-apt-repository ppa:ahasenack/uwsgi-plugin-php-ftbfs -y -u

To post a comment you must log in.
Revision history for this message
Andreas Hasenack (ahasenack) wrote :

Oh, my mistake, looks like uwsgi-dev pulls in python2, not python3:
$ dpkg -s uwsgi-dev|grep Depends
Depends: libpcre3-dev, libssl-dev, uuid-dev, zlib1g-dev, libcap-dev, binutils, gcc, python2, uwsgi-core, libc6-dev | libc-dev

Let me fix that.

Revision history for this message
Bryce Harrington (bryce) wrote :

Looks good, verified it installs properly, and that it pulls in php7.4 as expected, rather than php7.3.

Thanks for the fix, that was a good find.

review: Approve
Revision history for this message
Andreas Hasenack (ahasenack) wrote :

Thanks, tagging and uploading 9c16524904e563eee8c8ccdc62a6e82eeea85e4f

$ git push pkg upload/0.0.4ubuntu1
Enumerating objects: 18, done.
Counting objects: 100% (18/18), done.
Delta compression using up to 4 threads
Compressing objects: 100% (10/10), done.
Writing objects: 100% (13/13), 1.41 KiB | 1.41 MiB/s, done.
Total 13 (delta 6), reused 0 (delta 0)
To ssh://git.launchpad.net/~usd-import-team/ubuntu/+source/uwsgi-plugin-php
 * [new tag] upload/0.0.4ubuntu1 -> upload/0.0.4ubuntu1

$ dput ubuntu ../uwsgi-plugin-php_0.0.4ubuntu1_source.changes
Checking signature on .changes
gpg: ../uwsgi-plugin-php_0.0.4ubuntu1_source.changes: Valid signature from AC983EB5BF6BCBA9
Checking signature on .dsc
gpg: ../uwsgi-plugin-php_0.0.4ubuntu1.dsc: Valid signature from AC983EB5BF6BCBA9
Uploading to ubuntu (via ftp to upload.ubuntu.com):
  Uploading uwsgi-plugin-php_0.0.4ubuntu1.dsc: done.
  Uploading uwsgi-plugin-php_0.0.4ubuntu1.tar.xz: done.
  Uploading uwsgi-plugin-php_0.0.4ubuntu1_source.buildinfo: done.
  Uploading uwsgi-plugin-php_0.0.4ubuntu1_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 9730b85..135f00f 100644
3--- a/debian/changelog
4+++ b/debian/changelog
5@@ -1,3 +1,10 @@
6+uwsgi-plugin-php (0.0.4ubuntu1) focal; urgency=medium
7+
8+ * d/rules: be explicit about the python interpreter we want the
9+ plugin builder to use (LP: #1866693)
10+
11+ -- Andreas Hasenack <andreas@canonical.com> Mon, 09 Mar 2020 19:29:10 +0000
12+
13 uwsgi-plugin-php (0.0.4build1) focal; urgency=medium
14
15 * No-change rebuild for php7.4
16diff --git a/debian/control b/debian/control
17index 5e6a58a..e288d67 100644
18--- a/debian/control
19+++ b/debian/control
20@@ -1,7 +1,8 @@
21 Source: uwsgi-plugin-php
22 Section: httpd
23 Priority: optional
24-Maintainer: uWSGI packaging team <pkg-uwsgi-devel@lists.alioth.debian.org>
25+Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
26+XSBC-Original-Maintainer: uWSGI packaging team <pkg-uwsgi-devel@lists.alioth.debian.org>
27 Uploaders: Jonas Smedegaard <dr@jones.dk>
28 Build-Depends:
29 debhelper,
30diff --git a/debian/rules b/debian/rules
31index a426234..9055315 100755
32--- a/debian/rules
33+++ b/debian/rules
34@@ -26,7 +26,7 @@ OUR_BINARY_VERSION = $(subst -,+,$(UWSGI_VERSION))+$(DEB_VERSION)
35 dh $@ --with uwsgi
36
37 override_dh_auto_build:
38- uwsgi --build-plugin /usr/src/uwsgi/plugins/php
39+ PYTHON=python2 uwsgi --build-plugin /usr/src/uwsgi/plugins/php
40 help2man \
41 --name 'fast (pure C), self-healing, developer-friendly WSGI server' \
42 --section 1 \

Subscribers

People subscribed via source and target branches